[Lldb-commits] [lldb] [lldb][AIX] Taking Linux Host Info header's base for AIX (PR #106910)

[David Spickett via lldb-commits]

================
@@ -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