[PATCH] D105989: [AMDGPU] Define and use a helper for setting idxen

Fri Jul 16 07:40:15 PDT 2021

foad abandoned this revision.
foad added a comment.

In D105989#2877366 <https://reviews.llvm.org/D105989#2877366>, @arsenm wrote:

> In D105989#2877226 <https://reviews.llvm.org/D105989#2877226>, @foad wrote:
>
>> Why would that be required? Are there some instructions that don't work with idxen=0?
>
> I don't remember, but the old intrinsics only had the one offset and we inferred idxen based on the value which didn't work

OK, it seems that bounds checking does different things depending on the idxen bit in the instruction.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D105989/new/

https://reviews.llvm.org/D105989