[Lldb-commits] [lldb] [lldb][AIX] Some base #if _AIX changes of a minimal lldb build (PR #120979)

David Spickett via lldb-commits lldb-commits at lists.llvm.org
Tue Jan 7 02:02:36 PST 2025


================
@@ -715,7 +715,7 @@ ConnectionFileDescriptor::ConnectFD(llvm::StringRef s,
 ConnectionStatus ConnectionFileDescriptor::ConnectFile(
     llvm::StringRef s, socket_id_callback_type socket_id_callback,
     Status *error_ptr) {
-#if LLDB_ENABLE_POSIX
+#if LLDB_ENABLE_POSIX && !defined(_AIX)
----------------
DavidSpickett wrote:

https://www.man7.org/linux/man-pages/man3/termios.3.html

This would be a build error, so I think `#error Max baud rate unknown.` would be fine here. This forces the developer to check what their platform supports.

You could write it so that it tries the 3 in order B115200, B38400, B57600, then falls into the else and errors.

https://github.com/llvm/llvm-project/pull/120979


More information about the lldb-commits mailing list