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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits