[PATCH] D113304: [NewPM] Only invalidate modified functions' analyses in CGSCC passes + turn on eagerly invalidate analyses

Arthur Eubanks via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 8 15:32:36 PST 2021


aeubanks added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/Inliner.cpp:1017
+    // invalidate analyses for all functions in this SCC later.
+    FAM.invalidate(F, PreservedAnalyses::none());
   }
----------------
mtrofin wrote:
> Should we do this if !Changed? Actually, if this function did not change (`Changed` is per cgscc)
we already bail out at line 966 if this function hasn't been changed


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113304



More information about the llvm-commits mailing list