Skip to content

Commit

Permalink
Fix some another unit test for LLVM_ENABLE_EXPORTED_SYMBOLS_IN_EXECUT…
Browse files Browse the repository at this point in the history
…ABLES=Off.

Building with -DLLVM_ENABLE_EXPORTED_SYMBOLS_IN_EXECUTABLES=Off should not
prevent the AnalysisTests unit test from working.

This fix uses the approach implemented in
#101741.

rdar://135849875
  • Loading branch information
lhames committed Sep 12, 2024
1 parent cbcf531 commit 35a0fd5
Showing 1 changed file with 2 additions and 5 deletions.
7 changes: 2 additions & 5 deletions llvm/unittests/Analysis/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -67,6 +67,8 @@ endif()

add_llvm_unittest_with_input_files(AnalysisTests
${ANALYSIS_TEST_SOURCES}

EXPORT_SYMBOLS
)

add_dependencies(AnalysisTests intrinsics_gen)
Expand All @@ -79,10 +81,5 @@ if(CMAKE_SYSTEM_NAME STREQUAL "AIX")
set(CMAKE_EXE_LINKER_FLAGS "${CMAKE_EXE_LINKER_FLAGS} -Wl,-brtl")
endif()

# Export symbols from the plugins shared objects.
if(NOT WIN32)
export_executable_symbols_for_plugins(AnalysisTests)
endif()

add_subdirectory(InlineAdvisorPlugin)
add_subdirectory(InlineOrderPlugin)

0 comments on commit 35a0fd5

Please sign in to comment.