[PATCH] D50144: Add Windows support for the GNUstep Objective-C ABI V2.

Saleem Abdulrasool via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 14 10:54:27 PDT 2018


compnerd added inline comments.


================
Comment at: lib/AST/MicrosoftMangle.cpp:1886
   case BuiltinType::ObjCId:
-    mangleArtificalTagType(TTK_Struct, "objc_object");
+    mangleArtificalTagType(TTK_Struct, ".objc_object");
     break;
----------------
DHowett-MSFT wrote:
> I'm interested in @smeenai's take on this, as well.
Hmm, doesn't this break ObjC/ObjC++ interoperability?  I don't think that we should be changing the decoration here for the MS ABI case.


Repository:
  rC Clang

https://reviews.llvm.org/D50144





More information about the cfe-commits mailing list