[Lldb-commits] [lldb] [lldb] Simplify pipe2(2) handling on FreeBSD (PR #74019)

Brad Smith via lldb-commits lldb-commits at lists.llvm.org
Thu Nov 30 17:28:34 PST 2023


https://github.com/brad0 created https://github.com/llvm/llvm-project/pull/74019

FreeBSD 10.x and 11.x support has been dropped leaving 12.x as the minimum version. This FreeBSD check can be simplified.

https://github.com/llvm/llvm-project/commit/812dad536ed50abe94d6e2b2519f351791c24c59

>From 7209d0ac47c64a3fa0d55cfd645e332fa5054884 Mon Sep 17 00:00:00 2001
From: Brad Smith <brad at comstyle.com>
Date: Thu, 30 Nov 2023 20:22:50 -0500
Subject: [PATCH] [lldb] Simplify pipe2(2) handling on FreeBSD

FreeBSD 10.x and 11.x support has been dropped leaving 12.x as the
minimum version. This FreeBSD check can be simplified.

https://github.com/llvm/llvm-project/commit/812dad536ed50abe94d6e2b2519f351791c24c59
---
 lldb/source/Host/posix/PipePosix.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/lldb/source/Host/posix/PipePosix.cpp b/lldb/source/Host/posix/PipePosix.cpp
index 6fc4646953b4257d..afd3fe39059ac176 100644
--- a/lldb/source/Host/posix/PipePosix.cpp
+++ b/lldb/source/Host/posix/PipePosix.cpp
@@ -31,8 +31,8 @@ enum PIPES { READ, WRITE }; // Constants 0 and 1 for READ and WRITE
 
 // pipe2 is supported by a limited set of platforms
 // TODO: Add more platforms that support pipe2.
-#if defined(__linux__) || (defined(__FreeBSD__) && __FreeBSD__ >= 10) ||       \
-    defined(__NetBSD__) || defined(__OpenBSD__)
+#if defined(__linux__) || defined(__FreeBSD__) || defined(__NetBSD__) ||       \
+    defined(__OpenBSD__)
 #define PIPE2_SUPPORTED 1
 #else
 #define PIPE2_SUPPORTED 0



More information about the lldb-commits mailing list