[PATCH] D63527: InstCombine: Preserve nuw when reassociating nuw ops [2/3]

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri accepted this revision.
lebedev.ri added a comment.
This revision is now accepted and ready to land.

Alive agrees here, this looks good for `add`, `mul`.
https://rise4fun.com/Alive/vkbp
https://rise4fun.com/Alive/6Rd



================
Comment at: lib/Transforms/InstCombine/InstructionCombining.cpp:380
 
     if (I.isAssociative() && I.isCommutative()) {
       if (simplifyAssocCastAssoc(&I)) {
----------------
So for the purpose of this patch `add`/`mul` only, correct?


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

https://reviews.llvm.org/D63527