[PATCH] D56718: [clangd] Update docs to mention YCM integration and new LSP features

Tue Jan 15 09:24:36 PST 2019

ilya-biryukov added inline comments.

================
Comment at: docs/clangd.rst:125
+
+One of the options of using :program:`Clangd` in :program:`vim` is to utilize
+`YouCompleteMe
----------------
Given how little information we actually provide for each option here (it's literally just pointers to the docs), maybe make this a list instead to avoid copy-pasting between those two?
Something like:

You can use Clangd in vim with one of the following plugins:
- YouCompleteMe, see installation instructions.
- LanguageClient-neovim, see clangd wiki page for instructions.

================
Comment at: docs/clangd.rst:129
+`Installation section on your OS
+<https://valloric.github.io/YouCompleteMe/#installation>`_ for instructions.
+
----------------
I believe this does not mention how to install clangd, though (and libclang-based is still the default, right?) Do we want to mention how to configure it to use clangd here?

Repository:
  rCTE Clang Tools Extra

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D56718/new/

https://reviews.llvm.org/D56718