diff options
| author | Simon Pilgrim <llvm-dev@redking.me.uk> | 2018-04-07 11:37:21 +0000 |
|---|---|---|
| committer | Simon Pilgrim <llvm-dev@redking.me.uk> | 2018-04-07 11:37:21 +0000 |
| commit | bc89a781b76c90ec510aff73d949f62898ee18dd (patch) | |
| tree | 8d1ddff883c5d80767465d7537f88065f243845f /llvm/tools/llvm-exegesis/lib/PerfHelper.cpp | |
| parent | be119ca7123c36600e0f6e569dcc7bf464040996 (diff) | |
| download | bcm5719-llvm-bc89a781b76c90ec510aff73d949f62898ee18dd.tar.gz bcm5719-llvm-bc89a781b76c90ec510aff73d949f62898ee18dd.zip | |
[llvm-exegesis] Fix unused return value warning and add a useful error message for event counter reads.
llvm-svn: 329496
Diffstat (limited to 'llvm/tools/llvm-exegesis/lib/PerfHelper.cpp')
| -rw-r--r-- | llvm/tools/llvm-exegesis/lib/PerfHelper.cpp | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/llvm/tools/llvm-exegesis/lib/PerfHelper.cpp b/llvm/tools/llvm-exegesis/lib/PerfHelper.cpp index b7cbbf70e1a..8b77047bd0a 100644 --- a/llvm/tools/llvm-exegesis/lib/PerfHelper.cpp +++ b/llvm/tools/llvm-exegesis/lib/PerfHelper.cpp @@ -107,7 +107,9 @@ void Counter::stop() { ioctl(FileDescriptor, PERF_EVENT_IOC_DISABLE, 0); } int64_t Counter::read() const { int64_t Count = 0; - ::read(FileDescriptor, &Count, sizeof(Count)); + ssize_t ReadSize = ::read(FileDescriptor, &Count, sizeof(Count)); + if (ReadSize != sizeof(Count)) + llvm::errs() << "Failed to read event counter"; return Count; } |
