[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
Mon Jul 27 08:31:56 PDT 2020


serge-sans-paille marked an inline comment as done.
serge-sans-paille added inline comments.


================
Comment at: llvm/lib/IR/LegacyPassManager.cpp:1506-1507
     verifyPreservedAnalysis(FP);
-    removeNotPreservedAnalysis(FP);
+    if (LocalChanged)
+      removeNotPreservedAnalysis(FP);
     recordAvailableAnalysis(FP);
----------------
foad wrote:
> Do the same for LoopPass, RegionPass and CallGraphSCCPass?
AFAIU, these three passes are all handled by the FunctionPassmanager, so there shouldn't be anything to do.


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

https://reviews.llvm.org/D80707





More information about the llvm-commits mailing list