Author: Brett Wilson Date: 2022-08-12T18:39:20Z New Revision: 68266828b170d2f978d0bc2f508de2b3b9e41665
URL: https://github.com/llvm/llvm-project/commit/68266828b170d2f978d0bc2f508de2b3b9e41665 DIFF: https://github.com/llvm/llvm-project/commit/68266828b170d2f978d0bc2f508de2b3b9e41665.diff LOG: [clang-doc] Always emit the TagType for RecordInfo Always emit the TagType for RecordInfo in YAML output. Previously this omitted the type for "struct", considering it the default. But records in C++ don't really have a default type so always emitting this is more clear. Emit IsTypeDef in YAML. Previously this existed only in the Representation but was never written. Additionally, adds IsTypeDef to the record merge operation which was clearing it (all RecordInfo structures are merged with am empty RecordInfo during the reduce phase). Reviewed By: paulkirth Differential Revision: https://reviews.llvm.org/D131739 Added: Modified: clang-tools-extra/clang-doc/Representation.cpp clang-tools-extra/clang-doc/Representation.h clang-tools-extra/clang-doc/YAMLGenerator.cpp clang-tools-extra/unittests/clang-doc/MergeTest.cpp clang-tools-extra/unittests/clang-doc/YAMLGeneratorTest.cpp Removed: ################################################################################ diff --git a/clang-tools-extra/clang-doc/Representation.cpp b/clang-tools-extra/clang-doc/Representation.cpp index 0a78d8c190db9..deb508d9e92bb 100644 --- a/clang-tools-extra/clang-doc/Representation.cpp +++ b/clang-tools-extra/clang-doc/Representation.cpp @@ -222,6 +222,7 @@ void RecordInfo::merge(RecordInfo &&Other) { assert(mergeable(Other)); if (!TagType) TagType = Other.TagType; + IsTypeDef = IsTypeDef || Other.IsTypeDef; if (Members.empty()) Members = std::move(Other.Members); if (Bases.empty()) diff --git a/clang-tools-extra/clang-doc/Representation.h b/clang-tools-extra/clang-doc/Representation.h index 201480393e891..2738b6f5ca63a 100644 --- a/clang-tools-extra/clang-doc/Representation.h +++ b/clang-tools-extra/clang-doc/Representation.h @@ -348,10 +348,15 @@ struct RecordInfo : public SymbolInfo { void merge(RecordInfo &&I); - TagTypeKind TagType = TagTypeKind::TTK_Struct; // Type of this record - // (struct, class, union, - // interface). - bool IsTypeDef = false; // Indicates if record was declared using typedef + // Type of this record (struct, class, union, interface). + TagTypeKind TagType = TagTypeKind::TTK_Struct; + + // Indicates if the record was declared using a typedef. Things like anonymous + // structs in a typedef: + // typedef struct { ... } foo_t; + // are converted into records with the typedef as the Name + this flag set. + bool IsTypeDef = false; + llvm::SmallVector<MemberTypeInfo, 4> Members; // List of info about record members. llvm::SmallVector<Reference, 4> Parents; // List of base/parent records diff --git a/clang-tools-extra/clang-doc/YAMLGenerator.cpp b/clang-tools-extra/clang-doc/YAMLGenerator.cpp index 58cb4c8bfa602..ff7abee2eccf9 100644 --- a/clang-tools-extra/clang-doc/YAMLGenerator.cpp +++ b/clang-tools-extra/clang-doc/YAMLGenerator.cpp @@ -127,7 +127,8 @@ static void SymbolInfoMapping(IO &IO, SymbolInfo &I) { static void RecordInfoMapping(IO &IO, RecordInfo &I) { SymbolInfoMapping(IO, I); - IO.mapOptional("TagType", I.TagType, clang::TagTypeKind::TTK_Struct); + IO.mapOptional("TagType", I.TagType); + IO.mapOptional("IsTypeDef", I.IsTypeDef, false); IO.mapOptional("Members", I.Members); IO.mapOptional("Bases", I.Bases); IO.mapOptional("Parents", I.Parents, llvm::SmallVector<Reference, 4>()); diff --git a/clang-tools-extra/unittests/clang-doc/MergeTest.cpp b/clang-tools-extra/unittests/clang-doc/MergeTest.cpp index 31b565352ee20..3647ebdbc3771 100644 --- a/clang-tools-extra/unittests/clang-doc/MergeTest.cpp +++ b/clang-tools-extra/unittests/clang-doc/MergeTest.cpp @@ -78,6 +78,7 @@ TEST(MergeTest, mergeNamespaceInfos) { TEST(MergeTest, mergeRecordInfos) { RecordInfo One; One.Name = "r"; + One.IsTypeDef = true; One.Namespace.emplace_back(EmptySID, "A", InfoType::IT_namespace); One.DefLoc = Location(10, llvm::SmallString<16>{"test.cpp"}); @@ -119,6 +120,7 @@ TEST(MergeTest, mergeRecordInfos) { auto Expected = std::make_unique<RecordInfo>(); Expected->Name = "r"; + Expected->IsTypeDef = true; Expected->Namespace.emplace_back(EmptySID, "A", InfoType::IT_namespace); Expected->DefLoc = Location(10, llvm::SmallString<16>{"test.cpp"}); diff --git a/clang-tools-extra/unittests/clang-doc/YAMLGeneratorTest.cpp b/clang-tools-extra/unittests/clang-doc/YAMLGeneratorTest.cpp index b87b62bd9a4ec..ea30bfdefadae 100644 --- a/clang-tools-extra/unittests/clang-doc/YAMLGeneratorTest.cpp +++ b/clang-tools-extra/unittests/clang-doc/YAMLGeneratorTest.cpp @@ -76,6 +76,7 @@ TEST(YAMLGeneratorTest, emitRecordYAML) { RecordInfo I; I.Name = "r"; I.Path = "path/to/A"; + I.IsTypeDef = true; I.Namespace.emplace_back(EmptySID, "A", InfoType::IT_namespace); I.DefLoc = Location(10, llvm::SmallString<16>{"test.cpp"}); @@ -136,6 +137,7 @@ Path: 'path/to/A' - LineNumber: 12 Filename: 'test.cpp' TagType: Class +IsTypeDef: true Members: - Type: Name: 'int' @@ -154,6 +156,7 @@ TagType: Class - USR: '0000000000000000000000000000000000000000' Name: 'F' Path: 'path/to/F' + TagType: Struct Members: - Type: Name: 'int' _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
