Skip to content

Commit d785c29

Browse files
committed
Guard visibility variables on AIX
The addition of these visibility variables caused a test-suite build failure on AIX because visibility options aren't supported. This patch guards the variables on AIX. Reviewed By: daltenty Differential Revision: https://reviews.llvm.org/D122316
1 parent 4deb79a commit d785c29

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

MicroBenchmarks/libs/benchmark/CMakeLists.txt

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -51,8 +51,10 @@ option(BENCHMARK_USE_BUNDLED_GTEST "Use bundled GoogleTest. If disabled, the fin
5151
option(BENCHMARK_ENABLE_LIBPFM "Enable performance counters provided by libpfm" OFF)
5252

5353
# Export only public symbols
54-
set(CMAKE_CXX_VISIBILITY_PRESET hidden)
55-
set(CMAKE_VISIBILITY_INLINES_HIDDEN ON)
54+
if(NOT ${CMAKE_SYSTEM_NAME} MATCHES "AIX")
55+
set(CMAKE_CXX_VISIBILITY_PRESET hidden)
56+
set(CMAKE_VISIBILITY_INLINES_HIDDEN ON)
57+
endif()
5658

5759
if(MSVC)
5860
# As of CMake 3.18, CMAKE_SYSTEM_PROCESSOR is not set properly for MSVC and

0 commit comments

Comments
 (0)