[PATCH v2] x86: fold some prefix related attributes into a single one

H.J. Lu hjl.tools@gmail.com
Mon Mar 8 14:32:45 GMT 2021


On Mon, Mar 8, 2021 at 5:57 AM Jan Beulich <jbeulich@suse.com> wrote:
>
> RepPrefixOk, HLEPrefixOk, and NoTrackPrefixOk can't be specified
> together, so can share an enum-like field. IsLockable can be inferred
> from HLE setting and hence only needs specifying when neither of them
> is present.
>
> gas/
> 2021-03-XX  Jan Beulich  <jbeulich@suse.com>
>
>         * config/tc-i386.c (check_hle): Replace HLEPrefix* by Prefix*.
>         (md_assemble): Replace use of repprefixok, islockable, and
>         notrackprefixok.
>         (i386_index_check): Replace use of repprefixok.
>
> opcodes/
> 2021-03-XX  Jan Beulich  <jbeulich@suse.com>
>
>         * opcodes/i386-gen.c (opcode_modifiers): Delete NoTrackPrefixOk,
>         IsLockable, RepPrefixOk, and HLEPrefixOk elements. Add PrefixOk
>         element.
>         * opcodes/i386-opc.h (NoTrackPrefixOk, IsLockable, HLEPrefixNone,
>         HLEPrefixLock, HLEPrefixAny, HLEPrefixRelease): Delete.
>         (PrefixNone, PrefixRep, PrefixHLERelease, PrefixNoTrack,
>         PrefixLock, PrefixHLELock, PrefixHLEAny): Define.
>         (struct i386_opcode_modifier): Delete notrackprefixok,
>         islockable, hleprefixok, and repprefixok fields. Add prefixok
>         field.
>         * opcodes/i386-opc.tbl (RepPrefixOk, LockPrefixOk, HLEPrefixAny,
>         HLEPrefixLock, HLEPrefixRelease, NoTrackPrefixOk): Define.
>         (mov, xchg, add, inc, sub, dec, sbb, and, or, xor, adc, neg,
>         not, btc, btr, bts, xadd, cmpxchg, cmpxchg8b, movq, cmpxchg16b):
>         Replace HLEPrefixOk.
>         * opcodes/i386-tbl.h: Re-generate.
> ---
> v2: Drop Byte / Word / Dword / Qword removal.
>

OK.

Thanks.

-- 
H.J.


More information about the Binutils mailing list