[PATCH] D154275: [llvm-exegesis] Support older kernel versions in subprocess executor

[Mikael Holmén via Phabricator via llvm-commits]

uabelho added inline comments.


================
Comment at: llvm/tools/llvm-exegesis/lib/BenchmarkRunner.cpp:224
+
+    size_t BytesRead = recvmsg(SocketFD, &Message, 0);
+
----------------
Should be ssize_t ?

The
```
 BytesRead < 0
```
check below is currently always false since size_t is unsigned.

I noticed this when I compiled with gcc which warned 

```
../tools/llvm-exegesis/lib/BenchmarkRunner.cpp:226:19: warning: comparison of unsigned expression < 0 is always false [-Wtype-limits]
  226 |     if (BytesRead < 0)
      |         ~~~~~~~~~~^~~
```


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D154275/new/

https://reviews.llvm.org/D154275