[PATCH] D77062: [analyzer] Improve zero assumption in CStringChecke::assumeZero

Denys Petrov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 24 06:09:51 PDT 2020


ASDenysPetrov added a comment.

@steakhal

> I would not accept this patch unless this investigation is done. However, I'm not inherently against this change.

Actually I've done the investigation. You can find it here https://reviews.llvm.org/D77062#1977613

The bug is somewhere deep in the core. I see two solutions:

1. Leave the crash presented in the clang and dig deep into the core trying to solve the root cause.
2. Accept the change with a TODO promt and dig deep into the core trying to solve the root cause.




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

https://reviews.llvm.org/D77062



More information about the cfe-commits mailing list