[llvm] 4475470 - [ModuleInliner] Remove a stale comment (NFC)

[Kazu Hirata via llvm-commits]

Author: Kazu Hirata
Date: 2022-09-16T09:37:43-07:00
New Revision: 44754705290abd6bde8509221610ad96936aa422

URL: https://github.com/llvm/llvm-project/commit/44754705290abd6bde8509221610ad96936aa422
DIFF: https://github.com/llvm/llvm-project/commit/44754705290abd6bde8509221610ad96936aa422.diff

LOG: [ModuleInliner] Remove a stale comment (NFC)

These comments refer to the nested loop in the module inliner where
the inner loop grouped call sites from the same caller.  We don't
group call sites anymore, so the comment has become stale.

Added: 
    

Modified: 
    llvm/lib/Transforms/IPO/ModuleInliner.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Transforms/IPO/ModuleInliner.cpp b/llvm/lib/Transforms/IPO/ModuleInliner.cpp
index 8ae9bc3aa1de..b86a5ef6fef7 100644
--- a/llvm/lib/Transforms/IPO/ModuleInliner.cpp
+++ b/llvm/lib/Transforms/IPO/ModuleInliner.cpp
@@ -195,10 +195,6 @@ PreservedAnalyses ModuleInlinerPass::run(Module &M,
 
   // Loop forward over all of the calls.
   while (!Calls->empty()) {
-    // We expect the calls to typically be batched with sequences of calls that
-    // have the same caller, so we first set up some shared infrastructure for
-    // this caller. We also do any pruning we can at this layer on the caller
-    // alone.
     Function &F = *Calls->front().first->getCaller();
 
     LLVM_DEBUG(dbgs() << "Inlining calls in: " << F.getName() << "\n"
@@ -209,9 +205,6 @@ PreservedAnalyses ModuleInlinerPass::run(Module &M,
       return FAM.getResult<AssumptionAnalysis>(F);
     };
 
-    // Now process as many calls as we have within this caller in the sequence.
-    // We bail out as soon as the caller has to change so we can
-    // prepare the context of that new caller.
     auto P = Calls->pop();
     CallBase *CB = P.first;
     const int InlineHistoryID = P.second;