[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