[PATCH] D46821: Update llvm-exegesis to cover latency through another opcode.
Guillaume Chatelet via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon May 14 06:37:11 PDT 2018
gchatelet planned changes to this revision.
gchatelet added inline comments.
================
Comment at: tools/llvm-exegesis/lib/BenchmarkResult.cpp:79
+
void InstructionBenchmark::writeYamlOrDie(const llvm::StringRef Filename) {
if (Filename == "-") {
----------------
courbet wrote:
> what about letting the caller handle the error ?
Let's do this in a separate Patch if you don't mind.
Repository:
rL LLVM
https://reviews.llvm.org/D46821
More information about the llvm-commits
mailing list