[PATCH] D132634: [AArch64] Add index check before lowerInterleavedStore() uses ShuffleVectorInst's mask

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 25 13:38:08 PDT 2022


fhahn added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64ISelLowering.cpp:13369
+  // later.
+  for (unsigned Idx : Mask) {
+    if (Idx >= VecTy->getNumElements() * 2) {
----------------
This is probably too restrictive and not really matches the comment. 

There are a few test failures with the patch, including ` LLVM.Transforms/InterleavedAccess/AArch64::interleaved-accesses.ll`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D132634



More information about the llvm-commits mailing list