[llvm] llvm-reduce: Avoid worklist in simplify-instruction (PR #134066)

[Matt Arsenault via llvm-commits]

https://github.com/arsenm created https://github.com/llvm/llvm-project/pull/134066

None

>From c715634cc41f788fad01f6745eff6eea388528b6 Mon Sep 17 00:00:00 2001
From: Matt Arsenault <Matthew.Arsenault at amd.com>
Date: Wed, 2 Apr 2025 17:31:45 +0700
Subject: [PATCH] llvm-reduce: Avoid worklist in simplify-instruction

---
 .../tools/llvm-reduce/deltas/SimplifyInstructions.cpp | 11 ++---------
 1 file changed, 2 insertions(+), 9 deletions(-)

diff --git a/llvm/tools/llvm-reduce/deltas/SimplifyInstructions.cpp b/llvm/tools/llvm-reduce/deltas/SimplifyInstructions.cpp
index 7eb381d3c1905..ee92b7fb88731 100644
--- a/llvm/tools/llvm-reduce/deltas/SimplifyInstructions.cpp
+++ b/llvm/tools/llvm-reduce/deltas/SimplifyInstructions.cpp
@@ -20,27 +20,20 @@ using namespace llvm;
 /// Calls simplifyInstruction in each instruction in functions, and replaces
 /// their values.
 void llvm::simplifyInstructionsDeltaPass(Oracle &O, ReducerWorkItem &WorkItem) {
-  std::vector<Instruction *> InstsToDelete;
-
   Module &Program = WorkItem.getModule();
   const DataLayout &DL = Program.getDataLayout();
 
-  std::vector<Instruction *> InstToDelete;
   for (auto &F : Program) {
     for (auto &BB : F) {
-      for (auto &Inst : BB) {
-
+      for (auto &Inst : make_early_inc_range(BB)) {
         SimplifyQuery Q(DL, &Inst);
         if (Value *Simplified = simplifyInstruction(&Inst, Q)) {
           if (O.shouldKeep())
             continue;
           Inst.replaceAllUsesWith(Simplified);
-          InstToDelete.push_back(&Inst);
+          Inst.eraseFromParent();
         }
       }
     }
   }
-
-  for (Instruction *I : InstToDelete)
-    I->eraseFromParent();
 }