Szelethus requested changes to this revision.
Szelethus added a reviewer: steakhal.
Szelethus added a comment.
This revision now requires changes to proceed.

This patch is really cool, but I still feel anxious a bit about duplicating so 
much functionality, especially since we're working very hard to make 
`CallEvent` a widespread thing. @steakhal's patch D72035 
<https://reviews.llvm.org/D72035> already made some great progress in this 
direction, and should land before this one. @NoQ, do you agree that we should 
maybe stop for a bit before making the code a bit more consistent with the rest 
of the checkers?

I added @steakhal as reviewer if you don't mind.


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

https://reviews.llvm.org/D71524



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

Reply via email to