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, i386] Update to match latest AVX512 spec.


On Wed, Feb 19, 2014 at 9:16 AM, Ilya Tocar <tocarip.intel@gmail.com> wrote:
> Hi,
> Between [1] and [2] versions of the spec there were some changes.
> In particular:
>   1)prefetchwt1 instruction now has it's own CPUID
>   2)vptestnmd/vptesnmq instructions now have CPUID of AVX512F not AVX512CD
> Also there was version of vctps2ph with sae and only 1 register operand, which was encoded
> as if missing operand was equal to ymm0. I didn't found any references to it in [2],
> so I removed it.
> Attached is gzipped patch. Testing passes. There are a few trailing whitespaces,
> but only in auto-generated files.
> Ok for trunk?
>
> [1] http://download-software.intel.com/sites/default/files/319433-015.pdf
> [2] http://download-software.intel.com/sites/default/files/managed/50/1a/319433-018.pdf

Please break it into 3 separate patches:

1. Remove wrong cvtps2ph variant.
2. Add cpu PREFETCHWT1.
3. Update cpu for vptestnmd/vptestnmq.

Please provide proper commit log for each patch  You can take
a look at commits at:

https://sourceware.org/git/?p=binutils-gdb.git;a=summary

for examples.

Thanks.

-- 
H.J.


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