[PATCH] Start adding support for dllimport/dllexport on classes (PR11170)

Nico Rieck nico.rieck at gmail.com
Thu May 29 17:42:52 PDT 2014


================
Comment at: lib/Sema/SemaDeclCXX.cpp:4385
@@ +4384,3 @@
+    NewAttr->setInherited(true);
+    Member->addAttr(NewAttr);
+
----------------
Reid Kleckner wrote:
> We shouldn't be adding these attributes to FieldDecls.  I suppose we want to export nested classes, though.
I'd use a whitelist and only propagate it to methods and static data members. Nested classes aren't automatically exported or imported.

http://reviews.llvm.org/D3877






More information about the cfe-commits mailing list