[PATCH] D42672: [CFG] [analyzer] Heavier CFGConstructor elements.

[Devin Coughlin via Phabricator via cfe-commits]

dcoughlin accepted this revision.
dcoughlin added a comment.
This revision is now accepted and ready to land.

This looks good to me, although as I noted inline I think both the name and the comment for VisitForConstructionContext() are confusing. If you can be more precise I think it would help future maintainers.



================
Comment at: lib/Analysis/CFG.cpp:653
+  // triggered by the given trigger.
+  void VisitForConstructionContext(Stmt *Trigger, Stmt *Child);
+
----------------
I find the name of this method very confusing. It sounds like you are visiting an AST node called a 'ForConstructionContext', but that is not what this function does. It is also not clear from either the name or the comment what the method is supposed to do. Can you be more precise?

You can fix this in a follow-up commit if you like.


https://reviews.llvm.org/D42672