buildStaticIndex() is used by two other tools that I'm building, now it's useful outside of tool/ClangdMain.cpp.
|193 ↗||(On Diff #163797)|
this is deep copying the symbols for no reason
just build(std::move(Slab)) below.
|44 ↗||(On Diff #163797)|
nit: can you just say "index file", and call the parameter "SymbolFile"?
|47 ↗||(On Diff #163797)|
I'd prefer loadIndex for this function, what do you think?
"load" over "build" because much of the indexing work has already been done in producing the file, and in future potentially even more (serializing posting lists).