danlark abandoned this revision.
danlark added a comment.
Thanks!
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D97000/new/
https://reviews.llvm.org/D97000
___
cfe-commits mailing list
cfe-commits@lists.l
rsmith added a comment.
Thanks for the diagnosis and the patch!
We spend a significant amount of our AST file size on these lookup tables, so I
think the extra two bytes per lookup entry is best avoided if possible. We also
have the same problem for other on-disk hash tables. I went ahead and
danlark created this revision.
danlark added a reviewer: rsmith.
danlark requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
We faced the assert of too many declarations for serialized lookup for a very
generated C++ target. We
think moving th