[all-commits] [llvm/llvm-project] 58d0ef: [clangd] Fix remote index build failures due to la...

Kirill Bobyrev via All-commits all-commits at lists.llvm.org
Mon Oct 26 06:15:11 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 58d0ef2d0466a893ab400f6a9829057b9d851038
      https://github.com/llvm/llvm-project/commit/58d0ef2d0466a893ab400f6a9829057b9d851038
  Author: Kirill Bobyrev <kbobyrev at google.com>
  Date:   2020-10-26 (Mon, 26 Oct 2020)

  Changed paths:
    M clang-tools-extra/clangd/index/remote/CMakeLists.txt

  Log Message:
  -----------
  [clangd] Fix remote index build failures due to lack of proto dependency

Previous attempt (15f6bad6d74a993e366c8fc93a9c91f213ac6bc3) introduced
add_dependencies but unfortunately it does not actually add a dependency
between RemoteIndexProto and RemoteIndexServiceProto. This is likely due
to some requirements of it that clang_add_library violates.

As a workaround, we will link RemoteIndexProto library to
RemoteIndexServiceProto which is logical because the library can not be
without linking to RemoteIndexProto anyway.




More information about the All-commits mailing list