This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH] Fix memory size for AVX512 gather/scatter.
- From: "H.J. Lu" <hjl dot tools at gmail dot com>
- To: Ilya Tocar <tocarip dot intel at gmail dot com>
- Cc: Binutils <binutils at sourceware dot org>
- Date: Thu, 20 Mar 2014 10:23:01 -0700
- Subject: Re: [PATCH] Fix memory size for AVX512 gather/scatter.
- Authentication-results: sourceware.org; auth=none
- References: <20140320115947 dot GA40379 at msticlxl7 dot ims dot intel dot com>
On Thu, Mar 20, 2014 at 4:59 AM, Ilya Tocar <tocarip.intel@gmail.com> wrote:
> Hi,
>
> For gathers with indices larger than elements (e. g.)
>
> vpgatherqd ymm6{k1}, ZMMWORD PTR [ebp+zmm7*8-123]
>
> We currently treat memory size as a size of index register,
> while it is actually should be size of destination register:
>
> vpgatherqd ymm6{k1}, YMMWORD PTR [ebp+zmm7*8-123]
>
> This patch fixes it.
I checked it into master and backported it to binutils-2_24-branch
with the addition to commit log:
Fix memory size for gather/scatter instructions
For gathers with indices larger than elements (e. g.)
vpgatherqd ymm6{k1}, ZMMWORD PTR [ebp+zmm7*8-123]
We currently treat memory size as a size of index register, while it is
actually should be size of destination register:
vpgatherqd ymm6{k1}, YMMWORD PTR [ebp+zmm7*8-123]
This patch fixes it.
Thanks.
--
H.J.