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

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri marked an inline comment as done.
lebedev.ri added inline comments.


================
Comment at: lib/Transforms/InstCombine/InstructionCombining.cpp:380
 
     if (I.isAssociative() && I.isCommutative()) {
       if (simplifyAssocCastAssoc(&I)) {
----------------
arsenm wrote:
> lebedev.ri wrote:
> > So for the purpose of this patch `add`/`mul` only, correct?
> Yes. Bit-ops are also handled here, but they don't have the flags to preserve
Yes, that's what i meant.


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

https://reviews.llvm.org/D63527