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

x86: fold OP_Mwaitx() into OP_Mwait()

Browse Source 
There's no need to have separate functions, the difference can easily be
expressed using the function arguments.
This commit is contained in:
Jan Beulich
2019-11-05 09:19:50 +01:00
parent 267b8516f2
commit 7abb8d8111
2 changed files with 11 additions and 24 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
opcodes/ChangeLog
View File

@ -1,3 +1,9 @@
2019-11-05 Jan Beulich <jbeulich@suse.com>
* i386-dis.c (OP_Mwaitx): Delete.
(prefix_table): Use OP_Mwait for mwaitx entry.
(OP_Mwait): Also handle mwaitx.
2019-11-05 Jan Beulich <jbeulich@suse.com> 2019-11-05 Jan Beulich <jbeulich@suse.com>
* i386-dis.c (PREFIX_0F01_REG_7_MOD_3_RM_2, * i386-dis.c (PREFIX_0F01_REG_7_MOD_3_RM_2,

29
opcodes/i386-dis.c
View File

@ -101,7 +101,6 @@ static void VPCOM_Fixup (int, int);
static void OP_0f07 (int, int); static void OP_0f07 (int, int);
static void OP_Monitor (int, int); static void OP_Monitor (int, int);
static void OP_Mwait (int, int); static void OP_Mwait (int, int);
static void OP_Mwaitx (int, int);
static void NOP_Fixup1 (int, int); static void NOP_Fixup1 (int, int);
static void NOP_Fixup2 (int, int); static void NOP_Fixup2 (int, int);
static void OP_3DNowSuffix (int, int); static void OP_3DNowSuffix (int, int);
@ -3652,7 +3651,7 @@ static const struct dis386 prefix_table[][4] = {
/* PREFIX_0F01_REG_7_MOD_3_RM_3 */ /* PREFIX_0F01_REG_7_MOD_3_RM_3 */
{ {
{ "mwaitx", { { OP_Mwaitx, 0 } }, 0 }, { "mwaitx", { { OP_Mwait, eBX_reg } }, 0 },
}, },
/* PREFIX_0F09 */ /* PREFIX_0F09 */
@ -15516,35 +15515,17 @@ CMP_Fixup (int bytemode ATTRIBUTE_UNUSED, int sizeflag ATTRIBUTE_UNUSED)
} }
static void static void
OP_Mwaitx (int bytemode ATTRIBUTE_UNUSED, OP_Mwait (int bytemode, int sizeflag ATTRIBUTE_UNUSED)
int sizeflag ATTRIBUTE_UNUSED)
{ {
/* mwaitx %eax,%ecx,%ebx */ /* mwait %eax,%ecx / mwaitx %eax,%ecx,%ebx */
if (!intel_syntax)
{
const char **names = (address_mode == mode_64bit
? names64 : names32);
strcpy (op_out[0], names[0]);
strcpy (op_out[1], names[1]);
strcpy (op_out[2], names[3]);
two_source_ops = 1;
}
/* Skip mod/rm byte. */
MODRM_CHECK;
codep++;
}
static void
OP_Mwait (int bytemode ATTRIBUTE_UNUSED,
int sizeflag ATTRIBUTE_UNUSED)
{
/* mwait %eax,%ecx */
if (!intel_syntax) if (!intel_syntax)
{ {
const char **names = (address_mode == mode_64bit const char **names = (address_mode == mode_64bit
? names64 : names32); ? names64 : names32);
strcpy (op_out[0], names[0]); strcpy (op_out[0], names[0]);
strcpy (op_out[1], names[1]); strcpy (op_out[1], names[1]);
if (bytemode == eBX_reg)
strcpy (op_out[2], names[3]);
two_source_ops = 1; two_source_ops = 1;
} }
/* Skip mod/rm byte. */ /* Skip mod/rm byte. */