[PATCH] D83817: [clangd] Add option to use remote index as static index

Hans Wennborg via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 27 05:05:58 PDT 2020


hans added inline comments.


================
Comment at: clang-tools-extra/clangd/tool/ClangdMain.cpp:702
   }
+  if (RemoteIndexAddress.empty() != ProjectPath.empty()) {
+    llvm::errs() << "remote-index-address and project-path have to be "
----------------
hokein wrote:
> the new code section here should be guarded under `#ifdef CLANGD_ENABLE_REMOTE`
I think it should be an #if since the macro is always defined (0 or 1). Doing that in 40d11a878044711708fb6738e4b78a4c9ac3de7b


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83817





More information about the cfe-commits mailing list