[PATCH] D77062: [analyzer] Improve zero assumption in CStringChecke::assumeZero
Denys Petrov via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Oct 1 05:47:25 PDT 2020
ASDenysPetrov added a comment.
@steakhal
> @ASDenysPetrov Do you still want to rework the API of the `assumeZero`?
This patch more looks like NFC, being just refactored.
Actually I see that if we find and fix the root cause, we can freely refuse this patch.
Another thing I see is that this patch will work after a root cause fix as well.
And the last one is that as long a root cause stays unfixed, clang will emit an assertion without this patch, at least for my testcase.
So, I can't really evaluate this objectively. What do you think?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D77062/new/
https://reviews.llvm.org/D77062
More information about the cfe-commits
mailing list