[llvm] c4d74a9 - [InstCombine] add test for abs with dominating condition; NFC
Sanjay Patel via llvm-commits
llvm-commits at lists.llvm.org
Tue Mar 22 07:42:57 PDT 2022
Author: Sanjay Patel
Date: 2022-03-22T10:42:52-04:00
New Revision: c4d74a93f65c07db4d80e199291e0da378305731
URL: https://github.com/llvm/llvm-project/commit/c4d74a93f65c07db4d80e199291e0da378305731
DIFF: https://github.com/llvm/llvm-project/commit/c4d74a93f65c07db4d80e199291e0da378305731.diff
LOG: [InstCombine] add test for abs with dominating condition; NFC
There's a potential miscompile or missed optimization with
propagating 'nsw' in the transform proposed in D122013, so
we need at least one more test for coverage.
Added:
Modified:
llvm/test/Transforms/InstCombine/abs-intrinsic.ll
Removed:
################################################################################
diff --git a/llvm/test/Transforms/InstCombine/abs-intrinsic.ll b/llvm/test/Transforms/InstCombine/abs-intrinsic.ll
index ed987b0a80b3b..f314a53d65b31 100644
--- a/llvm/test/Transforms/InstCombine/abs-intrinsic.ll
+++ b/llvm/test/Transforms/InstCombine/abs-intrinsic.ll
@@ -480,6 +480,33 @@ cond.end:
ret i32 %r
}
+define i32 @sub_abs_lt_min_not_poison(i32 %x, i32 %y) {
+; CHECK-LABEL: @sub_abs_lt_min_not_poison(
+; CHECK-NEXT: entry:
+; CHECK-NEXT: [[CMP:%.*]] = icmp slt i32 [[X:%.*]], [[Y:%.*]]
+; CHECK-NEXT: br i1 [[CMP]], label [[COND_TRUE:%.*]], label [[COND_END:%.*]]
+; CHECK: cond.true:
+; CHECK-NEXT: [[SUB:%.*]] = sub nsw i32 [[X]], [[Y]]
+; CHECK-NEXT: [[TMP0:%.*]] = call i32 @llvm.abs.i32(i32 [[SUB]], i1 false)
+; CHECK-NEXT: br label [[COND_END]]
+; CHECK: cond.end:
+; CHECK-NEXT: [[R:%.*]] = phi i32 [ [[TMP0]], [[COND_TRUE]] ], [ 0, [[ENTRY:%.*]] ]
+; CHECK-NEXT: ret i32 [[R]]
+;
+entry:
+ %cmp = icmp slt i32 %x, %y
+ br i1 %cmp, label %cond.true, label %cond.end
+
+cond.true:
+ %sub = sub nsw i32 %x, %y
+ %0 = call i32 @llvm.abs.i32(i32 %sub, i1 false)
+ br label %cond.end
+
+cond.end:
+ %r = phi i32 [ %0, %cond.true ], [ 0, %entry ]
+ ret i32 %r
+}
+
define i32 @sub_abs_wrong_pred(i32 %x, i32 %y) {
; CHECK-LABEL: @sub_abs_wrong_pred(
; CHECK-NEXT: entry:
More information about the llvm-commits
mailing list