[PATCH] D98404: [clangd] Optionally add reflection for clangd-index-server

[Kirill Bobyrev via Phabricator via cfe-commits]

kbobyrev added inline comments.


================
Comment at: clang-tools-extra/clangd/index/remote/server/CMakeLists.txt:12
 
+if (ENABLE_GRPC_REFLECTION)
+  set(REFLECTION_LIBRARY grpc++_reflection)
----------------
kadircet wrote:
> can we move this piece into FindGRPC.cmake instead?
Yes, but I think only the `set(REFLECTION_LIBRARY grpc++_reflection)` part? Having the `-DENABLE_GRPC_REFLECTION` definition globally is probably not very nice and as with the other target under `clangd/index/remote`, `add_target_definition` doesn't work because of `add_clan_executable` :(


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D98404