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

[Guillaume Chatelet via Phabricator via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL342981: [llvm-exegesis] Fix missing document separator in YAML output. (authored by gchatelet, committed by ).

Repository:
  rL LLVM

https://reviews.llvm.org/D52496

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


Index: llvm/trunk/tools/llvm-exegesis/lib/BenchmarkResult.cpp
===================================================================
--- llvm/trunk/tools/llvm-exegesis/lib/BenchmarkResult.cpp
+++ llvm/trunk/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: llvm/trunk/test/tools/llvm-exegesis/X86/latency-by-opcode-name.s
===================================================================
--- llvm/trunk/test/tools/llvm-exegesis/X86/latency-by-opcode-name.s
+++ llvm/trunk/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.166917.patch
Type: text/x-patch
Size: 1129 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180925/87d7a0ea/attachment.bin>