[llvm] a747e86 - [InstCombine] Fold `fpto{s|u}i non-norm` to zero (#85569)
via llvm-commits
llvm-commits at lists.llvm.org
Tue Mar 19 02:16:53 PDT 2024
Author: Yingwei Zheng
Date: 2024-03-19T17:16:48+08:00
New Revision: a747e86caacf0ba3d5272474da8c10eb0b67dbc2
URL: https://github.com/llvm/llvm-project/commit/a747e86caacf0ba3d5272474da8c10eb0b67dbc2
DIFF: https://github.com/llvm/llvm-project/commit/a747e86caacf0ba3d5272474da8c10eb0b67dbc2.diff
LOG: [InstCombine] Fold `fpto{s|u}i non-norm` to zero (#85569)
This patch enables more optimization after canonicalizing `fmul X, 0.0`
into a copysign.
I decide to implement this fold in InstCombine because
`computeKnownFPClass` may be expensive.
Alive2: https://alive2.llvm.org/ce/z/ASM8tQ
Added:
Modified:
llvm/lib/Transforms/InstCombine/InstCombineCasts.cpp
llvm/test/Transforms/InstCombine/fpcast.ll
Removed:
################################################################################
diff --git a/llvm/lib/Transforms/InstCombine/InstCombineCasts.cpp b/llvm/lib/Transforms/InstCombine/InstCombineCasts.cpp
index 381cd858d26293..b19da1e075819c 100644
--- a/llvm/lib/Transforms/InstCombine/InstCombineCasts.cpp
+++ b/llvm/lib/Transforms/InstCombine/InstCombineCasts.cpp
@@ -1923,10 +1923,26 @@ Instruction *InstCombinerImpl::foldItoFPtoI(CastInst &FI) {
return replaceInstUsesWith(FI, X);
}
+static Instruction *foldFPtoI(Instruction &FI, InstCombiner &IC) {
+ // fpto{u/s}i non-norm --> 0
+ FPClassTest Mask =
+ FI.getOpcode() == Instruction::FPToUI ? fcPosNormal : fcNormal;
+ KnownFPClass FPClass =
+ computeKnownFPClass(FI.getOperand(0), Mask, /*Depth=*/0,
+ IC.getSimplifyQuery().getWithInstruction(&FI));
+ if (FPClass.isKnownNever(Mask))
+ return IC.replaceInstUsesWith(FI, ConstantInt::getNullValue(FI.getType()));
+
+ return nullptr;
+}
+
Instruction *InstCombinerImpl::visitFPToUI(FPToUIInst &FI) {
if (Instruction *I = foldItoFPtoI(FI))
return I;
+ if (Instruction *I = foldFPtoI(FI, *this))
+ return I;
+
return commonCastTransforms(FI);
}
@@ -1934,6 +1950,9 @@ Instruction *InstCombinerImpl::visitFPToSI(FPToSIInst &FI) {
if (Instruction *I = foldItoFPtoI(FI))
return I;
+ if (Instruction *I = foldFPtoI(FI, *this))
+ return I;
+
return commonCastTransforms(FI);
}
diff --git a/llvm/test/Transforms/InstCombine/fpcast.ll b/llvm/test/Transforms/InstCombine/fpcast.ll
index 88ca556f4d8f8b..1f0bfbc40ac1b6 100644
--- a/llvm/test/Transforms/InstCombine/fpcast.ll
+++ b/llvm/test/Transforms/InstCombine/fpcast.ll
@@ -347,3 +347,91 @@ define double @masked_uint_to_fpext3(i32 %x) {
%r = fpext float %f to double
ret double %r
}
+
+define i32 @fptosi_nonnorm(float nofpclass(norm) %x) {
+; CHECK-LABEL: @fptosi_nonnorm(
+; CHECK-NEXT: ret i32 0
+;
+ %ret = fptosi float %x to i32
+ ret i32 %ret
+}
+
+define i32 @fptoui_nonnorm(float nofpclass(pnorm) %x) {
+; CHECK-LABEL: @fptoui_nonnorm(
+; CHECK-NEXT: ret i32 0
+;
+ %ret = fptoui float %x to i32
+ ret i32 %ret
+}
+
+define i32 @fptosi_nonnnorm(float nofpclass(nnorm) %x) {
+; CHECK-LABEL: @fptosi_nonnnorm(
+; CHECK-NEXT: [[RET:%.*]] = fptosi float [[X:%.*]] to i32
+; CHECK-NEXT: ret i32 [[RET]]
+;
+ %ret = fptosi float %x to i32
+ ret i32 %ret
+}
+
+define i32 @fptoui_nonnnorm(float nofpclass(nnorm) %x) {
+; CHECK-LABEL: @fptoui_nonnnorm(
+; CHECK-NEXT: [[RET:%.*]] = fptoui float [[X:%.*]] to i32
+; CHECK-NEXT: ret i32 [[RET]]
+;
+ %ret = fptoui float %x to i32
+ ret i32 %ret
+}
+
+define i32 @fptosi_nonnorm_copysign(float %x) {
+; CHECK-LABEL: @fptosi_nonnorm_copysign(
+; CHECK-NEXT: ret i32 0
+;
+ %val = call float @llvm.copysign.f32(float 0.0, float %x)
+ %ret = fptosi float %val to i32
+ ret i32 %ret
+}
+
+define <2 x i32> @fptosi_nonnorm_copysign_vec(<2 x float> %x) {
+; CHECK-LABEL: @fptosi_nonnorm_copysign_vec(
+; CHECK-NEXT: ret <2 x i32> zeroinitializer
+;
+ %val = call <2 x float> @llvm.copysign.v2f32(<2 x float> zeroinitializer, <2 x float> %x)
+ %ret = fptosi <2 x float> %val to <2 x i32>
+ ret <2 x i32> %ret
+}
+
+define i32 @fptosi_nonnorm_fmul(float %x) {
+; CHECK-LABEL: @fptosi_nonnorm_fmul(
+; CHECK-NEXT: [[SEL:%.*]] = fmul float [[X:%.*]], 0.000000e+00
+; CHECK-NEXT: [[RET:%.*]] = fptosi float [[SEL]] to i32
+; CHECK-NEXT: ret i32 [[RET]]
+;
+ %sel = fmul float %x, 0.000000e+00
+ %ret = fptosi float %sel to i32
+ ret i32 %ret
+}
+
+define i32 @fptosi_select(i1 %cond) {
+; CHECK-LABEL: @fptosi_select(
+; CHECK-NEXT: [[RET:%.*]] = select i1 [[COND:%.*]], i32 1, i32 -1
+; CHECK-NEXT: ret i32 [[RET]]
+;
+ %sel = select i1 %cond, float 1.0, float -1.0
+ %ret = fptosi float %sel to i32
+ ret i32 %ret
+}
+
+define i32 @mul_pos_zero_convert(i32 %a) {
+; CHECK-LABEL: @mul_pos_zero_convert(
+; CHECK-NEXT: entry:
+; CHECK-NEXT: [[FP:%.*]] = sitofp i32 [[A:%.*]] to float
+; CHECK-NEXT: [[RET:%.*]] = fmul float [[FP]], 0.000000e+00
+; CHECK-NEXT: [[CONV:%.*]] = fptosi float [[RET]] to i32
+; CHECK-NEXT: ret i32 [[CONV]]
+;
+entry:
+ %fp = sitofp i32 %a to float
+ %ret = fmul float %fp, 0.000000e+00
+ %conv = fptosi float %ret to i32
+ ret i32 %conv
+}
More information about the llvm-commits
mailing list