[PATCH] D52496: [llvm-exegesis] Fix missing document separator in YAML output.

[Guillaume Chatelet via Phabricator via llvm-commits]

gchatelet created this revision.
gchatelet added a reviewer: courbet.
Herald added subscribers: llvm-commits, tschuett.

Repository:
  rL LLVM

https://reviews.llvm.org/D52496

Files:
  test/tools/llvm-exegesis/X86/latency-by-opcode-name.s
  tools/llvm-exegesis/lib/BenchmarkResult.cpp


Index: tools/llvm-exegesis/lib/BenchmarkResult.cpp
===================================================================
--- tools/llvm-exegesis/lib/BenchmarkResult.cpp
+++ tools/llvm-exegesis/lib/BenchmarkResult.cpp
@@ -269,7 +269,9 @@
                                        llvm::raw_ostream &OS) {
   llvm::yaml::Output Yout(OS);
   YamlContext Context(State);
+  Yout.beginDocuments();
   llvm::yaml::yamlize(Yout, *this, /*unused*/ true, Context);
+  Yout.endDocuments();
 }
 
 void InstructionBenchmark::readYamlFrom(const LLVMState &State,
Index: test/tools/llvm-exegesis/X86/latency-by-opcode-name.s
===================================================================
--- test/tools/llvm-exegesis/X86/latency-by-opcode-name.s
+++ test/tools/llvm-exegesis/X86/latency-by-opcode-name.s
@@ -1,6 +1,8 @@
 # RUN: llvm-exegesis -mode=latency -opcode-name=ADD32rr | FileCheck %s
 
-CHECK:      mode:            latency
+CHECK:      ---
+CHECK-NEXT: mode:            latency
 CHECK-NEXT: key:
 CHECK-NEXT:   instructions:
 CHECK-NEXT:     ADD32rr
+CHECK-LAST: ...


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D52496.166913.patch
Type: text/x-patch
Size: 1063 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180925/be933bc4/attachment.bin>