[PATCH] D105014: added some example code for llvm::Expected<T>

[David Blaikie via Phabricator via cfe-commits]

dblaikie added subscribers: lhames, dblaikie.
dblaikie added a comment.

+ at lhames for context as the author/owner of `llvm::Error` and associated things.

Perhaps it'd be handy to have some descriptions of the problems you encountered, @kuhnel, and how you went about trying to resolve them, to help understand where things are lacking.

> @sammccall 
> I agree Error is confusing and better docs might help.
> (Not as much as removing the confusing part of the design, but I that seems hard at this point!)

Maybe in this review, or maybe elsewhere: might be good to have some details on your perspective here?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105014