[PATCH] D36728: Switch to consumeError(), since this can crash otherwise.

[Lang Hames via Phabricator via cfe-commits]

lhames added a comment.

The preferred solution to this is actually to wrap the call with cantFail (See http://llvm.org/docs/ProgrammersManual.html#using-cantfail-to-simplify-safe-callsites) -- it will handle both the assertion and consumption of the value for you, and will simplify calls that return an Expected<T>.


Repository:
  rL LLVM

https://reviews.llvm.org/D36728