[PATCH] D52453: [clangd] Merge binary + YAML serialization behind a (mostly) common interface.

[Sam McCall via Phabricator via cfe-commits]

sammccall created this revision.
sammccall added a reviewer: kbobyrev.
Herald added subscribers: cfe-commits, kadircet, arphaman, jkorous, MaskRay, ioeric, ilya-biryukov, mgorny.

Interface is in one file, implementation in two as they have little in common.
A couple of ad-hoc YAML functions left exposed:

- symbol -> YAML I expect to keep for tools like dexp
- YAML -> symbol is used for the MR-style indexer, I think we can eliminate this (merge-on-the-fly, else use a different serialization)


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D52453

Files:
  clangd/CMakeLists.txt
  clangd/index/Serialization.cpp
  clangd/index/Serialization.h
  clangd/index/SymbolYAML.cpp
  clangd/index/SymbolYAML.h
  clangd/index/YAMLSerialization.cpp
  clangd/index/dex/dexp/Dexp.cpp
  clangd/indexer/IndexerMain.cpp
  clangd/tool/ClangdMain.cpp
  unittests/clangd/SerializationTests.cpp
  unittests/clangd/SymbolCollectorTests.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D52453.166822.patch
Type: text/x-patch
Size: 18175 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180925/6d89f307/attachment-0001.bin>