This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

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


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]