[PATCH] D60656: [LVI][CVP] Calculate with.overflow result range

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 10 15:17:55 PDT 2019


spatel added a comment.

In D60656#1498504 <https://reviews.llvm.org/D60656#1498504>, @nikic wrote:

> @spatel What do you think about going for the following abstraction instead? What I have in mind here is to make this also reusable for operations other than `binaryOp()`, such as `uadd_sat()` and friends.


I haven't spent much time in here, so I was just trying to avoid code duplication with my suggestion. If you see a better way and more enhancement potential, that's good with me. :)


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

https://reviews.llvm.org/D60656





More information about the llvm-commits mailing list