[libcxx][CI][AIX] Switch to LLVM_ENABLE_RUNTIMES

[daltenty]

Summary

and to the new runtimes top level CMakeLists.txt since the old path is now deprecated. This requires a slight adjustment of the libcxxabi CMake, since there are required macro definitions we previously got via the llvm/CMakeList.txt path.

[daltenty]

• Remove duplicate option

[daltenty]

• [libcxxabi] Avoid the VMX malloc to vec_malloc mapping
• Un-XFAIL test fixed by the more narrowly application of XOPEN_SOURCE macros

[ldionne]

libcxx/utils/ci/run-buildbot

116–120

Can we get rid of generate-cmake-aix entirely since it is only used in one place?

libcxxabi/CMakeLists.txt

522–526

Is this a different workaround for https://reviews.llvm.org/D112168?

524

[daltenty]

libcxx/utils/ci/run-buildbot

116–120

I think that makes sense, since these defines should be just last option wins, I'll move this in to the AIX block itself.

libcxxabi/CMakeLists.txt

522–526

Precisely, the XOPEN_SOURCE macro suppresses the malloc to vec_malloc mapping in the system header, so we don't see the problematic redefinition and corresponding alignment change (at least in libcxxabi where it causes the most disruption to existing applications).

[daltenty]

• Move DLLVM_ENABLE_RUNTIMES into aix block

libcxx/test/std/thread/thread.threads/thread.thread.class/thread.thread.static/hardware_concurrency.pass.cpp

 //===----------------------------------------------------------------------===//
 //
 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
 // See https://llvm.org/LICENSE.txt for license information.
 // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
 //
 //===----------------------------------------------------------------------===//
 //
 // UNSUPPORTED: libcpp-has-no-threads
-// XFAIL: LIBCXX-AIX-FIXME
 
 // <thread>
 
 // class thread
 
 // unsigned hardware_concurrency();
 
 #include <thread>

libcxx/utils/ci/run-buildbot

generate-cmake-base \
           -DCMAKE_C_COMPILER=clang-cl \
           -DCMAKE_CXX_COMPILER=clang-cl \
           -DLIBCXX_ENABLE_FILESYSTEM=YES \
           -DCMAKE_CXX_FLAGS="-D_LIBCPP_HAS_NO_INT128" \
           -DLIBCXX_TEST_COMPILER_FLAGS="-D_LIBCPP_HAS_NO_INT128" \
           "${@}"
 }
 
 function generate-cmake-aix() {
     generate-cmake-base \
-          -S "${MONOREPO_ROOT}/llvm" \
-          -DLLVM_ENABLE_PROJECTS="libcxx;libcxxabi" \
+          -DLLVM_ENABLE_RUNTIMES="libcxx;libcxxabi" \
           "${@}"
 }
 
 function check-runtimes() {
     echo "--- Installing libc++, libc++abi and libunwind to a fake location"
     ${NINJA} -vC "${BUILD_DIR}" install-cxx install-cxxabi install-unwind
 
     echo "+++ Running the libc++ tests"
     ${NINJA} -vC "${BUILD_DIR}" check-cxx
 

libcxxabi/CMakeLists.txt

if (LIBCXXABI_HAS_PTHREAD_LIB)
     add_definitions(-D_LIBCXXABI_LINK_PTHREAD_LIB)
   endif()
 endif()
 
 string(REPLACE ";" " " LIBCXXABI_CXX_FLAGS "${LIBCXXABI_CXX_FLAGS}")
 set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${LIBCXXABI_CXX_FLAGS}")
 set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} ${LIBCXXABI_C_FLAGS}")
 
+# On AIX, avoid picking up VMX extensions(i.e. vec_malloc) which would change
+# the default alignment of the allocators here.
+if (UNIX AND ${CMAKE_SYSTEM_NAME} MATCHES "AIX")
+  add_definitions("-D_XOPEN_SOURCE=700")
+endif()
+
 #===============================================================================
 # Setup Source Code
 #===============================================================================
 
 set(LIBCXXABI_LIBUNWIND_INCLUDES "${LIBCXXABI_LIBUNWIND_INCLUDES}" CACHE PATH
     "Specify path to libunwind includes." FORCE)
 set(LIBCXXABI_LIBUNWIND_PATH "${LIBCXXABI_LIBUNWIND_PATH}" CACHE PATH
     "Specify path to libunwind source." FORCE)