[PATCH] D87335: [json] Create some llvm::Expected-based accessors

[Sam McCall via Phabricator via llvm-commits]

sammccall added a comment.

In D87335#2284855 <https://reviews.llvm.org/D87335#2284855>, @sammccall wrote:

> A rough design sketch

I've fleshed this out in D88103 <https://reviews.llvm.org/D88103>.
As noted there it could be split into several patches, but I'm pretty happy with the ability there to show an error in context without producing an overwhelming amount of information.
(And without paying for rendering everything upfront when it may be unused or even a nonfatal error at the top level).

Interested whether this is something you could use?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D87335/new/

https://reviews.llvm.org/D87335