[PATCH] D80707: [legacyPM] Do not compute preserved analysis if there's no local change

serge via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun May 31 23:08:24 PDT 2020


serge-sans-paille added a comment.



> Perhaps if a pass returns false we should verify all analyses, rather than just the ones that the pass claims to preserve?

That sounds like a good idea! Do you think that should be guarded by NDEBUG or by EXPENSIVE_CHECK?


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

https://reviews.llvm.org/D80707





More information about the llvm-commits mailing list