[clang] [Serialization] Load Specializations Lazily (PR #76774)
Chuanqi Xu via cfe-commits
cfe-commits at lists.llvm.org
Mon Jan 29 01:16:37 PST 2024
ChuanqiXu9 wrote:
> As far as I can tell from [#76774 (comment)](https://github.com/llvm/llvm-project/pull/76774#issuecomment-1914177330) above, the last push only changed the default value of `LoadExternalSpecializationsLazily`. In that case, my test results from [#76774 (comment)](https://github.com/llvm/llvm-project/pull/76774#issuecomment-1912182171) remain fully valid, especially
>
> > Switching the new `LoadExternalSpecializationsLazily` to disabled by default (somehow the argument didn't work on its own) instead crashes, most of the cases involving `MultiOnDiskHashTable`. I suspect some kind of memory error maybe?
No, this newest patch changes besides changing the default value. For example, https://github.com/llvm/llvm-project/pull/76774/commits/22c9d1145eb57d9c2cb2ef490b7c474598dd5d12#diff-125f472e690aa3d973bc42aa3c5d580226c5c47661551aca2889f960681aa64dR232-R237 now we won't write specializations into the hash table, while the original patch tries to load all the decls from the consumers' side only. This patch touches the producer's side.
Maybe I shouldn't push force to make the history clear..
https://github.com/llvm/llvm-project/pull/76774
More information about the cfe-commits
mailing list