[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
Wed Jul 26 11:56:49 PDT 2023
anna added inline comments.
================
Comment at: llvm/lib/Analysis/VectorUtils.cpp:1226-1227
+ << *A << " and " << *DependentInst << '\n');
+ StoreGroups.remove(GroupA);
+ releaseGroup(GroupA);
+ }
----------------
MatzeB wrote:
> For the crashes I see `GroupA == GroupB` so any later access to `GroupB` after freeing `GroupA` here fail.
thank you for the reproducer! I will revert for now and fix it.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D155520/new/
https://reviews.llvm.org/D155520
More information about the llvm-commits
mailing list