[llvm] [Inliner] Handle CGSCC changes from #94815 (PR #96274)
Mircea Trofin via llvm-commits
llvm-commits at lists.llvm.org
Fri Jun 21 07:42:43 PDT 2024
================
@@ -187,8 +187,8 @@ unsigned MLInlineAdvisor::getInitialFunctionLevel(const Function &F) const {
return CG.lookup(F) ? FunctionLevels.at(CG.lookup(F)) : 0;
}
-void MLInlineAdvisor::onPassEntry(LazyCallGraph::SCC *LastSCC) {
- if (!LastSCC || ForceStop)
+void MLInlineAdvisor::onPassEntry(LazyCallGraph::SCC *CurSCC) {
----------------
mtrofin wrote:
IIUC s/LastSCC/CurSCC is a clarity refactoring, the semantics didn't change, correct?
we could just do a quick NFC change with just this renaming, to keep the change focused to just the fallout from 94815? (up to you - I don't think that precursor change should need review, and happy to do it, too, if you want)
https://github.com/llvm/llvm-project/pull/96274
More information about the llvm-commits
mailing list