[PATCH] D66381: [analyzer] Enable control dependency condition tracking by default

Kristóf Umann via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 21 13:58:16 PDT 2019


Szelethus added a comment.

In D66381#1639679 <https://reviews.llvm.org/D66381#1639679>, @NoQ wrote:

> The code looks good and there seems to be a lot of test coverage :]


Thank you! It would not have been possible on my own! And actually, there are a couple minor nits, such as caching the retrieved `CFGBlock` for each `ExplodedNode` (at least for the node we're calculating control dependencies for), so I'll address that soon™.


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

https://reviews.llvm.org/D66381





More information about the cfe-commits mailing list