[PATCH] D58872: [InstCombine] Start canonicalizing to uadd.sat and usub.sat

[Nikita Popov via Phabricator via llvm-commits]

nikic added a comment.

In D58872#1416090 <https://reviews.llvm.org/D58872#1416090>, @lebedev.ri wrote:

> Now that it's a single instruction, what about one-use checks there?


I've left them in place to avoid redundant calculations if the intrinsic needs to be expanded in the backend. If that's not a concern I can drop them (apart from the case that inserts an extra sub).


Repository:
  rL LLVM

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

https://reviews.llvm.org/D58872