[PATCH] D18953: [ms][dll] #26935 Defining a dllimport function should cause it to be exported

[David Majnemer via cfe-commits]

majnemer added a subscriber: majnemer.

================
Comment at: test/SemaCXX/dllimport.cpp:179
@@ -140,1 +178,3 @@
+template <typename T>
+int ExternVarTmplDeclInit = 1;
 
----------------
avt77 wrote:
> rnk wrote:
> > Can you check with MSVC 2015 update 2 actually does with definitions of dllimport variable templates? I bet it doesn't export them.
> They don't support variable templates at all:
> 
> error C2399: variable templates are not supported in this release
Your compiler is too old, they are definitely supported.

> Previously a template declaration was only allowed to be a function, class, or alias. Now, in the MSVC compiler it can be a variable as well.

https://blogs.msdn.microsoft.com/vcblog/2016/02/11/compiler-improvements-in-vs-2015-update-2/


http://reviews.llvm.org/D18953