[PATCH] D51422: [clangd] Factor out the data-swapping functionality from MemIndex/DexIndex.

[Kirill Bobyrev via Phabricator via cfe-commits]

kbobyrev added inline comments.


================
Comment at: clangd/index/Index.h:416
+  // until the call returns (even if reset() is called).
+  bool fuzzyFind(const FuzzyFindRequest &,
+                 llvm::function_ref<void(const Symbol &)>) const override;
----------------
Do we want these functions to be `final`? Since `SwapIndex` is a wrapper around an immutable index structure, I believe it would be unlikely that anything would derive from it.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D51422