[clang] [serialization] No transitive type change (PR #92511)

Chuanqi Xu via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 20 03:54:32 PDT 2024


================
@@ -7392,27 +7388,28 @@ QualType ASTReader::GetType(TypeID ID) {
   return TypesLoaded[Index].withFastQualifiers(FastQuals);
 }
 
-QualType ASTReader::getLocalType(ModuleFile &F, unsigned LocalID) {
+QualType ASTReader::getLocalType(ModuleFile &F, TypeID LocalID) {
----------------
ChuanqiXu9 wrote:

Agreed. I tried to address this by introducing a new type alias `using LocalTypeID = TypeID;` with a FIXME that we'd better to make them classes like we did for DeclID. Then at least I believe the interface will have better readability.

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


More information about the cfe-commits mailing list