[PATCH] D102696: [Analyzer] Find constraints that are directly attached to a BinOp
Artem Dergachev via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed May 19 13:00:24 PDT 2021
NoQ added a comment.
In D102696#2767894 <https://reviews.llvm.org/D102696#2767894>, @martong wrote:
>> In this case the equations are $y == 0 and $x + 0 == 0 which is much easier to solve.
>
> Yes, you are right.
>
>> This happens for the same reason that your patch is needed in the first place: we're eagerly substituting a constant.
>
> Absolutely, that's the point. On the other hand, it is very important to emphasize that we cannot solve this problem with a stronger solver, see my example with 3 variables and two equations above.
Well, in that other example, we should be asking about `$x + $y + 0 == 0` at some point. And then the solver should be able to deduce it from `$x + $y + $z == 0` and `$z == 0`. If we're not asking this question, that's obviously the first problem to get out of our way.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D102696/new/
https://reviews.llvm.org/D102696
More information about the cfe-commits
mailing list