[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