Skip to content

Commit 01bcc3e

Browse files
committed
[InstCombine] prevent infinite loop with sub/abs of constant expression
PR45539: https://bugs.llvm.org/show_bug.cgi?id=45539
1 parent 3a61245 commit 01bcc3e

File tree

2 files changed

+20
-1
lines changed

2 files changed

+20
-1
lines changed

llvm/lib/Transforms/InstCombine/InstCombineAddSub.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1761,7 +1761,7 @@ Instruction *InstCombiner::visitSub(BinaryOperator &I) {
17611761
Constant *C2;
17621762

17631763
// C-(C2-X) --> X+(C-C2)
1764-
if (match(Op1, m_Sub(m_Constant(C2), m_Value(X))))
1764+
if (match(Op1, m_Sub(m_Constant(C2), m_Value(X))) && !isa<ConstantExpr>(C2))
17651765
return BinaryOperator::CreateAdd(X, ConstantExpr::getSub(C, C2));
17661766

17671767
// C-(X+C2) --> (C-C2)-X

llvm/test/Transforms/InstCombine/abs-1.ll

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -628,3 +628,22 @@ define i8 @nabs_different_constants(i8 %a) {
628628
%m2 = select i1 %cmp2, i8 %neg, i8 %a
629629
ret i8 %m2
630630
}
631+
632+
@g = external global i64
633+
634+
; PR45539 - https://bugs.llvm.org/show_bug.cgi?id=45539
635+
636+
define i64 @infinite_loop_constant_expression_abs(i64 %arg) {
637+
; CHECK-LABEL: @infinite_loop_constant_expression_abs(
638+
; CHECK-NEXT: [[T:%.*]] = sub i64 ptrtoint (i64* @g to i64), [[ARG:%.*]]
639+
; CHECK-NEXT: [[T1:%.*]] = icmp slt i64 [[T]], 0
640+
; CHECK-NEXT: [[T2:%.*]] = sub nsw i64 0, [[T]]
641+
; CHECK-NEXT: [[T3:%.*]] = select i1 [[T1]], i64 [[T2]], i64 [[T]]
642+
; CHECK-NEXT: ret i64 [[T3]]
643+
;
644+
%t = sub i64 ptrtoint (i64* @g to i64), %arg
645+
%t1 = icmp slt i64 %t, 0
646+
%t2 = sub nsw i64 0, %t
647+
%t3 = select i1 %t1, i64 %t2, i64 %t
648+
ret i64 %t3
649+
}

0 commit comments

Comments
 (0)