ARM: Remove unnecessary use_empty check

[arsenm]

The previous statement removed all uses.

[arsenm]

• ARM: Remove unnecessary use_empty check #137338 👈 (View in Graphite)
• ARM: Use use_empty instead of hasNUses(0) #137337
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[llvmbot]

@llvm/pr-subscribers-backend-arm

Author: Matt Arsenault (arsenm)

Changes

The previous statement removed all uses.

---

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

1 Files Affected:

• (modified) llvm/lib/Target/ARM/MVEGatherScatterLowering.cpp (+1-2)

diff --git a/llvm/lib/Target/ARM/MVEGatherScatterLowering.cpp b/llvm/lib/Target/ARM/MVEGatherScatterLowering.cpp
index e998359bd3496..ce59ae0c95dcf 100644
--- a/llvm/lib/Target/ARM/MVEGatherScatterLowering.cpp
+++ b/llvm/lib/Target/ARM/MVEGatherScatterLowering.cpp
@@ -1099,8 +1099,7 @@ bool MVEGatherScatterLowering::optimiseOffsets(Value *Offsets, BasicBlock *BB,
 
   // The instruction has now been "absorbed" into the phi value
   Offs->replaceAllUsesWith(NewPhi);
-  if (Offs->use_empty())
-    Offs->eraseFromParent();
+  Offs->eraseFromParent();
   // Clean up the old increment in case it's unused because we built a new
   // one
   if (IncInstruction->use_empty())

[arsenm]

Merge activity

• Apr 25, 11:24 AM EDT: A user started a stack merge that includes this pull request via Graphite.
• Apr 25, 11:32 AM EDT: Graphite rebased this pull request as part of a merge.
• Apr 25, 11:34 AM EDT: @arsenm merged this pull request with Graphite.