[PATCH] D152386: [LoadStoreVectorizer] Only upgrade align for alloca
Justin Lebar via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jun 7 19:48:22 PDT 2023
jlebar accepted this revision.
jlebar added a comment.
This revision is now accepted and ready to land.
Looks reasonable to me. Thank you, and sorry for the breakage.
================
Comment at: llvm/lib/Transforms/Vectorize/LoadStoreVectorizer.cpp:805
+ Value *PtrOperand = getLoadStorePointerOperand(C[CBegin].Inst);
+ bool IsAllocaAccess = isa<AllocaInst>(PtrOperand->stripInBoundsOffsets());
Align Alignment = getLoadStoreAlignment(C[CBegin].Inst);
----------------
getOrEnforceKnownAlignment will merely stripPointerCasts, which is weaker than stripInBoundsOffsets. Presumably we should do the same here?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D152386/new/
https://reviews.llvm.org/D152386
More information about the llvm-commits
mailing list