majnemer added inline comments.
================
Comment at: lib/CodeGen/CGVTables.cpp:887-888
+ if (CGM.getTriple().isWindowsGNUEnvironment() &&
RD->hasAttr<DLLImportAttr>())
+ return true;
+
----------------
Maybe a comment like "VTables of classes declared as dllimport are always
considered to be external." ?
================
Comment at: lib/CodeGen/CGVTables.cpp:896-900
// Otherwise, if the class is an instantiated template, the
// vtable must be defined here.
if (TSK == TSK_ImplicitInstantiation ||
TSK == TSK_ExplicitInstantiationDefinition)
return false;
----------------
It would be good to have tests for what would have happened if these paths got
hit.
https://reviews.llvm.org/D42641
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
