This is an archive of the discontinued LLVM Phabricator instance.

[clangd] Remove JSON library in favor of llvm/Support/JSON
ClosedPublic

Authored by sammccall on Jul 9 2018, 7:09 AM.

Details

Summary

The library has graduated from clangd to llvm/Support.
This is a mechanical change to move to the new API and remove the old one.

Main API changes:

  • namespace clang::clangd::json --> llvm::json
  • json::Expr --> json::Value
  • Expr::asString() etc --> Value::getAsString() etc
  • unsigned longs need a cast (due to r336541 adding lossless integer support)

Diff Detail

Event Timeline

sammccall created this revision.Jul 9 2018, 7:09 AM
This revision is now accepted and ready to land.Jul 9 2018, 7:27 AM
This revision was automatically updated to reflect the committed changes.
clangd/Trace.cpp