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

Sam McCall via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 30 10:44:35 PDT 2018


sammccall added a comment.

@chandlerc This is ready for you again whenever you have cycles. To summarize:

- I think everything above was resolved except for the question of whether the `mutable`/`initializer_list` tricks are too gross to use.
- a few more potential users seem pleased about this idea, and @simon_tatham has verified that it works for tablegen in https://reviews.llvm.org/D46054.
- I've had a couple of suggestions which have been moved into https://reviews.llvm.org/D46209 (numeric precision) and https://reviews.llvm.org/D46274 (unicode validation), I don't think you personally need to review those.


Repository:
  rL LLVM

https://reviews.llvm.org/D45753





More information about the llvm-commits mailing list