[PATCH] D145811: [SystemZ] Don't reuse a sub that can overflow during isel.

[Nikita Popov via Phabricator via llvm-commits]

nikic added inline comments.


================
Comment at: llvm/lib/Target/SystemZ/SystemZISelLowering.cpp:2426
       if (N->getOpcode() == ISD::SUB &&
+          (!N->getFlags().hasNoSignedWrap() && !N->getFlags().hasNoUnsignedWrap()) &&
           ((N->getOperand(0) == C.Op0 && N->getOperand(1) == C.Op1) ||
----------------
It would be better to drop the flags instead -- the general preference is to drop nowrap flags rather than block optimizations due to their presence.


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

https://reviews.llvm.org/D145811