[PATCH] D69252: [InstCombine] Canonicalize sadd.with.overflow to sadd.sat

[Dave Green via Phabricator via llvm-commits]

dmgreen updated this revision to Diff 228627.
dmgreen marked 2 inline comments as done.
dmgreen added a comment.

Right. I was aiming for a symmetry between the adds and the subs. But you are probably correct that it's better simplified like this.


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

https://reviews.llvm.org/D69252

Files:
  llvm/lib/Transforms/InstCombine/InstCombineSelect.cpp
  llvm/test/Transforms/InstCombine/overflow_to_sat.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D69252.228627.patch
Type: text/x-patch
Size: 10645 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20191110/2194043d/attachment-0001.bin>