[all-commits] [llvm/llvm-project] c6b2b7: [clangd-remote] Replace YAML serialization with pr...

Kirill Bobyrev via All-commits all-commits at lists.llvm.org
Tue May 19 08:08:10 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: c6b2b784299b5b8c2081b4e7e76eeab80e9b81ee
      https://github.com/llvm/llvm-project/commit/c6b2b784299b5b8c2081b4e7e76eeab80e9b81ee
  Author: Kirill Bobyrev <kbobyrev at google.com>
  Date:   2020-05-19 (Tue, 19 May 2020)

  Changed paths:
    M clang-tools-extra/clangd/index/remote/Client.cpp
    M clang-tools-extra/clangd/index/remote/Index.proto
    M clang-tools-extra/clangd/index/remote/marshalling/Marshalling.cpp
    M clang-tools-extra/clangd/unittests/CMakeLists.txt
    M clang-tools-extra/clangd/unittests/TestTU.cpp
    M clang-tools-extra/clangd/unittests/TestTU.h
    A clang-tools-extra/clangd/unittests/remote/MarshallingTests.cpp

  Log Message:
  -----------
  [clangd-remote] Replace YAML serialization with proper Protobuf messages

Summary:
YAML serialization was used in the Proof of Concept for simplicity.
This patch replaces implements Protobuf (de) serialization of almost all
types that need to be transferred over the protocol.

Reviewers: sammccall

Reviewed By: sammccall

Subscribers: mgorny, ilya-biryukov, MaskRay, jkorous, arphaman, kadircet, usaxena95, cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D79862




More information about the All-commits mailing list