[PATCH] D52443: Thread safety analysis: Examine constructor arguments

Delesley Hutchins via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 27 15:27:09 PDT 2018


delesley added a comment.

This looks good, and resolves an outstanding bug that was on my list.   Thanks for the patch!



================
Comment at: lib/Analysis/ThreadSafety.cpp:1537
   void handleCall(const Expr *Exp, const NamedDecl *D, VarDecl *VD = nullptr);
+  void ExamineCallArguments(const FunctionDecl *FD,
+                            CallExpr::const_arg_iterator ArgBegin,
----------------
Nit: capitalization does not match other methods.


Repository:
  rC Clang

https://reviews.llvm.org/D52443





More information about the cfe-commits mailing list