From 15d9d483f9b8c5cced7ead987ba55a369651a76a Mon Sep 17 00:00:00 2001 From: Zheng Junjie Date: Wed, 15 Nov 2023 00:30:04 +0800 Subject: [PATCH] [InstCombine] Follow-up to "When -A + B both have nsw flag, set nsw flag." In 3c037b7306f57039e24a1470687cc39a795584ac, I've miss suggested about cast. --- llvm/lib/Transforms/InstCombine/InstCombineAddSub.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/llvm/lib/Transforms/InstCombine/InstCombineAddSub.cpp b/llvm/lib/Transforms/InstCombine/InstCombineAddSub.cpp index d390d3a581fca..2bd738c95ecaa 100644 --- a/llvm/lib/Transforms/InstCombine/InstCombineAddSub.cpp +++ b/llvm/lib/Transforms/InstCombine/InstCombineAddSub.cpp @@ -1488,7 +1488,7 @@ Instruction *InstCombinerImpl::visitAdd(BinaryOperator &I) { // -A + B --> B - A auto *Sub = BinaryOperator::CreateSub(RHS, A); - auto *OB0 = dyn_cast(LHS); + auto *OB0 = cast(LHS); Sub->setHasNoSignedWrap(I.hasNoSignedWrap() && OB0->hasNoSignedWrap()); return Sub;