espressif/binutils-gdb
1
0
Fork 0
mirror of https://github.com/espressif/binutils-gdb.git synced 2025-06-27 14:39:09 +08:00
Code Issues Packages Projects Releases Wiki Activity

x86: replace "anymem" checks where possible

Browse Source 
Once operand parsing has completed, the simpler check of Operand_Mem can
be used in places where i.types[] got passed to operand_type_check().
Note that this has shown a couple of omissions of adjusting i.flags[]
when playing with i.op[] / i.types[] / i.tm.operand_types[].  Not all of
them get added here, just all of the ones needed in process_operands().
This commit is contained in:
Jan Beulich
2019-07-17 09:16:31 +02:00
parent 1d942ae908
commit 8dc0818eb2
2 changed files with 22 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
gas/ChangeLog
View File

@ -1,3 +1,11 @@
2019-07-17 Jan Beulich <jbeulich@suse.com>
* config/tc-i386.c (check_hle, md_assemble, check_VecOperands,
match_template, check_string, build_modrm_byte): Replace
operand_type_check(..., anymem) by Operand_Mem ones.
(process_operands): Also copy i.flags[] when copying other
operand properties.
2019-07-16 Jan Beulich <jbeulich@suse.com> 2019-07-16 Jan Beulich <jbeulich@suse.com>
* config/tc-i386.c (match_template): Adjust regmem reference. * config/tc-i386.c (match_template): Adjust regmem reference.

22
gas/config/tc-i386.c
View File

@ -3918,8 +3918,7 @@ check_hle (void)
i.tm.name); i.tm.name);
return 0; return 0;
} }
if (i.mem_operands == 0 if (i.mem_operands == 0 || !(i.flags[i.operands - 1] & Operand_Mem))
|| !operand_type_check (i.types[i.operands - 1], anymem))
{ {
as_bad (_("memory destination needed for instruction `%s'" as_bad (_("memory destination needed for instruction `%s'"
" after `xrelease'"), i.tm.name); " after `xrelease'"), i.tm.name);
@ -4358,7 +4357,7 @@ md_assemble (char *line)
&& (!i.tm.opcode_modifier.islockable && (!i.tm.opcode_modifier.islockable
|| i.mem_operands == 0 || i.mem_operands == 0
|| (i.tm.base_opcode != 0x86 || (i.tm.base_opcode != 0x86
&& !operand_type_check (i.types[i.operands - 1], anymem)))) && !(i.flags[i.operands - 1] & Operand_Mem))))
{ {
as_bad (_("expecting lockable instruction after `lock'")); as_bad (_("expecting lockable instruction after `lock'"));
return; return;
@ -5473,7 +5472,7 @@ check_VecOperands (const insn_template *t)
{ {
/* Find memory operand. */ /* Find memory operand. */
for (op = 0; op < i.operands; op++) for (op = 0; op < i.operands; op++)
if (operand_type_check (i.types[op], anymem)) if (i.flags[op] & Operand_Mem)
break; break;
gas_assert (op < i.operands); gas_assert (op < i.operands);
/* Check size of the memory operand. */ /* Check size of the memory operand. */
@ -5563,7 +5562,7 @@ check_VecOperands (const insn_template *t)
i.memshift = 0; i.memshift = 0;
for (op = 0; op < i.operands; op++) for (op = 0; op < i.operands; op++)
if (operand_type_check (i.types[op], anymem)) if (i.flags[op] & Operand_Mem)
{ {
if (t->opcode_modifier.evex == EVEXLIG) if (t->opcode_modifier.evex == EVEXLIG)
i.memshift = 2 + (i.suffix == QWORD_MNEM_SUFFIX); i.memshift = 2 + (i.suffix == QWORD_MNEM_SUFFIX);
@ -5892,7 +5891,7 @@ match_template (char mnem_suffix)
&& i.hle_prefix && i.hle_prefix
&& t->base_opcode == 0xa0 && t->base_opcode == 0xa0
&& i.types[0].bitfield.acc && i.types[0].bitfield.acc
&& operand_type_check (i.types[1], anymem)) && (i.flags[1] & Operand_Mem))
continue; continue;
/* Fall through. */ /* Fall through. */
@ -6191,7 +6190,8 @@ check_reverse:
static int static int
check_string (void) check_string (void)
{ {
int mem_op = operand_type_check (i.types[0], anymem) ? 0 : 1; unsigned int mem_op = i.flags[0] & Operand_Mem ? 0 : 1;
if (i.tm.operand_types[mem_op].bitfield.esseg) if (i.tm.operand_types[mem_op].bitfield.esseg)
{ {
if (i.seg[0] != NULL && i.seg[0] != &es) if (i.seg[0] != NULL && i.seg[0] != &es)
@ -6893,6 +6893,7 @@ process_operands (void)
i.op[j - 1] = i.op[j]; i.op[j - 1] = i.op[j];
i.types[j - 1] = i.types[j]; i.types[j - 1] = i.types[j];
i.tm.operand_types[j - 1] = i.tm.operand_types[j]; i.tm.operand_types[j - 1] = i.tm.operand_types[j];
i.flags[j - 1] = i.flags[j];
} }
} }
} }
@ -6909,6 +6910,7 @@ process_operands (void)
i.op[j] = i.op[j - 1]; i.op[j] = i.op[j - 1];
i.types[j] = i.types[j - 1]; i.types[j] = i.types[j - 1];
i.tm.operand_types[j] = i.tm.operand_types[j - 1]; i.tm.operand_types[j] = i.tm.operand_types[j - 1];
i.flags[j] = i.flags[j - 1];
} }
i.op[0].regs i.op[0].regs
= (const reg_entry *) hash_find (reg_hash, "xmm0"); = (const reg_entry *) hash_find (reg_hash, "xmm0");
@ -6924,6 +6926,7 @@ process_operands (void)
i.op[dupl] = i.op[dest]; i.op[dupl] = i.op[dest];
i.types[dupl] = i.types[dest]; i.types[dupl] = i.types[dest];
i.tm.operand_types[dupl] = i.tm.operand_types[dest]; i.tm.operand_types[dupl] = i.tm.operand_types[dest];
i.flags[dupl] = i.flags[dest];
} }
else else
{ {
@ -6935,6 +6938,7 @@ duplicate:
i.op[dupl] = i.op[dest]; i.op[dupl] = i.op[dest];
i.types[dupl] = i.types[dest]; i.types[dupl] = i.types[dest];
i.tm.operand_types[dupl] = i.tm.operand_types[dest]; i.tm.operand_types[dupl] = i.tm.operand_types[dest];
i.flags[dupl] = i.flags[dest];
} }
if (i.tm.opcode_modifier.immext) if (i.tm.opcode_modifier.immext)
@ -6953,6 +6957,8 @@ duplicate:
/* We need to adjust fields in i.tm since they are used by /* We need to adjust fields in i.tm since they are used by
build_modrm_byte. */ build_modrm_byte. */
i.tm.operand_types [j - 1] = i.tm.operand_types [j]; i.tm.operand_types [j - 1] = i.tm.operand_types [j];
i.flags[j - 1] = i.flags[j];
} }
i.operands--; i.operands--;
@ -7358,7 +7364,7 @@ build_modrm_byte (void)
unsigned int op; unsigned int op;
for (op = 0; op < i.operands; op++) for (op = 0; op < i.operands; op++)
if (operand_type_check (i.types[op], anymem)) if (i.flags[op] & Operand_Mem)
break; break;
gas_assert (op < i.operands); gas_assert (op < i.operands);