[PATCH] D49010: YAML output for index-while-building

[Ilya Biryukov via Phabricator via cfe-commits]

ilya-biryukov added a comment.

JSON is a subset of YAML, do we really need both outputs?  I've noticed the schema for the output is actually different between YAML and JSON, wouldn't that be confusing?

> Should be compatible with the current index format accepted by clangd

Do we really want that?
Note that we were not planning to keep the YAML index in clangd long-term, it was merely a convenient serialization routine available in LLVM land, so our implementation uses it for now. I would discourage to build tools that rely on the YAML format staying the same.

Maybe we could teach clangd to consume the index-while-build output (in binary format) when building the index instead? WDYT?


https://reviews.llvm.org/D49010