[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