sammccall added a comment.
Is this patch still relevant after haojian's string deduplication?
================
Comment at: clangd/global-symbol-builder/GlobalSymbolBuilderMain.cpp:53
+/// Combines occurrences of the same symbols across translation units.
+class SymbolMerger {
----------------
Seems reasonably likely we would actually have contention here? merging
per-thread (combiner) and then globally at the end (reducer) might be the way
to go (might be significantly faster). But not sure how big the impact is.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D45478
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
