diff options
| author | Jan Beulich <jbeulich@novell.com> | 2019-06-25 09:41:33 +0200 |
|---|---|---|
| committer | Jan Beulich <jbeulich@suse.com> | 2019-06-25 09:41:33 +0200 |
| commit | 2c70385689542a4c4cbe160601e54f9f18a4c4c5 (patch) | |
| tree | 4e3fac1720511587ea3672d30794c14116a770b2 /opcodes/i386-init.h | |
| parent | 4970191fa557c4769697fb41db06f2dcb5f7de2e (diff) | |
| download | fsf-binutils-gdb-2c70385689542a4c4cbe160601e54f9f18a4c4c5.zip fsf-binutils-gdb-2c70385689542a4c4cbe160601e54f9f18a4c4c5.tar.gz fsf-binutils-gdb-2c70385689542a4c4cbe160601e54f9f18a4c4c5.tar.bz2 | |
x86: correct / adjust debug printing
For quite some time we've been using combinations of bits for
specifying various registers in operands and templates. I think it was
Alan who had indicated that likely the debug printing would need
adjustment as a result. Here we go.
Accumulator handling for GPRs gets changed to match that for FPU regs.
For this to work, OPERAND_TYPE_ACC{32,64} get repurposed, with their
original uses replaced by direct checks of the two bits of interest,
which is cheaper than operand_type_equal() invocations.
For SIMD registers nothing similar appears to be needed, as respective
operands get stripped from the (copy of the) template before pt() is
reached.
The type change on pi() is to silence a compiler diagnostic. Arguably
its other parameter could also be const-qualified.
Diffstat (limited to 'opcodes/i386-init.h')
| -rw-r--r-- | opcodes/i386-init.h | 23 |
1 files changed, 14 insertions, 9 deletions
diff --git a/opcodes/i386-init.h b/opcodes/i386-init.h index 1d9c17f..735403a 100644 --- a/opcodes/i386-init.h +++ b/opcodes/i386-init.h @@ -1458,8 +1458,8 @@ 0, 0 } } #define OPERAND_TYPE_DEBUG \ - { { 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 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, 0, 0, 0, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, \ + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, \ 0, 0 } } #define OPERAND_TYPE_FLOATREG \ @@ -1482,11 +1482,6 @@ 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, \ 0, 0 } } -#define OPERAND_TYPE_ACC \ - { { 0, 0, 0, 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, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, \ - 0, 0 } } - #define OPERAND_TYPE_JUMPABSOLUTE \ { { 0, 0, 0, 0, 0, 0, 0, 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, 0, 0, 0, 0, 0, 0, \ @@ -1522,13 +1517,23 @@ 0, 0, 0, 0, 0, 0, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, \ 0, 0 } } +#define OPERAND_TYPE_ACC8 \ + { { 0, 0, 0, 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, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, \ + 0, 0 } } + +#define OPERAND_TYPE_ACC16 \ + { { 0, 0, 0, 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, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, \ + 0, 0 } } + #define OPERAND_TYPE_ACC32 \ - { { 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, \ + { { 0, 0, 0, 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, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, \ 0, 0 } } #define OPERAND_TYPE_ACC64 \ - { { 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, \ + { { 0, 0, 0, 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, 0, 0, 1, 0, 0, 0, 0, 0, 0, 0, \ 0, 0 } } |