timon-ul wrote: > And one final request before merging: could we double check the index size > numbers reported in > https://github.com/llvm/llvm-project/pull/177273#issuecomment-3786398323? The > size increase there is almost suspiciously low, and I'd just like to verify > that it wasn't due to some issue with an earlier version of the patch.
I agree I should give numbers corresponding to the final patch, that being said I think none of my changes since have reduced the scope of indexing, so unless the infinit recursion made clangd crash and not record a lot of relations (doubt) they should be representative. Maybe our code base is not the best example though so if you think it is better to run the test on the clangd code itself let me know. https://github.com/llvm/llvm-project/pull/177273 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
