SAtacker added inline comments.
================
Comment at: clang/lib/Serialization/ASTWriterDecl.cpp:288
+ }
+ for (auto &SpecInfo : LazySpecializations) {
+ Record.push_back(SpecInfo.DeclID);
----------------
ChuanqiXu wrote:
> v.g.vassilev wrote:
> > We should not store the lazy specialization information as an array of
> > items because that makes the search slow. Instead we should use the
> > `OnDiskHashTable` approach which we use already to store the identifier
> > data.
> Do you want to implement it in this patch? Or this is a note for future
> optimizations?
I tried it here https://reviews.llvm.org/D144831 but to my surprise it gives
worse performance for my testing environment which generates random
specializations per module and a single main file.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D41416/new/
https://reviews.llvm.org/D41416
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
