[PATCH] D34523: AST: mangle BlockDecls under MS ABI

David Majnemer via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Jun 24 16:39:28 PDT 2017


majnemer added inline comments.


================
Comment at: lib/AST/MicrosoftMangle.cpp:985
+
+      Out << '?' << Discriminate("_block_invoke", Discriminator) << '@';
+      if (const auto *RD = dyn_cast<RecordDecl>(DC))
----------------
Should this be `Out << '?' << mangleSourceName(Discriminate("_block_invoke", Discriminator));`


================
Comment at: lib/AST/MicrosoftMangle.cpp:990-994
+      Out << "YAX";
+      Out << "PA";
+      mangleArtificalTagType(TTK_Struct,
+                             Discriminate("__block_literal", Discriminator));
+      Out << "@Z";
----------------
You should probably add comments explaining these bits.


Repository:
  rL LLVM

https://reviews.llvm.org/D34523





More information about the cfe-commits mailing list