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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to