[PATCH] D45753: Lift JSON library from clang-tools-extra/clangd to llvm/Support.

Chandler Carruth via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 24 03:46:55 PDT 2018


chandlerc added a comment.

In https://reviews.llvm.org/D45753#1075648, @sammccall wrote:

> @chandlerc: any interest in reviewing this JSON library (either at a high level for suitability in `llvm/Support`, or for detailed review)?


I'm happy to tackle both of these. I'll want to do a bit of homework to make sure that the high level concerns that came up previously are adequately addressed (or there is some plan to address them).

> It has been used in clangd for a while, and seems suitable for lldb's purposes too. `llvm/Support` seems like their common ancestor, but I'm also happy if there's somewhere better.

This seems at least sufficient for us to invest some time sorting this all out one way or another.


Repository:
  rL LLVM

https://reviews.llvm.org/D45753





More information about the llvm-commits mailing list