diff options
author | Chuanqi Xu <yedeng.yd@linux.alibaba.com> | 2024-05-13 13:56:25 +0800 |
---|---|---|
committer | Chuanqi Xu <yedeng.yd@linux.alibaba.com> | 2024-05-13 14:05:02 +0800 |
commit | e74a34b6932965dfdc182b69f779e5bee551585a (patch) | |
tree | a6a3f7ffd7056012f9e55d3f09cfbb5c6ed83716 /clang/lib | |
parent | e6785fd75284f53b9e23db6f249598e09f3fc39f (diff) | |
download | llvm-e74a34b6932965dfdc182b69f779e5bee551585a.zip llvm-e74a34b6932965dfdc182b69f779e5bee551585a.tar.gz llvm-e74a34b6932965dfdc182b69f779e5bee551585a.tar.bz2 |
[NFC] [Serialization] Merge IdentID with IdentifierID
In ASTBitCodes.h, there are two type alias for the ID type of
Identifiers with the same underlying type. It is confusing. This patch
tries to merge the `IdentID` to `IdentifierID` to erase such confusion.
Diffstat (limited to 'clang/lib')
-rw-r--r-- | clang/lib/Frontend/FrontendAction.cpp | 2 | ||||
-rw-r--r-- | clang/lib/Frontend/MultiplexConsumer.cpp | 2 | ||||
-rw-r--r-- | clang/lib/Serialization/ASTReader.cpp | 4 | ||||
-rw-r--r-- | clang/lib/Serialization/ASTReaderInternals.h | 2 | ||||
-rw-r--r-- | clang/lib/Serialization/ASTWriter.cpp | 18 |
5 files changed, 14 insertions, 14 deletions
diff --git a/clang/lib/Frontend/FrontendAction.cpp b/clang/lib/Frontend/FrontendAction.cpp index 9ae7664..a9c45e5 100644 --- a/clang/lib/Frontend/FrontendAction.cpp +++ b/clang/lib/Frontend/FrontendAction.cpp @@ -71,7 +71,7 @@ public: if (Previous) Previous->ReaderInitialized(Reader); } - void IdentifierRead(serialization::IdentID ID, + void IdentifierRead(serialization::IdentifierID ID, IdentifierInfo *II) override { if (Previous) Previous->IdentifierRead(ID, II); diff --git a/clang/lib/Frontend/MultiplexConsumer.cpp b/clang/lib/Frontend/MultiplexConsumer.cpp index c74bfd8..8fdc7f5 100644 --- a/clang/lib/Frontend/MultiplexConsumer.cpp +++ b/clang/lib/Frontend/MultiplexConsumer.cpp @@ -35,7 +35,7 @@ void MultiplexASTDeserializationListener::ReaderInitialized( } void MultiplexASTDeserializationListener::IdentifierRead( - serialization::IdentID ID, IdentifierInfo *II) { + serialization::IdentifierID ID, IdentifierInfo *II) { for (size_t i = 0, e = Listeners.size(); i != e; ++i) Listeners[i]->IdentifierRead(ID, II); } diff --git a/clang/lib/Serialization/ASTReader.cpp b/clang/lib/Serialization/ASTReader.cpp index 78e4df4..7627996 100644 --- a/clang/lib/Serialization/ASTReader.cpp +++ b/clang/lib/Serialization/ASTReader.cpp @@ -1005,7 +1005,7 @@ static bool readBit(unsigned &Bits) { return Value; } -IdentID ASTIdentifierLookupTrait::ReadIdentifierID(const unsigned char *d) { +IdentifierID ASTIdentifierLookupTrait::ReadIdentifierID(const unsigned char *d) { using namespace llvm::support; unsigned RawID = endian::readNext<uint32_t, llvm::endianness::little>(d); @@ -1041,7 +1041,7 @@ IdentifierInfo *ASTIdentifierLookupTrait::ReadData(const internal_key_type& k, markIdentifierFromAST(Reader, *II); Reader.markIdentifierUpToDate(II); - IdentID ID = Reader.getGlobalIdentifierID(F, RawID); + IdentifierID ID = Reader.getGlobalIdentifierID(F, RawID); if (!IsInteresting) { // For uninteresting identifiers, there's nothing else to do. Just notify // the reader that we've finished loading this identifier. diff --git a/clang/lib/Serialization/ASTReaderInternals.h b/clang/lib/Serialization/ASTReaderInternals.h index 49268ad..536b19f 100644 --- a/clang/lib/Serialization/ASTReaderInternals.h +++ b/clang/lib/Serialization/ASTReaderInternals.h @@ -175,7 +175,7 @@ public: const unsigned char* d, unsigned DataLen); - IdentID ReadIdentifierID(const unsigned char *d); + IdentifierID ReadIdentifierID(const unsigned char *d); ASTReader &getReader() const { return Reader; } }; diff --git a/clang/lib/Serialization/ASTWriter.cpp b/clang/lib/Serialization/ASTWriter.cpp index ab07cf3..6154ead 100644 --- a/clang/lib/Serialization/ASTWriter.cpp +++ b/clang/lib/Serialization/ASTWriter.cpp @@ -3746,7 +3746,7 @@ public: using key_type = const IdentifierInfo *; using key_type_ref = key_type; - using data_type = IdentID; + using data_type = IdentifierID; using data_type_ref = data_type; using hash_value_type = unsigned; @@ -3775,7 +3775,7 @@ public: } std::pair<unsigned, unsigned> - EmitKeyDataLength(raw_ostream &Out, const IdentifierInfo *II, IdentID ID) { + EmitKeyDataLength(raw_ostream &Out, const IdentifierInfo *II, IdentifierID ID) { // Record the location of the identifier data. This is used when generating // the mapping from persistent IDs to strings. Writer.SetIdentifierOffset(II, Out.tell()); @@ -3807,7 +3807,7 @@ public: Out.write(II->getNameStart(), KeyLen); } - void EmitData(raw_ostream &Out, const IdentifierInfo *II, IdentID ID, + void EmitData(raw_ostream &Out, const IdentifierInfo *II, IdentifierID ID, unsigned) { using namespace llvm::support; @@ -3891,7 +3891,7 @@ void ASTWriter::WriteIdentifierTable(Preprocessor &PP, IdentifierOffsets.resize(NextIdentID - FirstIdentID); for (auto IdentIDPair : IdentifierIDs) { const IdentifierInfo *II = IdentIDPair.first; - IdentID ID = IdentIDPair.second; + IdentifierID ID = IdentIDPair.second; assert(II && "NULL identifier in identifier table"); // Write out identifiers if either the ID is local or the identifier has @@ -4779,7 +4779,7 @@ void ASTWriter::AddVersionTuple(const VersionTuple &Version, /// Note that the identifier II occurs at the given offset /// within the identifier table. void ASTWriter::SetIdentifierOffset(const IdentifierInfo *II, uint32_t Offset) { - IdentID ID = IdentifierIDs[II]; + IdentifierID ID = IdentifierIDs[II]; // Only store offsets new to this AST file. Other identifier names are looked // up earlier in the chain and thus don't need an offset. if (ID >= FirstIdentID) @@ -5945,11 +5945,11 @@ void ASTWriter::AddIdentifierRef(const IdentifierInfo *II, RecordDataImpl &Recor Record.push_back(getIdentifierRef(II)); } -IdentID ASTWriter::getIdentifierRef(const IdentifierInfo *II) { +IdentifierID ASTWriter::getIdentifierRef(const IdentifierInfo *II) { if (!II) return 0; - IdentID &ID = IdentifierIDs[II]; + IdentifierID &ID = IdentifierIDs[II]; if (ID == 0) ID = NextIdentID++; return ID; @@ -6610,9 +6610,9 @@ void ASTWriter::ReaderInitialized(ASTReader *Reader) { NextSubmoduleID = FirstSubmoduleID; } -void ASTWriter::IdentifierRead(IdentID ID, IdentifierInfo *II) { +void ASTWriter::IdentifierRead(IdentifierID ID, IdentifierInfo *II) { // Always keep the highest ID. See \p TypeRead() for more information. - IdentID &StoredID = IdentifierIDs[II]; + IdentifierID &StoredID = IdentifierIDs[II]; if (ID > StoredID) StoredID = ID; } |