[PATCH] D155520: [LV] Complete load groups and release store groups in presence of dependency

Anna Thomas via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 27 10:08:48 PDT 2023


anna updated this revision to Diff 544837.
anna added a comment.

fixed use-after-free error (with added testcase). This bug wasn't there before the patch because we would break out of the inner loop accessing A, whereas now we were continuing to see which other A accesses needed to release the group.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D155520

Files:
  llvm/lib/Analysis/VectorUtils.cpp
  llvm/test/Transforms/LoopVectorize/X86/interleaved-accesses-hoist-load-across-store.ll
  llvm/test/Transforms/LoopVectorize/X86/interleaved-accesses-sink-store-across-load.ll
  llvm/test/Transforms/LoopVectorize/interleaved-accesses-use-after-free.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D155520.544837.patch
Type: text/x-patch
Size: 23665 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230727/5a2b3b91/attachment.bin>


More information about the llvm-commits mailing list