[llvm] a5a2e2f - [CodeGen]RegisterCoalescer: Remove the print() method (#125042)

[via llvm-commits]

Author: Akshat Oke
Date: 2025-01-31T11:45:55+05:30
New Revision: a5a2e2fc62824287aca02e80ce237c695ea89000

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

LOG: [CodeGen]RegisterCoalescer: Remove the print() method (#125042)

I saw that there used to be an overload `operator<<(OS, const Pass &P)`.

Now that it's gone, I still don't find anyone using the print method
except the internal debug printing hidden under `LLVM_DEBUG` in this
function.

Added: 
    

Modified: 
    llvm/lib/CodeGen/RegisterCoalescer.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/CodeGen/RegisterCoalescer.cpp b/llvm/lib/CodeGen/RegisterCoalescer.cpp
index 62c15949e46dac..d2f7684ec83a48 100644
--- a/llvm/lib/CodeGen/RegisterCoalescer.cpp
+++ b/llvm/lib/CodeGen/RegisterCoalescer.cpp
@@ -390,9 +390,6 @@ class RegisterCoalescer : public MachineFunctionPass,
 
   /// This is the pass entry point.
   bool runOnMachineFunction(MachineFunction &) override;
-
-  /// Implement the dump method.
-  void print(raw_ostream &O, const Module * = nullptr) const override;
 };
 
 } // end anonymous namespace
@@ -4344,12 +4341,8 @@ bool RegisterCoalescer::runOnMachineFunction(MachineFunction &fn) {
   PHIValToPos.clear();
   RegToPHIIdx.clear();
 
-  LLVM_DEBUG(dump());
+  LLVM_DEBUG(LIS->dump());
   if (VerifyCoalescing)
     MF->verify(this, "After register coalescing", &errs());
   return true;
 }
-
-void RegisterCoalescer::print(raw_ostream &O, const Module *m) const {
-  LIS->print(O);
-}