[PATCH] D34600: [Error] add expectSuccess()

[Lang Hames via Phabricator via llvm-commits]

lhames added a comment.

> I think it will be a bit more confusing in other contexts:
> 
>   return assertSuccess(foo(...)); // void or T? Both compile...

Good point. I withdraw my objection to the suffix. :)


https://reviews.llvm.org/D34600