diff --git a/clang-tools-extra/clangd/CMakeLists.txt b/clang-tools-extra/clangd/CMakeLists.txt --- a/clang-tools-extra/clangd/CMakeLists.txt +++ b/clang-tools-extra/clangd/CMakeLists.txt @@ -114,6 +114,15 @@ omp_gen ) +# FIXME(kirillbobyrev): Document this in the LLVM docs once remote index is stable. +option(CLANGD_ENABLE_REMOTE "Use gRPC library to enable remote index support for Clangd" OFF) +set(GRPC_INSTALL_PATH "" CACHE PATH "Path to gRPC library manual installation.") + +if (CLANGD_ENABLE_REMOTE) + include(FindGRPC) +endif() +add_subdirectory(index/remote) + clang_target_link_libraries(clangDaemon PRIVATE clangAST @@ -131,6 +140,7 @@ clangToolingInclusions clangToolingRefactoring clangToolingSyntax + clangdRemoteIndex ) add_subdirectory(refactor/tweaks) @@ -153,12 +163,4 @@ add_subdirectory(unittests) endif() -# FIXME(kirillbobyrev): Document this in the LLVM docs once remote index is stable. -option(CLANGD_ENABLE_REMOTE "Use gRPC library to enable remote index support for Clangd" OFF) -set(GRPC_INSTALL_PATH "" CACHE PATH "Path to gRPC library manual installation.") - -if (CLANGD_ENABLE_REMOTE) - include(FindGRPC) -endif() -add_subdirectory(index/remote) add_subdirectory(index/dex/dexp) diff --git a/clang-tools-extra/clangd/Features.inc.in b/clang-tools-extra/clangd/Features.inc.in --- a/clang-tools-extra/clangd/Features.inc.in +++ b/clang-tools-extra/clangd/Features.inc.in @@ -1 +1,2 @@ #define CLANGD_BUILD_XPC @CLANGD_BUILD_XPC@ +#define CLANGD_ENABLE_REMOTE @CLANGD_ENABLE_REMTE@ diff --git a/clang-tools-extra/clangd/tool/ClangdMain.cpp b/clang-tools-extra/clangd/tool/ClangdMain.cpp --- a/clang-tools-extra/clangd/tool/ClangdMain.cpp +++ b/clang-tools-extra/clangd/tool/ClangdMain.cpp @@ -14,6 +14,7 @@ #include "Transport.h" #include "index/Background.h" #include "index/Serialization.h" +#include "index/remote/Client.h" #include "refactor/Rename.h" #include "support/Path.h" #include "support/Shutdown.h" @@ -449,6 +450,21 @@ init(true), }; +#ifdef CLANGD_ENABLE_REMOTE +opt RemoteIndexAddress{ + "remote-index-address", + cat(Features), + desc("Address of the remote index server"), +}; + +// FIXME(kirillbobyrev): Should this be the location of compile_commands.json? +opt ProjectRoot{ + "project-path", + cat(Features), + desc("Path to the project root. Requires remote-index-address to be set."), +}; +#endif + /// Supports a test URI scheme with relaxed constraints for lit tests. /// The path in a test URI will be combined with a platform-specific fake /// directory to form an absolute path. For example, test:///a.cpp is resolved @@ -680,6 +696,23 @@ if (Sync) AsyncIndexLoad.wait(); } +#ifdef CLANGD_ENABLE_REMOTE + if (RemoteIndexAddress.empty() != ProjectRoot.empty()) { + llvm::errs() << "remote-index-address and project-path have to be " + "specified at the same time."; + return 1; + } + if (!RemoteIndexAddress.empty()) { + if (IndexFile.empty()) { + log("Connecting to remote index at {0}", RemoteIndexAddress); + StaticIdx = remote::getClient(RemoteIndexAddress, ProjectRoot); + EnableBackgroundIndex = false; + } else { + elog("When enabling remote index, IndexFile should not be specified. " + "Only one can be used at time. Remote index will ignored."); + } + } +#endif Opts.StaticIndex = StaticIdx.get(); Opts.AsyncThreadsCount = WorkerThreadsCount; Opts.BuildRecoveryAST = RecoveryAST;