[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