[llvm] 53d6039 - [CMake] Use old DynamicLibrary symbol behavior on AIX for now (#108692)

[via llvm-commits]

Author: Jake Egan
Date: 2024-09-16T10:57:06-04:00
New Revision: 53d60398efb06df683f1189a554f547873331c81

URL: https://github.com/llvm/llvm-project/commit/53d60398efb06df683f1189a554f547873331c81
DIFF: https://github.com/llvm/llvm-project/commit/53d60398efb06df683f1189a554f547873331c81.diff

LOG: [CMake] Use old DynamicLibrary symbol behavior on AIX for now (#108692)

New behavior broke the AIX bot, so fall back to the old behavior on AIX
for now to give time to investigate

Added: 
    

Modified: 
    llvm/unittests/Support/DynamicLibrary/CMakeLists.txt

Removed: 
    


################################################################################
diff  --git a/llvm/unittests/Support/DynamicLibrary/CMakeLists.txt b/llvm/unittests/Support/DynamicLibrary/CMakeLists.txt
index d8dff1ef4a3f77..2366209c25aee0 100644
--- a/llvm/unittests/Support/DynamicLibrary/CMakeLists.txt
+++ b/llvm/unittests/Support/DynamicLibrary/CMakeLists.txt
@@ -15,12 +15,22 @@ set_output_directory(DynamicLibraryLib
   LIBRARY_DIR ${LLVM_LIBRARY_OUTPUT_INTDIR}
   )
 
-add_llvm_unittest(DynamicLibraryTests
-  DynamicLibraryTest.cpp
+# FIXME: Find out why AIX fails with new DynamicLibrary symbols behavior.
+if(${CMAKE_SYSTEM_NAME} MATCHES "AIX")
+  add_llvm_unittest(DynamicLibraryTests
+    DynamicLibraryTest.cpp
+    )
+else()
+  add_llvm_unittest(DynamicLibraryTests
+    DynamicLibraryTest.cpp
 
-  EXPORT_SYMBOLS
-  )
+    EXPORT_SYMBOLS
+    )
+endif()
 target_link_libraries(DynamicLibraryTests PRIVATE DynamicLibraryLib)
+if(${CMAKE_SYSTEM_NAME} MATCHES "AIX")
+  export_executable_symbols(DynamicLibraryTests)
+endif()
 
 function(dynlib_add_module NAME)
   add_library(${NAME} MODULE