[llvm] [InstCombine] fold icmp of select with invertible shl (PR #147182)
via llvm-commits
llvm-commits at lists.llvm.org
Sat Oct 25 06:26:09 PDT 2025
================
@@ -5948,6 +5954,29 @@ static Instruction *foldICmpEqualityWithOffset(ICmpInst &I,
collectOffsetOp(Op1, OffsetOps, /*AllowRecursion=*/true);
auto ApplyOffsetImpl = [&](Value *V, unsigned BinOpc, Value *RHS) -> Value * {
+ switch (BinOpc) {
+ // V = shl nsw X, RHS => X = ashr V, RHS
+ case Instruction::AShr: {
+ const APInt *CV, *CRHS;
+ if (!(match(V, m_APInt(CV)) && match(RHS, m_APInt(CRHS)) &&
+ CV->ashr(*CRHS).shl(*CRHS) == *CV) &&
+ !match(V, m_NSWShl(m_Value(), m_Specific(RHS))))
+ return nullptr;
+ break;
+ }
+ // V = shl nuw X, RHS => X = lshr V, RHS
+ case Instruction::LShr: {
+ const APInt *CV, *CRHS;
+ if (!(match(V, m_APInt(CV)) && match(RHS, m_APInt(CRHS)) &&
+ CV->lshr(*CRHS).shl(*CRHS) == *CV) &&
+ !match(V, m_NUWShl(m_Value(), m_Specific(RHS))))
+ return nullptr;
+ break;
+ }
+ default:
+ break;
+ }
----------------
actinks wrote:
Yes, as we discussed [above](https://github.com/llvm/llvm-project/pull/147182#issuecomment-3060152336). The logic for checking non-constants here is unnecessary, and I will remove it.
https://github.com/llvm/llvm-project/pull/147182
More information about the llvm-commits
mailing list