Author: Prajwal Nadig Date: 2025-07-07T09:14:47-06:00 New Revision: 0f48bafafc38581da6f7388b4f3cc143cdbbdede
URL: https://github.com/llvm/llvm-project/commit/0f48bafafc38581da6f7388b4f3cc143cdbbdede DIFF: https://github.com/llvm/llvm-project/commit/0f48bafafc38581da6f7388b4f3cc143cdbbdede.diff LOG: [ExtractAPI] Include tilde in destructor name (#146001) The subheading for a destructor contained only the identifier. The tilde must also be included as it is necessary to differentiate the destructor from any constructors present. rdar://129587608 Added: Modified: clang/lib/ExtractAPI/DeclarationFragments.cpp clang/test/ExtractAPI/constructor_destructor.cpp Removed: ################################################################################ diff --git a/clang/lib/ExtractAPI/DeclarationFragments.cpp b/clang/lib/ExtractAPI/DeclarationFragments.cpp index 52349324d7829..791afc1a97575 100644 --- a/clang/lib/ExtractAPI/DeclarationFragments.cpp +++ b/clang/lib/ExtractAPI/DeclarationFragments.cpp @@ -1617,10 +1617,13 @@ DeclarationFragmentsBuilder::getFunctionSignature(const ObjCMethodDecl *); DeclarationFragments DeclarationFragmentsBuilder::getSubHeading(const NamedDecl *Decl) { DeclarationFragments Fragments; - if (isa<CXXConstructorDecl>(Decl) || isa<CXXDestructorDecl>(Decl)) + if (isa<CXXConstructorDecl>(Decl)) { Fragments.append(cast<CXXRecordDecl>(Decl->getDeclContext())->getName(), DeclarationFragments::FragmentKind::Identifier); - else if (isa<CXXConversionDecl>(Decl)) { + } else if (isa<CXXDestructorDecl>(Decl)) { + Fragments.append(cast<CXXDestructorDecl>(Decl)->getNameAsString(), + DeclarationFragments::FragmentKind::Identifier); + } else if (isa<CXXConversionDecl>(Decl)) { Fragments.append( cast<CXXConversionDecl>(Decl)->getConversionType().getAsString(), DeclarationFragments::FragmentKind::Identifier); @@ -1634,9 +1637,11 @@ DeclarationFragmentsBuilder::getSubHeading(const NamedDecl *Decl) { } else if (Decl->getIdentifier()) { Fragments.append(Decl->getName(), DeclarationFragments::FragmentKind::Identifier); - } else + } else { Fragments.append(Decl->getDeclName().getAsString(), DeclarationFragments::FragmentKind::Identifier); + } + return Fragments; } diff --git a/clang/test/ExtractAPI/constructor_destructor.cpp b/clang/test/ExtractAPI/constructor_destructor.cpp index 27112c95ac45c..2f2150a6d0da0 100644 --- a/clang/test/ExtractAPI/constructor_destructor.cpp +++ b/clang/test/ExtractAPI/constructor_destructor.cpp @@ -213,7 +213,7 @@ class Foo { "subHeading": [ { "kind": "identifier", - "spelling": "Foo" + "spelling": "~Foo" } ], "title": "~Foo" _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits