[PATCH] D34714: [MS] Don't statically initialize dllimport member function pointers

Reid Kleckner via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jun 27 17:53:31 PDT 2017


rnk added a comment.

Did you locally add a test case for the dllimport member function pointer template argument?



================
Comment at: lib/Sema/SemaTemplate.cpp:5704
+  else
+    NPV = isNullPointerValueTemplateArgument(S, Param, ParamType, ResultArg);
+
----------------
Think we should pass in Entity as an optional parameter to isNullPointerValueTemplateArgument to share the logic?


https://reviews.llvm.org/D34714





More information about the cfe-commits mailing list