[PATCH] D48283: [SCEV] Properly solve quadratic equations

Krzysztof Parzyszek via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 2 11:30:40 PDT 2018


kparzysz added inline comments.


================
Comment at: include/llvm/ADT/APInt.h:2193
+/// an *unsigned* overflow (if the values are treated as unsigned).
+/// To find a solution for a signed overflow, use a range with of BW-1.
+///
----------------
sanjoy wrote:
> Do you meant to say that "if the coeffs are positive then use a range of BW-1 to find a solution for signed overflow"?
It should work with with any coefficients.  I will clarify the comments.


Repository:
  rL LLVM

https://reviews.llvm.org/D48283





More information about the llvm-commits mailing list