Index: lib/Sema/SemaDeclCXX.cpp =================================================================== --- lib/Sema/SemaDeclCXX.cpp +++ lib/Sema/SemaDeclCXX.cpp @@ -5476,7 +5476,7 @@ } } -static void ReferenceDllExportedMethods(Sema &S, CXXRecordDecl *Class) { +static void ReferenceDllExportedMembers(Sema &S, CXXRecordDecl *Class) { Attr *ClassAttr = getDLLAttr(Class); if (!ClassAttr) return; @@ -5491,6 +5491,16 @@ return; for (Decl *Member : Class->decls()) { + // Defined static variables that are members of an exported base + // class must be marked export too. Push them to implicit instantiation + // queue. + auto *VD = dyn_cast(Member); + if (VD && Member->getAttr() && + VD->getStorageClass() == SC_Static && + TSK == TSK_ImplicitInstantiation) + S.PendingLocalImplicitInstantiations.push_back( + std::make_pair(VD, VD->getLocation())); + auto *MD = dyn_cast(Member); if (!MD) continue; @@ -10902,12 +10912,12 @@ void Sema::referenceDLLExportedClassMethods() { if (!DelayedDllExportClasses.empty()) { - // Calling ReferenceDllExportedMethods might cause the current function to + // Calling ReferenceDllExportedMembers might cause the current function to // be called again, so use a local copy of DelayedDllExportClasses. SmallVector WorkList; std::swap(DelayedDllExportClasses, WorkList); for (CXXRecordDecl *Class : WorkList) - ReferenceDllExportedMethods(*this, Class); + ReferenceDllExportedMembers(*this, Class); } } Index: test/CodeGenCXX/dllexport.cpp =================================================================== --- test/CodeGenCXX/dllexport.cpp +++ test/CodeGenCXX/dllexport.cpp @@ -28,6 +28,7 @@ // The vftable for struct W is comdat largest because we have RTTI. // M32-DAG: $"\01??_7W@@6B@" = comdat largest +// M32-DAG: $"\01?smember@?$Base@H@PR32992@@0HA" = comdat any //===----------------------------------------------------------------------===// @@ -977,3 +978,21 @@ // MSVC2015-DAG: define weak_odr dllexport {{.+}}ACE_Service_Object@@Q{{.+}}@$$Q // The declarations should not be exported. // MSVC2013-NOT: define weak_odr dllexport {{.+}}ACE_Service_Object@@Q{{.+}}@$$Q + +namespace PR32992 { +// Static data members of a instantiated base class should be exported. +template +class Base { + virtual void myfunc() {} + static int smember; +}; +// MS-DAG: @"\01?smember@?$Base@H@PR32992@@0HA" = weak_odr dllexport global i32 77, comdat, align 4 +template int Base::smember = 77; +template +class __declspec(dllexport) Derived2 : Base { + void myfunc() {} +}; +class Derived : public Derived2 { + void myfunc() {} +}; +} // namespace PR32992