InstCombine: Remove a check for pointer bitcasts

[arsenm]

No description provided.

[arsenm]

• InstCombine: Remove a check for pointer bitcasts #128491 👈 (View in Graphite)
• main

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

[llvmbot]

@llvm/pr-subscribers-llvm-transforms

Author: Matt Arsenault (arsenm)

Changes

---

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

1 Files Affected:

• (modified) llvm/lib/Transforms/InstCombine/InstructionCombining.cpp (+1-2)

diff --git a/llvm/lib/Transforms/InstCombine/InstructionCombining.cpp b/llvm/lib/Transforms/InstCombine/InstructionCombining.cpp
index 81b057c10b484..90cb499947601 100644
--- a/llvm/lib/Transforms/InstCombine/InstructionCombining.cpp
+++ b/llvm/lib/Transforms/InstCombine/InstructionCombining.cpp
@@ -3661,8 +3661,7 @@ Instruction *InstCombinerImpl::visitUnconditionalBranchInst(BranchInst &BI) {
 
   auto GetLastSinkableStore = [](BasicBlock::iterator BBI) {
     auto IsNoopInstrForStoreMerging = [](BasicBlock::iterator BBI) {
-      return BBI->isDebugOrPseudoInst() ||
-             (isa<BitCastInst>(BBI) && BBI->getType()->isPointerTy());
+      return BBI->isDebugOrPseudoInst();
     };
 
     BasicBlock::iterator FirstInstr = BBI->getParent()->begin();

[nikic]

Inline the lambda, now that it's trivial?

[arsenm]

Wasn't sure if this should be updated to include assumes