[clang] [serialization] no transitive decl change (PR #92083)
Chuanqi Xu via cfe-commits
cfe-commits at lists.llvm.org
Mon Jun 17 18:45:12 PDT 2024
ChuanqiXu9 wrote:
> Oh, I didn't realize you were eager to land this in Clang 19, thanks for sharing that. In that case, I think a specialiazed hash function for `GlobalDeclID` is indeed the way to go.
>
> I was also worried a little there are other performance implications of this change that would block us, but we won't know until we run a full release testing cycle, which may take a week or more. Hopefully it will be okay, as my observations around the increase in PCM sizes align with the numbers from this patch and performance seems to be on par as well, if we change the hash function. However, I still wanted to mention it so that we won't come out completely of the blue.
>
> I'll try to help reviewing the two patches you posted, thanks!
Thanks. I've added you as reviewers. I hope to land these 2 patches soon so that we can have more baking time for them.
https://github.com/llvm/llvm-project/pull/92083
More information about the cfe-commits
mailing list