[PATCH] D79950: [clangd] Avoid wasteful data structures in RefSlab::Builder

2020-05-18 Thread Sam McCall via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. sammccall marked 2 inline comments as done. Closed by commit rGd19265b31e65: [clangd] Avoid wasteful data structures in RefSlab::Builder (authored by sammccall). Changed prior to commit: https://reviews.llvm.org/D79950?vs

[PATCH] D79950: [clangd] Avoid wasteful data structures in RefSlab::Builder

2020-05-18 Thread Sam McCall via Phabricator via cfe-commits
sammccall marked 4 inline comments as done. sammccall added inline comments. Comment at: clang-tools-extra/clangd/index/Ref.cpp:11 +#include "llvm/ADT/STLExtras.h" +#include "llvm/Support/Allocator.h" +#include "llvm/Support/StringSaver.h" kbobyrev wrote: > This

[PATCH] D79950: [clangd] Avoid wasteful data structures in RefSlab::Builder

2020-05-18 Thread Kirill Bobyrev via Phabricator via cfe-commits
kbobyrev accepted this revision. kbobyrev added a comment. This revision is now accepted and ready to land. LGTM with a couple of nits. Thank you for the patch, this is a good idea! Comment at: clang-tools-extra/clangd/index/Ref.cpp:11 +#include "llvm/ADT/STLExtras.h" +#includ

[PATCH] D79950: [clangd] Avoid wasteful data structures in RefSlab::Builder

2020-05-14 Thread Sam McCall via Phabricator via cfe-commits
sammccall created this revision. sammccall added a reviewer: kbobyrev. Herald added subscribers: cfe-commits, usaxena95, kadircet, arphaman, mgrang, jkorous, MaskRay, ilya-biryukov. Herald added a project: clang. This is worth another 10% or so on InedxBenchmark.DexBuild. Repository: rG LLVM