[PATCH] D100095: [InstCombine] Conditionally emitting nsw/nuw flags when combining two add operations

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 13 11:44:56 PDT 2021


nikic added a comment.

Yeah, the implementation looks good to me now. Could you please also add a test for this case (that shows nuw is not preserved for negative numbers)?

If you don't have commit access, please also share your `Name <and at email>` to use when landing this change.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D100095/new/

https://reviews.llvm.org/D100095



More information about the llvm-commits mailing list