[llvm-branch-commits] [GISel] Remove DoNotRun hook from Localizer (PR #217192)

[via llvm-branch-commits]

llvmorg-github-actions[bot] wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-llvm-globalisel

Author: Aiden Grossman (boomanaiden154)

<details>
<summary>Changes</summary>

No upstream targets use this, so remove it.


---
Full diff: https://github.com/llvm/llvm-project/pull/217192.diff


2 Files Affected:

- (modified) llvm/include/llvm/CodeGen/GlobalISel/Localizer.h (-5) 
- (modified) llvm/lib/CodeGen/GlobalISel/Localizer.cpp (+1-8) 


``````````diff
diff --git a/llvm/include/llvm/CodeGen/GlobalISel/Localizer.h b/llvm/include/llvm/CodeGen/GlobalISel/Localizer.h
index 909c99b1e6591..5ef1452e41cf1 100644
--- a/llvm/include/llvm/CodeGen/GlobalISel/Localizer.h
+++ b/llvm/include/llvm/CodeGen/GlobalISel/Localizer.h
@@ -45,10 +45,6 @@ class LLVM_ABI Localizer : public MachineFunctionPass {
   static char ID;
 
 private:
-  /// An input function to decide if the pass should run or not
-  /// on the given MachineFunction.
-  std::function<bool(const MachineFunction &)> DoNotRunPass;
-
   /// MRI contains all the register class/bank information that this
   /// pass uses and updates.
   MachineRegisterInfo *MRI = nullptr;
@@ -80,7 +76,6 @@ class LLVM_ABI Localizer : public MachineFunctionPass {
 
 public:
   Localizer();
-  Localizer(std::function<bool(const MachineFunction &)>);
 
   StringRef getPassName() const override { return "Localizer"; }
 
diff --git a/llvm/lib/CodeGen/GlobalISel/Localizer.cpp b/llvm/lib/CodeGen/GlobalISel/Localizer.cpp
index 77ca27bcc43e2..9b8b36d1843b6 100644
--- a/llvm/lib/CodeGen/GlobalISel/Localizer.cpp
+++ b/llvm/lib/CodeGen/GlobalISel/Localizer.cpp
@@ -33,11 +33,8 @@ INITIALIZE_PASS_END(Localizer, DEBUG_TYPE,
                     "Move/duplicate certain instructions close to their use",
                     false, false)
 
-Localizer::Localizer(std::function<bool(const MachineFunction &)> F)
-    : MachineFunctionPass(ID), DoNotRunPass(F) {}
-
 Localizer::Localizer()
-    : Localizer([](const MachineFunction &) { return false; }) {}
+    : MachineFunctionPass(ID) {}
 
 void Localizer::init(MachineFunction &MF) {
   MRI = &MF.getRegInfo();
@@ -206,10 +203,6 @@ bool Localizer::runOnMachineFunction(MachineFunction &MF) {
   if (MF.getProperties().hasFailedISel())
     return false;
 
-  // Don't run the pass if the target asked so.
-  if (DoNotRunPass(MF))
-    return false;
-
   LLVM_DEBUG(dbgs() << "Localize instructions for: " << MF.getName() << '\n');
 
   init(MF);

``````````

</details>


https://github.com/llvm/llvm-project/pull/217192