[PATCH] D71773: [AArch64][SVE] Update the definition of AdvSIMD_GatherLoad_VecTorBase_Intrinsic

[Andrzej Warzynski via Phabricator via llvm-commits]

andwar updated this revision to Diff 238536.
andwar added a comment.
Herald added a subscriber: arphaman.

Fix the scaling of the offset

When switching from GLD1_IMM to GLD1 or GLD1_UXTW, the original index needs to be scaled by the size of the underlying data to be loaded. Also, the index needs to be in the range [0, 31]. This patch addresses that.

Also:

- Renamed `llvm.aarch64.sve.ld1.gather.scalar.offset` to `llvm.aarch64.sve.ld1.gather.scalar.index`
- Renamed the corresponding test files
- Updated the tests so that 32 is used as the lowest out of range value for an immediate index
- Similar changes for scatter stores


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71773

Files:
  llvm/include/llvm/IR/IntrinsicsAArch64.td
  llvm/lib/Target/AArch64/AArch64ISelLowering.cpp
  llvm/lib/Target/AArch64/AArch64SVEInstrInfo.td
  llvm/test/CodeGen/AArch64/sve-gather-scatter-dag-combine.ll
  llvm/test/CodeGen/AArch64/sve-intrinsics-gather-loads-vector-base-imm-index.ll
  llvm/test/CodeGen/AArch64/sve-intrinsics-gather-loads-vector-base-scalar-index.ll
  llvm/test/CodeGen/AArch64/sve-intrinsics-gather-loads-vector-base.ll
  llvm/test/CodeGen/AArch64/sve-intrinsics-scatter-stores-vector-base-imm-index.ll
  llvm/test/CodeGen/AArch64/sve-intrinsics-scatter-stores-vector-base-scalar-index.ll
  llvm/test/CodeGen/AArch64/sve-intrinsics-scatter-stores-vector-base.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D71773.238536.patch
Type: text/x-patch
Size: 83304 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200116/98a93e1c/attachment-0001.bin>