================
@@ -11,6 +11,11 @@ if(APPLE)
set(CMAKE_EXE_LINKER_FLAGS "${CMAKE_EXE_LINKER_FLAGS}
-Wl,-sectcreate,__TEXT,__info_plist,${CMAKE_CURRENT_BINARY_DIR}/lldb-Info.plist")
endif()
+if (UNIX AND ${CMAKE_SYSTEM_NAME} MATCHES "AIX")
+ remove_definitions("-D_XOPEN_SOURCE=700")
----------------
labath wrote:
Thanks. The way I'd start is the see if the `remove_definitions` call is
actually doing something (in my experiments, it didn't, but I only tried it on
toy programs). If it isn't, then you already have both of these defined, and
there's nothing further to check.
That said, if you can get it touch with them, it might be good to check they
were trying to achieve, as having different definition in different parts of
the program is a recipe for problems. From that comment, it sounds like they
had problems building llvm with _ALL_SOURCE, but adding the definition here
still requires llvm (at least, parts included by this code) to build in this
mode.
https://github.com/llvm/llvm-project/pull/120607
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
