Author: Chuanqi Xu Date: 2024-06-24T11:20:09+08:00 New Revision: 4061354ab3157255238cf2664b9482bd6297dc83
URL: https://github.com/llvm/llvm-project/commit/4061354ab3157255238cf2664b9482bd6297dc83 DIFF: https://github.com/llvm/llvm-project/commit/4061354ab3157255238cf2664b9482bd6297dc83.diff LOG: [Serialization] Revert specialization for DenseMapInfo<GlobalDeclID>::getHashValue The FIXME says to revert this when the underlying issue got fixed. And now the underlying issue got fixed in https://github.com/llvm/llvm-project/pull/95734. So I think it should be fine to rever that one now. Added: Modified: clang/include/clang/AST/DeclID.h Removed: ################################################################################ diff --git a/clang/include/clang/AST/DeclID.h b/clang/include/clang/AST/DeclID.h index e8f4860e13f1f..65d8108470e7b 100644 --- a/clang/include/clang/AST/DeclID.h +++ b/clang/include/clang/AST/DeclID.h @@ -259,11 +259,7 @@ template <> struct DenseMapInfo<clang::GlobalDeclID> { } static unsigned getHashValue(const GlobalDeclID &Key) { - // Our default hash algorithm for 64 bits integer may not be very good. - // In GlobalDeclID's case, it is pretty common that the lower 32 bits can - // be same. - // FIXME: Remove this when we fix the underlying issue. - return llvm::hash_value(Key.getRawValue()); + return DenseMapInfo<DeclID>::getHashValue(Key.getRawValue()); } static bool isEqual(const GlobalDeclID &L, const GlobalDeclID &R) { _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits