[llvm] [Reassociate] Preserve NUW flags after expr tree rewriting (PR #72360)

Nikita Popov via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 15 02:45:02 PST 2023


================
@@ -1225,7 +1234,7 @@ Value *ReassociatePass::RemoveFactorFromExpression(Value *V, Value *Factor) {
     RedoInsts.insert(BO);
     V = Factors[0].Op;
   } else {
-    RewriteExprTree(BO, Factors);
+    RewriteExprTree(BO, Factors, HasNUW);
----------------
nikic wrote:

Could there be a problem if NeedsNegate is set?

https://github.com/llvm/llvm-project/pull/72360


More information about the llvm-commits mailing list