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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
