diff options
| author | Jan Beulich <jbeulich@suse.com> | 2021-03-30 14:08:11 +0200 |
|---|---|---|
| committer | Jan Beulich <jbeulich@suse.com> | 2021-03-30 14:08:11 +0200 |
| commit | 6288d05f11827f993308e6a2693516e2c123c0fb (patch) | |
| tree | c83372fd531e4f4c3fd551da638244be6b13b138 /opcodes | |
| parent | ca5312a2416f065517c354cb6a9cc2616174761b (diff) | |
| download | binutils-6288d05f11827f993308e6a2693516e2c123c0fb.zip binutils-6288d05f11827f993308e6a2693516e2c123c0fb.tar.gz binutils-6288d05f11827f993308e6a2693516e2c123c0fb.tar.bz2 | |
x86: adjust st(<N>) parsing
st(1) ... st(7) will never be looked up in the hash table, so there's no
point inserting the entries. It's also not really necessary to do a 2nd
hash lookup after parsing the register number, nor is there a real
reason for having both st and st(0) entries. Plus we can easily do away
with the need for st to be first in the table.
Diffstat (limited to 'opcodes')
| -rw-r--r-- | opcodes/ChangeLog | 7 | ||||
| -rw-r--r-- | opcodes/i386-opc.h | 6 | ||||
| -rw-r--r-- | opcodes/i386-reg.tbl | 6 | ||||
| -rw-r--r-- | opcodes/i386-tbl.h | 6 |
4 files changed, 13 insertions, 12 deletions
diff --git a/opcodes/ChangeLog b/opcodes/ChangeLog index 81c0fa2..5acb021 100644 --- a/opcodes/ChangeLog +++ b/opcodes/ChangeLog @@ -1,3 +1,10 @@ +2021-03-30 Jan Beulich <jbeulich@suse.com> + + * i386-opc.h (REGNAM_AL, REGNAM_AX, REGNAM_EAX): Adjust values. + * i386-reg.tbl (st): Move down. + (st(0)): Delete. Extend comment. + * i386-tbl.h: Re-generate. + 2021-03-29 Jan Beulich <jbeulich@suse.com> * i386-opc.tbl (movq, movabs): Move next to mov counterparts. diff --git a/opcodes/i386-opc.h b/opcodes/i386-opc.h index 115895c..a2a657e 100644 --- a/opcodes/i386-opc.h +++ b/opcodes/i386-opc.h @@ -991,9 +991,9 @@ typedef struct reg_entry; /* Entries in i386_regtab. */ -#define REGNAM_AL 1 -#define REGNAM_AX 25 -#define REGNAM_EAX 41 +#define REGNAM_AL 0 +#define REGNAM_AX 24 +#define REGNAM_EAX 40 extern const reg_entry i386_regtab[]; extern const unsigned int i386_regtab_size; diff --git a/opcodes/i386-reg.tbl b/opcodes/i386-reg.tbl index 7ea974d..be2c1cc 100644 --- a/opcodes/i386-reg.tbl +++ b/opcodes/i386-reg.tbl @@ -18,8 +18,6 @@ // Software Foundation, 51 Franklin Street - Fifth Floor, Boston, MA // 02110-1301, USA. -// Make %st first as we test for it. -st, Class=Reg|Instance=Accum|Tbyte, 0, 0, 11, 33 // 8 bit regs al, Class=Reg|Instance=Accum|Byte, 0, 0, Dw2Inval, Dw2Inval cl, Class=Reg|Instance=RegC|Byte, 0, 1, Dw2Inval, Dw2Inval @@ -300,8 +298,8 @@ eip, Dword, RegRex64, RegIP, 8, Dw2Inval // for addressing. riz, Qword|BaseIndex, RegRex64, RegIZ, Dw2Inval, Dw2Inval eiz, Dword|BaseIndex, 0, RegIZ, Dw2Inval, Dw2Inval -// fp regs. -st(0), Class=Reg|Instance=Accum|Tbyte, 0, 0, 11, 33 +// fp regs. No need for an explicit st(0) here. +st, Class=Reg|Instance=Accum|Tbyte, 0, 0, 11, 33 st(1), Class=Reg|Tbyte, 0, 1, 12, 34 st(2), Class=Reg|Tbyte, 0, 2, 13, 35 st(3), Class=Reg|Tbyte, 0, 3, 14, 36 diff --git a/opcodes/i386-tbl.h b/opcodes/i386-tbl.h index 0a326fd..d2ca247 100644 --- a/opcodes/i386-tbl.h +++ b/opcodes/i386-tbl.h @@ -64204,10 +64204,6 @@ const insn_template i386_optab[] = const reg_entry i386_regtab[] = { - { "st", - { { 1, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, - 1, 0, 0, 0, 0, 0, 0 } }, - 0, 0, { 11, 33 } }, { "al", { { 1, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0 } }, @@ -65260,7 +65256,7 @@ const reg_entry i386_regtab[] = { { 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 0, 0, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0 } }, 0, RegIZ, { Dw2Inval, Dw2Inval } }, - { "st(0)", + { "st", { { 1, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 0, 0, 0, 0, 0, 0 } }, 0, 0, { 11, 33 } }, |