This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH 5/5] x86: drop VecESize
>>> On 21.03.18 at 20:19, <hjl.tools@gmail.com> wrote:
> On Wed, Mar 21, 2018 at 7:23 AM, Jan Beulich <JBeulich@suse.com> wrote:
>> It again can be inferred from other information.
>>
>> The vpopcntd templates all need to have Dword added to their memory
>> operands; the lack thereof was actually a bug preventing certain Intel
>> syntax code to assemble, so test cases get extended.
>>
>> gas/
>> 2018-03-21 Jan Beulich <jbeulich@suse.com>
>>
>> * config/tc-i386.c (check_VecOperands): Replace uses of
>> .vecesize. Always initialize op.
>> * testsuite/gas/i386/avx512_vpopcntdq.s,
>> testsuite/gas/i386/avx512bitalg_vl.s: Add Intel syntax vpopcnt
>> broadcast cases with explicit operand size.
>> * testsuite/gas/i386/avx512_vpopcntdq.d,
>> testsuite/gas/i386/avx512_vpopcntdq-intel.d,
>> testsuite/gas/i386/avx512bitalg_vl.d
>> testsuite/gas/i386/avx512bitalg_vl-intel.d: Adjust expectations.
>>
>> opcodes/
>> 2018-03-21 Jan Beulich <jbeulich@suse.com>
>>
>> * i386-gen.c (opcode_modifiers): Delete VecESize.
>> * i386-opc.h (VecESize): Delete.
>> (struct i386_opcode_modifier): Delete vecesize.
>> * i386-opc.tbl: Drop VecESize.
>> * i386-tlb.h: Re-generate.
>>
>
> OK.
Thanks, but what about patch 4? Also, without patch 2 the one
here will need slight modification to apply.
Jan