[PATCH] D53288: [clangd] Optionally use dex for the preamble parts of the dynamic index.

[Haojian Wu via Phabricator via cfe-commits]

hokein accepted this revision.
hokein added a comment.
This revision is now accepted and ready to land.

LGTM.



================
Comment at: clangd/ClangdServer.h:79
+    /// Use a heavier and faster in-memory index implementation.
+    /// FIXME: we should make this true if it isn't too slow!.
+    bool HeavyweightDynamicSymbolIndex = false;
----------------
"too slow" seems confusing, dex is faster, I think here it means too slow to build?


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D53288