[PATCH] D100917: [NewPM] Only invalidate modified functions' analyses in CGSCC passes

Arthur Eubanks via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 21 21:34:43 PDT 2021


aeubanks added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/FunctionAttrs.cpp:1631
+  for (Function *Changed : ChangedFunctions)
+    FAM.invalidate(*Changed, PreservedAnalyses::none());
+  // We have not changed the call graph or removed/added functions.
----------------
ychen wrote:
> Is the function CFG preserved?
Yes, will update.
Doing this causes globalaa-retained.ll to fail, which I believe is fixed by https://reviews.llvm.org/D101017


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100917



More information about the llvm-commits mailing list