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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to