================
Comment at: lib/AST/MicrosoftMangle.cpp:2313
@@ -2313,4 +2312,3 @@
   MicrosoftCXXNameMangler Mangler(*this, Out);
-  Mangler.getStream() << "_CT??_R0";
-  Mangler.mangleType(T, SourceRange(), MicrosoftCXXNameMangler::QMM_Result);
-  Mangler.getStream() << "@8";
+  Mangler.getStream() << "_CT";
+
----------------
rnk wrote:
> Does this get mangled to __CT... on 32-bit? Do we need \01?
Yes, it gets mangled to __CT on 32-bit which would mean that we wouldn't want 
\01, right?

================
Comment at: lib/AST/MicrosoftMangle.cpp:2315-2320
@@ +2314,8 @@
+
+  llvm::SmallString<64> RTTIMangling;
+  {
+    llvm::raw_svector_ostream Stream(RTTIMangling);
+    mangleCXXRTTI(T, Stream);
+  }
+  Mangler.getStream() << RTTIMangling.substr(1);
+
----------------
rnk wrote:
> Stupid '\01' mangling. When we drop it, we can simply run the mangler on the 
> Out stream.
Happily :)

http://reviews.llvm.org/D8101

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