[PATCH] D41944: [LLVM][IR][LIT] support of 'no-overflow' flag for sdiv\udiv instructions

Mohammed Agabaria via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jan 13 23:09:05 PST 2018


magabari added a comment.

In https://reviews.llvm.org/D41944#974178, @sanjoy wrote:

> This looks like something that will be better served by an intrinsic (`llvm.safe_(s|u)div` or something like that), at least to begin with.  Experimental intrinsics are a low-cost preferred way of trying out ideas like this without changing fundamental IR semantics.


@Sanjoy we tried that in our first RFC (https://groups.google.com/forum/#!msg/llvm-dev/ooMhG28jZG0/UP5GadYvCQAJ;context-place=msg/llvm-dev/eFtnCwpMMhs/eAHQj8rJCAAJ )
and it has been rejected with this recommendation.


Repository:
  rL LLVM

https://reviews.llvm.org/D41944





More information about the llvm-commits mailing list