emaste added a subscriber: emaste.

================
Comment at: source/Host/posix/PipePosix.cpp:39
@@ -38,3 +38,3 @@
 
-// pipe2 is supported by Linux, FreeBSD v10 and higher.
+// pipe2 is supported by Linux, FreeBSD v10 and higher, NetBSD 6.0 and newer
 // TODO: Add more platforms that support pipe2.
----------------
My preference is that we don't repeat the list of platforms in the comments 
when we have cases like this - perhaps just change to `// pipe2 is supported by 
a limited set of platforms` or so?

Presumably you're only interested in recent NetBSD versions while getting 
support working?


Repository:
  rL LLVM

http://reviews.llvm.org/D12746



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

Reply via email to