[PATCH] D84959: [NewPM][LVI] Abandon LVI after CVP
Teresa Johnson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jul 30 11:17:07 PDT 2020
tejohnson added a comment.
Thanks for the fix! I tend to agree with @aeubanks comment that it might be better to make this unconditional and change the tests (and the comment "Any invalidation that shows up here is a bug" that shows up at the top of at least one of the pipeline tests - it should probably just say something like "Any invalidation that shows up in a non-abandoned analysis is a bug".
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D84959/new/
https://reviews.llvm.org/D84959
More information about the llvm-commits
mailing list