jfb added a comment.

What's the return code of the driver when the pipe is broken that way?



================
Comment at: lib/Driver/Driver.cpp:1406
+    // for SIGPIPE. Do not print diagnostics for this case.
+    if (Res == 71)
+      continue;
----------------
Ditto on magical number in a header.


================
Comment at: lib/Driver/Driver.cpp:1406
+    // for SIGPIPE. Do not print diagnostics for this case.
+    if (Res == 71)
+      continue;
----------------
jfb wrote:
> Ditto on magical number in a header.
I think you want to create a new diagnostic here for "broken pipe" before the 
`continue`.


Repository:
  rC Clang

https://reviews.llvm.org/D53001



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to