[llvm] [InstCombine] Fold `(icmp pred (trunc nuw/nsw X), C)` -> `(icmp pred X, (zext/sext C))` (PR #87935)
via llvm-commits
llvm-commits at lists.llvm.org
Sat May 11 08:58:40 PDT 2024
================
@@ -1409,6 +1409,19 @@ Instruction *InstCombinerImpl::foldICmpTruncConstant(ICmpInst &Cmp,
const APInt &C) {
ICmpInst::Predicate Pred = Cmp.getPredicate();
Value *X = Trunc->getOperand(0);
+ Type *SrcTy = X->getType();
+ unsigned DstBits = Trunc->getType()->getScalarSizeInBits(),
+ SrcBits = SrcTy->getScalarSizeInBits();
+
+ // Match (icmp pred (trunc nuw/nsw X), C)
+ // Which we can convert to (icmp pred X, (sext/zext C))
+ if (shouldChangeType(DstBits, SrcBits)) {
----------------
goldsteinn wrote:
Sure, in this case the `shouldChnageType` that accepts types just forwards the `primitiveBitWidth` to the bitwidth helper, but if you think this version is a more future proof happy to change.
https://github.com/llvm/llvm-project/pull/87935
More information about the llvm-commits
mailing list