[PATCH] Don't dllimport/export class members with internal linkage (PR23308)

Nico Weber thakis at chromium.org
Tue Apr 21 21:04:59 PDT 2015


Looks good to me, thanks.


================
Comment at: lib/Sema/SemaDeclCXX.cpp:4807
@@ -4806,1 +4806,3 @@
 
+    if (!cast<NamedDecl>(Member)->isExternallyVisible()) {
+      continue;
----------------
nit: no braces

http://reviews.llvm.org/D9182

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list