[PATCH] D39031: [Analyzer] Correctly handle parameters passed by reference when bodyfarming std::call_once

Daniel Marjamäki via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 19 08:31:09 PDT 2017


danielmarjamaki added a reviewer: danielmarjamaki.
danielmarjamaki added a comment.

Stylistically this looks pretty good to me. Just a minor nit.



================
Comment at: lib/Analysis/BodyFarm.cpp:389
+  for (unsigned int i = 2; i < D->getNumParams(); i++) {
+
+    const ParmVarDecl *PDecl = D->getParamDecl(i);
----------------
ehm.. I would remove this blank


https://reviews.llvm.org/D39031





More information about the cfe-commits mailing list