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

[Ayal Zaks via Phabricator via llvm-commits]

Ayal added inline comments.


================
Comment at: llvm/test/Transforms/LoopVectorize/interleaved-accesses-use-after-free.ll:25
+
+; CHECK-NOT: Too many dependences, stopped recording
+; If too many dependences are recorded, LoopAccessAnalysis just conservatively
----------------
Ahh, but this test does exceed the threshold leading to no dependencies being recorded, which is needed to reproduce the bug?

Checking if GroupA!=GroupB above before calling canReorderMemAccessesForInterleavedGroups() fixes interleave group construction when this threshold is exceeded?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D155520