diff options
| author | Jan Beulich <jbeulich@suse.com> | 2024-01-09 13:50:27 +0100 |
|---|---|---|
| committer | Jan Beulich <jbeulich@suse.com> | 2024-01-09 13:50:27 +0100 |
| commit | 7c3df3c680c2bee0b17e60bfbadc751c67e05ed0 (patch) | |
| tree | cfd73af5e18b93d100a381b2bfd16716f8147492 /opcodes/i386-opc.tbl | |
| parent | 7440781b67631ebdf1b0587724f5d77c3748b034 (diff) | |
| download | binutils-7c3df3c680c2bee0b17e60bfbadc751c67e05ed0.zip binutils-7c3df3c680c2bee0b17e60bfbadc751c67e05ed0.tar.gz binutils-7c3df3c680c2bee0b17e60bfbadc751c67e05ed0.tar.bz2 | |
x86: add missing APX logic to cpu_flags_match()
As already indicated during review, we can't get away without certain
adjustments here: Without these, respective {evex}-prefixed insns are
assembled to APX encodings even when APX_F is turned off.
While there also extend the respective comment in the opcode table, to
explain why this construct is used.
Diffstat (limited to 'opcodes/i386-opc.tbl')
| -rw-r--r-- | opcodes/i386-opc.tbl | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/opcodes/i386-opc.tbl b/opcodes/i386-opc.tbl index 1751eff..aceac97 100644 --- a/opcodes/i386-opc.tbl +++ b/opcodes/i386-opc.tbl @@ -143,7 +143,13 @@ #define DstVVVV VexVVVV=VexVVVV_DST -// The template supports VEX format for cpuid and EVEX format for cpuid & apx_f. +// The template supports VEX format for cpuid and EVEX format for cpuid & APX_F. +// While therefore we really mean cpuid|(cpuid&APX_F) here, this can't be +// expressed in the generated templates. It's equivalent to just cpuid|APX_F +// anyway, but that is not what we want (as APX_F alone isn't a sufficient +// prereq for such insns). Instead the assembler will massage the CPU specifier +// to the equivalent of either cpuid&(cpuid) or cpuid&(APX_F) (or something +// substantially similar), depending on what encoding was requested. #define APX_F(cpuid) cpuid&(cpuid|APX_F) // The EVEX purpose of StaticRounding appears only together with SAE. Re-use |