[clang] [ItaniumMangle] Use mangleType instead of mangleNameOrStandardSubstitution in mangleCXXCtorVTable function (PR #109970)

via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 25 20:11:21 PDT 2024


================
@@ -7326,11 +7326,13 @@ void ItaniumMangleContextImpl::mangleCXXCtorVTable(const CXXRecordDecl *RD,
                                                    raw_ostream &Out) {
   // <special-name> ::= TC <type> <offset number> _ <base type>
   CXXNameMangler Mangler(*this, Out);
+  QualType RDType = getASTContext().getRecordType(RD);
+  QualType TypeType = getASTContext().getRecordType(Type);
   Mangler.getStream() << "_ZTC";
-  Mangler.mangleNameOrStandardSubstitution(RD);
+  Mangler.mangleType(RDType);
----------------
tcwzxx wrote:

> I can't find this production in the ItaniumABI docs, but if the comment is right, I think performing this substitution is correct.

I also can't find any Itanium ABI documentation, but I found that GCC follows the same process. So, I believe the comment is correct, and it is necessary to perform the substitution for RD.

https://github.com/llvm/llvm-project/pull/109970


More information about the cfe-commits mailing list