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