This is an archive of the discontinued LLVM Phabricator instance.

Differential D52496

[llvm-exegesis] Fix missing document separator in YAML output.
ClosedPublic

Authored by gchatelet on Sep 25 2018, 7:39 AM.

Details

Reviewers
courbet
Commits
rG6078f82241cf: [llvm-exegesis] Fix missing document separator in YAML output.
rL342981: [llvm-exegesis] Fix missing document separator in YAML output.

Diff Detail

Event Timeline

gchatelet created this revision.Sep 25 2018, 7:39 AM
Herald added subscribers: llvm-commits, tschuett. · View Herald TranscriptSep 25 2018, 7:39 AM
Harbormaster completed remote builds in B23087: Diff 166913.Sep 25 2018, 7:41 AM
courbet accepted this revision.Sep 25 2018, 7:41 AM
This revision is now accepted and ready to land.Sep 25 2018, 7:41 AM
Closed by commit rL342981: [llvm-exegesis] Fix missing document separator in YAML output. (authored by gchatelet). · Explain WhySep 25 2018, 7:49 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents

PathSize
test/
tools/
llvm-exegesis/
X86/
4 lines
tools/
llvm-exegesis/
lib/
2 lines

Diff 166913

test/tools/llvm-exegesis/X86/latency-by-opcode-name.s

# RUN: llvm-exegesis -mode=latency -opcode-name=ADD32rr | FileCheck %s # RUN: llvm-exegesis -mode=latency -opcode-name=ADD32rr | FileCheck %s
CHECK: mode: latency CHECK: ---
CHECK-NEXT: mode: latency
CHECK-NEXT: key: CHECK-NEXT: key:
CHECK-NEXT: instructions: CHECK-NEXT: instructions:
CHECK-NEXT: ADD32rr CHECK-NEXT: ADD32rr
CHECK-LAST: ...

tools/llvm-exegesis/lib/BenchmarkResult.cpp

Show First 20 Lines • Show All 263 Lines • ▼ Show 20 Lines if (auto ExpectedMemoryBuffer =
return ExpectedMemoryBuffer.takeError(); return ExpectedMemoryBuffer.takeError();
} }
} }
void InstructionBenchmark::writeYamlTo(const LLVMState &State, void InstructionBenchmark::writeYamlTo(const LLVMState &State,
llvm::raw_ostream &OS) { llvm::raw_ostream &OS) {
llvm::yaml::Output Yout(OS); llvm::yaml::Output Yout(OS);
YamlContext Context(State); YamlContext Context(State);
Yout.beginDocuments();
llvm::yaml::yamlize(Yout, *this, /*unused*/ true, Context); llvm::yaml::yamlize(Yout, *this, /*unused*/ true, Context);
Yout.endDocuments();
} }
void InstructionBenchmark::readYamlFrom(const LLVMState &State, void InstructionBenchmark::readYamlFrom(const LLVMState &State,
llvm::StringRef InputContent) { llvm::StringRef InputContent) {
llvm::yaml::Input Yin(InputContent); llvm::yaml::Input Yin(InputContent);
YamlContext Context(State); YamlContext Context(State);
if (Yin.setCurrentDocument()) if (Yin.setCurrentDocument())
llvm::yaml::yamlize(Yin, *this, /*unused*/ true, Context); llvm::yaml::yamlize(Yin, *this, /*unused*/ true, Context);
Show All 30 Lines