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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to