[PATCH] D21982: WIP: Implement a per-thread inmemory log

David Majnemer via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 5 22:45:24 PDT 2016


majnemer added inline comments.

================
Comment at: lib/xray/xray_inmemory_log.cc:71
@@ +70,3 @@
+    auto Written = write(Fd, Begin, Items * sizeof(XRayRecord));
+    if (Written <= 0) {
+      if (errno == EINTR)
----------------
mehdi_amini wrote:
> majnemer wrote:
> > I don't believe `write` can actually return zero for this case.
> When does write can actually return zero according to the spec?
>From my recollection, it should only be possible if you pass write a count of zero bytes.


http://reviews.llvm.org/D21982





More information about the llvm-commits mailing list