[PATCH] D60020: [DAGCombiner][X86] Canonicalize SSUBO with immediate RHS to SADDO by negating the immediate.

[Craig Topper via Phabricator via llvm-commits]

craig.topper updated this revision to Diff 193380.
craig.topper added a comment.

-Add a check for INT_MIN.
-Update a SystemZ test I missed cause I only had X86 building previously. I think this one of these cases is a regression, but that's probably also going to happen with the InstCombine version of this patch too. So probably needs to be fixed regardless.


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

https://reviews.llvm.org/D60020

Files:
  llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
  llvm/test/CodeGen/SystemZ/int-ssub-07.ll
  llvm/test/CodeGen/X86/sub-with-overflow.ll
  llvm/test/CodeGen/X86/xaluo.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D60020.193380.patch
Type: text/x-patch
Size: 4580 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190402/9ab43bab/attachment.bin>