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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to