[PATCH] D103096: [analyzer] Implement cast for ranges of symbolic integers.

Denys Petrov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 16 13:15:53 PDT 2021


ASDenysPetrov added a comment.

In D103096#2821750 <https://reviews.llvm.org/D103096#2821750>, @vsavchenko wrote:

> Hey, great work!  I think that casts are extremely important, but it looks like you mixed so many things into this patch.  Let's make one step at a time a split it into (at least) a couple of patches.

Thanks for the tips. I'll adress them in the next update. Actually, I thought about splitting before the first upload and splitted it into D103094 <https://reviews.llvm.org/D103094> and the current one. This particular patch provides full mechanism implementing feasibility of the test cases. Honestly, I don't know what part could be cut to keep this mechanism holistic and self-sufficient. But I'll see what i can do.


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

https://reviews.llvm.org/D103096



More information about the cfe-commits mailing list