Skip to content

Commit 47ec6be

Browse files
mstorsjollvmbot
authored andcommitted
[OpenMP] [cmake] Don't use -fno-semantic-interposition on Windows (llvm#81113)
This was added in 4b7beab. When the flag was added implicitly elsewhere, it was added via llvm/cmake/modules/HandleLLVMOptions.cmake, where it wasn't added on Windows/Cygwin targets. This avoids one warning per object file in OpenMP. (cherry picked from commit 72f04fa)
1 parent 391d948 commit 47ec6be

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

Diff for: openmp/cmake/HandleOpenMPOptions.cmake

+5-1
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,11 @@ append_if(OPENMP_HAVE_WEXTRA_FLAG "-Wno-extra" CMAKE_C_FLAGS CMAKE_CXX_FLAGS)
4646
append_if(OPENMP_HAVE_WPEDANTIC_FLAG "-Wno-pedantic" CMAKE_C_FLAGS CMAKE_CXX_FLAGS)
4747
append_if(OPENMP_HAVE_WMAYBE_UNINITIALIZED_FLAG "-Wno-maybe-uninitialized" CMAKE_C_FLAGS CMAKE_CXX_FLAGS)
4848

49-
append_if(OPENMP_HAVE_NO_SEMANTIC_INTERPOSITION "-fno-semantic-interposition" CMAKE_C_FLAGS CMAKE_CXX_FLAGS)
49+
if (NOT (WIN32 OR CYGWIN))
50+
# This flag is not relevant on Windows; the flag is accepted, but produces warnings
51+
# about argument unused during compilation.
52+
append_if(OPENMP_HAVE_NO_SEMANTIC_INTERPOSITION "-fno-semantic-interposition" CMAKE_C_FLAGS CMAKE_CXX_FLAGS)
53+
endif()
5054
append_if(OPENMP_HAVE_FUNCTION_SECTIONS "-ffunction-section" CMAKE_C_FLAGS CMAKE_CXX_FLAGS)
5155
append_if(OPENMP_HAVE_DATA_SECTIONS "-fdata-sections" CMAKE_C_FLAGS CMAKE_CXX_FLAGS)
5256

0 commit comments

Comments
 (0)