This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH] Fix memory operand size for vcvtt?ps2u?qq instructions.
- From: "H.J. Lu" <hjl dot tools at gmail dot com>
- To: Alexander Fomin <afomin dot mailbox at gmail dot com>
- Cc: Binutils <binutils at sourceware dot org>
- Date: Wed, 22 Jul 2015 13:30:41 -0700
- Subject: Re: [PATCH] Fix memory operand size for vcvtt?ps2u?qq instructions.
- Authentication-results: sourceware.org; auth=none
- References: <20150721162910 dot GA13299 at msticlxl57 dot ims dot intel dot com>
On Tue, Jul 21, 2015 at 9:29 AM, Alexander Fomin
<afomin.mailbox@gmail.com> wrote:
> When disassembling AVX512 vcvtt?ps2u?q instructions with data
> broadcasting enabled, memory operand size should be DWORD.
>
> gas/testsuite
> * gas/i386avx512dq-intel.d: Replace "QWORD" with "DWORD" in
> vcvtt?ps2u?qq instructions disassembly regexes.
> Add disassembly regex for new test.
> * gas/i386/avx512dq.d: Likewise.
> * gas/i386/avx512dq_vl-intel.d: Likewise.
> * gas/i386/avx512dq_vl.d: Likewise.
> * gas/i386/x86-64-avx512dq-intel.d: Likewise.
> * gas/i386/x86-64-avx512dq.d: Likewise.
> * gas/i386/x86-64-avx512dq_vl-intel.d: Likewise.
> * gas/i386/x86-64-avx512dq_vl.d: Likewise.
> * gas/i386/avx512dq.s: Add new test for Intel syntax with memory
> operand and broadcasting enabled.
> * gas/i386/avx512dq_vl.s: Likewise.
> * gas/i386/x86-64-avx512dq.s: Likewise.
> * gas/i386/x86-64-avx512dq_vl.s: Likewise.
>
> opcodes/
> * i386-dis-evex.h (EVEX_W_0F78_P_2): Replace "EXxmmq" with
> "EXEvexHalfBcstXmmq" for the second operand.
> (EVEX_W_0F79_P_2): Likewise.
> (EVEX_W_0F7A_P_2): Likewise.
> (EVEX_W_0F7B_P_2): Likewise.
I checked in it for you with updated ChangeLog entries:
1. Add PR binutils/18631.
2. Use TAB.
Please provide better ChangeLog entries next time.
Thanks.
--
H.J.