Skip to content

Commit 53d6039

Browse files
authored
[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
1 parent 5cead0c commit 53d6039

File tree

1 file changed

+14
-4
lines changed

1 file changed

+14
-4
lines changed

llvm/unittests/Support/DynamicLibrary/CMakeLists.txt

Lines changed: 14 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -15,12 +15,22 @@ set_output_directory(DynamicLibraryLib
1515
LIBRARY_DIR ${LLVM_LIBRARY_OUTPUT_INTDIR}
1616
)
1717

18-
add_llvm_unittest(DynamicLibraryTests
19-
DynamicLibraryTest.cpp
18+
# FIXME: Find out why AIX fails with new DynamicLibrary symbols behavior.
19+
if(${CMAKE_SYSTEM_NAME} MATCHES "AIX")
20+
add_llvm_unittest(DynamicLibraryTests
21+
DynamicLibraryTest.cpp
22+
)
23+
else()
24+
add_llvm_unittest(DynamicLibraryTests
25+
DynamicLibraryTest.cpp
2026

21-
EXPORT_SYMBOLS
22-
)
27+
EXPORT_SYMBOLS
28+
)
29+
endif()
2330
target_link_libraries(DynamicLibraryTests PRIVATE DynamicLibraryLib)
31+
if(${CMAKE_SYSTEM_NAME} MATCHES "AIX")
32+
export_executable_symbols(DynamicLibraryTests)
33+
endif()
2434

2535
function(dynlib_add_module NAME)
2636
add_library(${NAME} MODULE

0 commit comments

Comments
 (0)