[PATCH] D55715: Add AddressSpace mangling to MS mode

David Majnemer via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Dec 14 11:44:43 PST 2018


majnemer added inline comments.


================
Comment at: lib/AST/MicrosoftMangle.cpp:1806-1836
+    Extra.mangleSourceName("AS_");
+    Extra.mangleIntegerLiteral(llvm::APSInt::getUnsigned(TargetAS),
+                               /*IsBoolean*/ false);
+  } else {
+    switch (AS) {
+    default:
+      llvm_unreachable("Not a language specific address space");
----------------
Don't these need to be _AS_Foobar to avoid collisions with normal code?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D55715/new/

https://reviews.llvm.org/D55715





More information about the cfe-commits mailing list