================
@@ -38,6 +38,12 @@ endif()
 include(LLDBConfig)
 include(AddLLDB)
 
+# This has been added to keep the AIX build isolated for now.
+# It will need to be modified later.
+if (UNIX AND ${CMAKE_SYSTEM_NAME} MATCHES "AIX")
+  add_definitions("-D__AIX__")
----------------
DavidSpickett wrote:

There is `_AIX` at least in clang: https://godbolt.org/z/95MzjcsKn

And according to this (totally official, legit) doc 
(https://www.lnf.infn.it/computing/doc/aixcxx/html/language/ref/rnmcradd.htm) 
they exist for the AIX system compiler as well. I'm sure you can find the 
proper page :)

https://github.com/llvm/llvm-project/blob/0fa78b6c7bd43c2498700a98c47a02cf4fd06388/clang/lib/Basic/Targets/OSTargets.h#L641

If for some reason that's not going to work, then we would use something like 
`LLDB_...` as we have done for `LLDB_ENABLE_LIBXML2` and friends.

(but I'm yet to look at the whole PR so I don't know yet)

https://github.com/llvm/llvm-project/pull/106910
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to