[PATCH] D68651: [InstCombine] Signed saturation patterns

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 10 09:27:20 PDT 2019


lebedev.ri added a comment.

In D68651#1703888 <https://reviews.llvm.org/D68651#1703888>, @dmgreen wrote:

> > .. on where and how to draw the line ..
>
> Not sure about in general though!
>
> > - and get it documented (and the document to be kept up-to-date).
>
> Any ideas what it would look like? And where this would live? At some point when we get into enough of the details the code becomes the best documentation, but I'm guessing you will disagree with that...


I'm not sure this can really be documented and enforced.


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

https://reviews.llvm.org/D68651





More information about the llvm-commits mailing list