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