[PATCH] D87335: [json] Create some llvm::Expected-based accessors
Sam McCall via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Sep 22 09:21:07 PDT 2020
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
More information about the llvm-commits
mailing list