[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