Skip to content

Commit

Permalink
[really re-add -fvisibility-inlines-hidden on Apple targets]
Browse files Browse the repository at this point in the history
  • Loading branch information
kinke committed Jul 23, 2023
1 parent e3b605e commit 04fa0d5
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -686,7 +686,7 @@ if(NOT WIN32 AND NOT CYGWIN)
# different translation units being compiled with different visibility settings."
# See LLVM's cmake/modules/HandleLLVMOptions.cmake.
check_cxx_compiler_flag("-fvisibility-inlines-hidden" SUPPORTS_FVISIBILITY_INLINES_HIDDEN_FLAG)
if (UNIX AND LDC_ENABLE_PLUGINS)
if (LDC_ENABLE_PLUGINS AND NOT APPLE)
# For plugins, we shouldn't apply this flag because it hides the inline methods of e.g. Visitor. On macOS it's OK to add.
elseif (${SUPPORTS_FVISIBILITY_INLINES_HIDDEN_FLAG})
append("-fvisibility-inlines-hidden" LDC_CXXFLAGS)
Expand Down

0 comments on commit 04fa0d5

Please sign in to comment.