diff --git a/clang/include/clang/APINotes/Types.h b/clang/include/clang/APINotes/Types.h index 79c8079191fef..354458588e309 100644 --- a/clang/include/clang/APINotes/Types.h +++ b/clang/include/clang/APINotes/Types.h @@ -660,6 +660,10 @@ class TagInfo : public CommonTypeInfo { unsigned IsFlagEnum : 1; public: + std::optional SwiftImportAs; + std::optional SwiftRetainOp; + std::optional SwiftReleaseOp; + std::optional EnumExtensibility; TagInfo() : HasFlagEnum(0), IsFlagEnum(0) {} @@ -677,6 +681,13 @@ class TagInfo : public CommonTypeInfo { TagInfo &operator|=(const TagInfo &RHS) { static_cast(*this) |= RHS; + if (!SwiftImportAs) + SwiftImportAs = RHS.SwiftImportAs; + if (!SwiftRetainOp) + SwiftRetainOp = RHS.SwiftRetainOp; + if (!SwiftReleaseOp) + SwiftReleaseOp = RHS.SwiftReleaseOp; + if (!HasFlagEnum) setFlagEnum(RHS.isFlagEnum()); @@ -693,6 +704,9 @@ class TagInfo : public CommonTypeInfo { inline bool operator==(const TagInfo &LHS, const TagInfo &RHS) { return static_cast(LHS) == RHS && + LHS.SwiftImportAs == RHS.SwiftImportAs && + LHS.SwiftRetainOp == RHS.SwiftRetainOp && + LHS.SwiftReleaseOp == RHS.SwiftReleaseOp && LHS.isFlagEnum() == RHS.isFlagEnum() && LHS.EnumExtensibility == RHS.EnumExtensibility; } diff --git a/clang/lib/APINotes/APINotesFormat.h b/clang/lib/APINotes/APINotesFormat.h index b52f017901dbc..5897b45d3796d 100644 --- a/clang/lib/APINotes/APINotesFormat.h +++ b/clang/lib/APINotes/APINotesFormat.h @@ -24,7 +24,7 @@ const uint16_t VERSION_MAJOR = 0; /// API notes file minor version number. /// /// When the format changes IN ANY WAY, this number should be incremented. -const uint16_t VERSION_MINOR = 24; // EnumExtensibility + FlagEnum +const uint16_t VERSION_MINOR = 25; // SwiftImportAs using IdentifierID = llvm::PointerEmbeddedInt; using IdentifierIDField = llvm::BCVBR<16>; diff --git a/clang/lib/APINotes/APINotesWriter.cpp b/clang/lib/APINotes/APINotesWriter.cpp index 0126c5ec0798d..a92b379a8e56a 100644 --- a/clang/lib/APINotes/APINotesWriter.cpp +++ b/clang/lib/APINotes/APINotesWriter.cpp @@ -1124,7 +1124,10 @@ class CommonTypeTableInfo class TagTableInfo : public CommonTypeTableInfo { public: unsigned getUnversionedInfoSize(const TagInfo &TI) { - return 1 + getCommonTypeInfoSize(TI); + return 2 + (TI.SwiftImportAs ? TI.SwiftImportAs->size() : 0) + + 2 + (TI.SwiftRetainOp ? TI.SwiftRetainOp->size() : 0) + + 2 + (TI.SwiftReleaseOp ? TI.SwiftReleaseOp->size() : 0) + + 1 + getCommonTypeInfoSize(TI); } void emitUnversionedInfo(raw_ostream &OS, const TagInfo &TI) { @@ -1142,6 +1145,25 @@ class TagTableInfo : public CommonTypeTableInfo { writer.write(Flags); + if (auto ImportAs = TI.SwiftImportAs) { + writer.write(ImportAs->size() + 1); + OS.write(ImportAs->c_str(), ImportAs->size()); + } else { + writer.write(0); + } + if (auto RetainOp = TI.SwiftRetainOp) { + writer.write(RetainOp->size() + 1); + OS.write(RetainOp->c_str(), RetainOp->size()); + } else { + writer.write(0); + } + if (auto ReleaseOp = TI.SwiftReleaseOp) { + writer.write(ReleaseOp->size() + 1); + OS.write(ReleaseOp->c_str(), ReleaseOp->size()); + } else { + writer.write(0); + } + emitCommonTypeInfo(OS, TI); } }; diff --git a/clang/lib/APINotes/APINotesYAMLCompiler.cpp b/clang/lib/APINotes/APINotesYAMLCompiler.cpp index e430956e28d08..647455111214c 100644 --- a/clang/lib/APINotes/APINotesYAMLCompiler.cpp +++ b/clang/lib/APINotes/APINotesYAMLCompiler.cpp @@ -410,6 +410,9 @@ struct Tag { std::optional SwiftPrivate; std::optional SwiftBridge; std::optional NSErrorDomain; + std::optional SwiftImportAs; + std::optional SwiftRetainOp; + std::optional SwiftReleaseOp; std::optional EnumExtensibility; std::optional FlagEnum; std::optional EnumConvenienceKind; @@ -440,6 +443,9 @@ template <> struct MappingTraits { IO.mapOptional("SwiftName", T.SwiftName, StringRef("")); IO.mapOptional("SwiftBridge", T.SwiftBridge); IO.mapOptional("NSErrorDomain", T.NSErrorDomain); + IO.mapOptional("SwiftImportAs", T.SwiftImportAs); + IO.mapOptional("SwiftReleaseOp", T.SwiftReleaseOp); + IO.mapOptional("SwiftRetainOp", T.SwiftRetainOp); IO.mapOptional("EnumExtensibility", T.EnumExtensibility); IO.mapOptional("FlagEnum", T.FlagEnum); IO.mapOptional("EnumKind", T.EnumConvenienceKind);