1berry/pawn-compiler
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

emit/__emit: Avoid excess instruction name copying in emit_findopcode()

Browse Source
This commit is contained in:
Daniel_Cortez 2017-10-30 18:59:21 +07:00
parent d15c11f420
commit af31143ec3
1 changed files with 3 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
source/compiler/sc1.c
View File

@ -6290,14 +6290,10 @@ static EMIT_OPCODE emit_opcodelist[] = {
{ 92, "zero.s", emit_parm1_num }, { 92, "zero.s", emit_parm1_num },
}; };
static int emit_findopcode(char *instr,int maxlen) static int emit_findopcode(const char *instr,int maxlen)
{ {
int low,high,mid,cmp; int low,high,mid,cmp;
char str[MAX_INSTR_LEN];
if (maxlen>=MAX_INSTR_LEN)
return 0;
strlcpy(str,instr,maxlen+1);
/* look up the instruction with a binary search /* look up the instruction with a binary search
* the assembler is case insensitive to instructions (but case sensitive * the assembler is case insensitive to instructions (but case sensitive
* to symbols) * to symbols)
@ -6307,7 +6303,7 @@ static int emit_findopcode(char *instr,int maxlen)
while (low<high) { while (low<high) {
mid=(low+high)/2; mid=(low+high)/2;
assert(emit_opcodelist[mid].name!=NULL); assert(emit_opcodelist[mid].name!=NULL);
cmp=stricmp(str,emit_opcodelist[mid].name); cmp=stricmp(instr,emit_opcodelist[mid].name);
if (cmp>0) if (cmp>0)
low=mid+1; low=mid+1;
else else
@ -6315,7 +6311,7 @@ static int emit_findopcode(char *instr,int maxlen)
} /* while */ } /* while */
assert(low==high); assert(low==high);
if (stricmp(str,emit_opcodelist[low].name)==0) if (stricmp(instr,emit_opcodelist[low].name)==0)
return low; /* found */ return low; /* found */
return 0; /* not found, return special index */ return 0; /* not found, return special index */
} }