kadircet added inline comments.
================ Comment at: clang-tools-extra/clangd/index/remote/server/CMakeLists.txt:12 +if (ENABLE_GRPC_REFLECTION) + set(REFLECTION_LIBRARY grpc++_reflection) ---------------- kbobyrev wrote: > 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` :( ah i forgot about that one, can we have this in `clang-tools-extra/clangd/Features.inc.in` then, with rest of our definitions? you might also want to canonicalize the option via `llvm_canonicalize_cmake_booleans` 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
