This is an archive of the discontinued LLVM Phabricator instance.

[llvm-exegesis] Improve error reporting.
ClosedPublic

Authored by gchatelet on Jun 7 2018, 12:26 AM.

Details

Summary

BenchmarkResult IO functions now return an Error or Expected so caller can deal take proper action.

Diff Detail

Event Timeline

gchatelet created this revision.Jun 7 2018, 12:26 AM
courbet added inline comments.Jun 7 2018, 12:32 AM
tools/llvm-exegesis/lib/BenchmarkResult.h
60–61

remove OrDie (same below)

gchatelet updated this revision to Diff 150266.Jun 7 2018, 12:40 AM
gchatelet marked an inline comment as done.

Updating D47868: [llvm-exegesis] Improve error reporting.

courbet accepted this revision.Jun 7 2018, 12:46 AM
This revision is now accepted and ready to land.Jun 7 2018, 12:46 AM
This revision was automatically updated to reflect the committed changes.