[PATCH] D55014: Add a new interceptor for getvfsstat(2) from NetBSD

Kamil Rytarowski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 29 19:10:14 PST 2018


krytarowski marked an inline comment as done.
krytarowski added inline comments.


================
Comment at: test/sanitizer_common/TestCases/NetBSD/getvfsstat.cc:18
+  if ((rv = getvfsstat(NULL, 0, ST_WAIT)) == -1)
+    err(EXIT_FAILURE, "getvfsstat");
+
----------------
vitalybuka wrote:
> you can simplify the test with simple asserts here
Do you mean: `assert(rv != -1)` ? err(3) is already simple enough so I will keep it.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D55014





More information about the llvm-commits mailing list