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

David Majnemer via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jun 27 18:48:07 PDT 2017


majnemer added a comment.

In https://reviews.llvm.org/D34714#793205, @rnk wrote:

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


Arg, yes. Forgot to add the file...



================
Comment at: lib/Sema/SemaTemplate.cpp:5704
+  else
+    NPV = isNullPointerValueTemplateArgument(S, Param, ParamType, ResultArg);
+
----------------
rnk wrote:
> Think we should pass in Entity as an optional parameter to isNullPointerValueTemplateArgument to share the logic?
Seems reasonable, I'll give that a go.


https://reviews.llvm.org/D34714





More information about the cfe-commits mailing list