This is an archive of the discontinued LLVM Phabricator instance.

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

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

Reviewers

Commits

This revision is now accepted and ready to land.Sep 25 2018, 7:41 AM

This revision was automatically updated to reflect the committed changes.

Path

Size

llvm/

trunk/

test/

tools/

llvm-exegesis/

X86/

4 lines

tools/

llvm-exegesis/

lib/

2 lines

	# 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: ...

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