[PATCH] D17332: [LV] Fix PR26600: avoid out of bounds loads for interleaved access vectorization
dewen via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Mar 14 00:46:34 PDT 2023
dewen added inline comments.
Herald added subscribers: pcwang-thead, nemanjai.
Herald added projects: LLVM, All.
================
Comment at: llvm/trunk/lib/Transforms/Vectorize/LoopVectorize.cpp:4803
+ // This guarantees that we won't do speculative out of bounds loads.
+ for (InterleaveGroup *Group : LoadGroups)
+ if (!Group->getMember(0) || !Group->getMember(Group->getFactor() - 1))
----------------
Hi, @sbaranga. I have some doubts about this part of the code, whether it is too conservative, and I have not found a case to verify your statement.
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D17332/new/
https://reviews.llvm.org/D17332
More information about the llvm-commits
mailing list