[PATCH] D40915: Always evaluate the second argument for CHECK() lazily.

Rafael Avila de Espindola via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 6 12:55:04 PST 2017


Rui Ueyama via Phabricator <reviews at reviews.llvm.org> writes:


> -template <class T> T check(ErrorOr<T> E, const Twine &Prefix) {
> +template <class T> T check2(ErrorOr<T> E, std::function<std::string()> Prefix) {

Why do you need a std::function, a function_ref should work, no?

Cheers,
Rafael


More information about the llvm-commits mailing list