[PATCH] D146206: [LAA] Fix transitive analysis invalidation bug by implementing LoopAccessInfoManager::invalidate

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 16 10:57:18 PDT 2023


fhahn accepted this revision.
fhahn added a comment.

Fix LGTM with comments by other people addressed.



================
Comment at: llvm/test/Analysis/LoopAccessAnalysis/invalidation.ll:2
+; Test that access-info gets invalidated when the analyses it depends on are
+; invalidated.
+
----------------
Might be good to include a reference to the bug as well.


================
Comment at: llvm/test/Analysis/LoopAccessAnalysis/invalidation.ll:75
+
+attributes #0 = { nofree norecurse nosync nounwind memory(argmem: write) }
----------------
are those needed?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146206



More information about the llvm-commits mailing list