[PATCH] D49013: [Support] Allow formatv() to consume llvm::Error by-value without crashing.
Zachary Turner via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 10 08:51:27 PDT 2018
zturner added a comment.
In the samples you gave in the original code review, it was always just
`formatv(“Error: {0}”, E);`
Is it always this simple and consistent? Or do you frequently construct
complex messages with many other parameters in the same format string? If
it’s always (or even often) that simple, maybe you just need a function
like void `logError(Error E)`
Repository:
rL LLVM
https://reviews.llvm.org/D49013
More information about the llvm-commits
mailing list