gribozavr marked 4 inline comments as done. gribozavr added inline comments.
================ Comment at: clang-tools-extra/docs/clangd/Installation.rst:360 + +- Pass an experimental `-background-index` command line argument. With + this feature enabled, clangd incrementally builds an index of projects ---------------- ilya-biryukov wrote: > I'd avoid mentioning it at all, we didn't vet it for the 8 release (and I > believe we landed critical fixes **after** the release cut). > We could add it back during `9.0` release. Removed in this patch (that I plan to cherry-pick into the 8.0 release branch). Will re-add in a separate patch, so that it goes into 9.0. ================ Comment at: clang-tools-extra/docs/clangd/index.rst:20 -- Passing experimental `-background-index` commandline argument, which will - incrementally build an index of projects that you work on and make use of that - in clangd automatically. -- Generate an index file using `clangd-indexer - <https://github.com/llvm-mirror/clang-tools-extra/blob/master/clangd/indexer/IndexerMain.cpp>`_ - Afterwards you can pass generated index file to clangd using - `-index-file=/path/to/index_file`. *Note that clangd-indexer isn't included - alongside clangd in the standard clang-tools package. You will likely have to - build from source to use this option* +clangd is an implementation of the `Language Server Protocol +<https://github.com/Microsoft/language-server-protocol>`__ and can work with ---------------- ilya-biryukov wrote: > The original "clangd is a **language server**" looked somewhat better. > Having a link to the LSP website is a good bet, though. Reworded to add the phrase "language server". Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D58710/new/ https://reviews.llvm.org/D58710 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
