================
Comment at: lib/AST/MicrosoftMangle.cpp:1647
@@ -1645,3 +1646,3 @@
       }
-      Out << "@Z";
+      Out << 'Z';
       return;
----------------
This changes from "XX@Z" to "XXZ" for the Ctor_DefaultClosure case. Just double 
checking that that's intended.

================
Comment at: lib/CodeGen/CodeGenTypes.h:268
@@ -268,1 +267,3 @@
+  const CGFunctionInfo &arrangeMSCtorClosure(const CXXConstructorDecl *CD,
+                                                 CXXCtorType CT);
 
----------------
nit: indent is off?

================
Comment at: test/CodeGenCXX/dllexport.cpp:496
@@ -487,2 +495,3 @@
+};
 
 struct __declspec(dllexport) T {
----------------
Add a test where dllexport is on the class rather than the member?

http://reviews.llvm.org/D8331

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



_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to