[PATCH] D79862: [clangd-remote] Replace YAML serialization with proper Protobuf messages

[Kirill Bobyrev via Phabricator via cfe-commits]

kbobyrev added inline comments.


================
Comment at: clang-tools-extra/clangd/unittests/remote/CMakeLists.txt:2
+set(LLVM_LINK_COMPONENTS
+  Support
+  )
----------------
kbobyrev wrote:
> sammccall wrote:
> > Can we conditionally include these tests into the main ClangdTests, instead of setting up another set of targets?
> I can do that, although I saw that XPC has their own set of tests if I am correct, so I thought this should be OK.
> 
> Conditionally including tests into the main ClangdTests is not super easy because... CMake stuff. But I. can look into that.
Actually, nevermind, it looks pretty straightforward.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79862