Skip to content

Commit 024e99c

Browse files
committed
Prevent warnings treated as errors internally for level-zero thirdparty
1 parent 4d3a534 commit 024e99c

File tree

1 file changed

+11
-7
lines changed

1 file changed

+11
-7
lines changed

thirdparty/level_zero/CMakeLists.txt

+11-7
Original file line numberDiff line numberDiff line change
@@ -10,24 +10,28 @@ set(BUILD_SHARED_LIBS OFF)
1010
set(CMAKE_DEBUG_POSTFIX "")
1111

1212
# Skip warnings as errors for thirdparty
13+
set(ze_loader_sources_flags) # list, not string
14+
1315
if(CMAKE_CXX_COMPILER_ID STREQUAL "MSVC")
14-
ov_add_compiler_flags(/WX-)
16+
set(ze_loader_sources_flags /WX-)
1517
# solve pdb access issue
1618
set(USE_Z7 ON)
1719
# Close spectre for ze loader
18-
add_compile_options("/Qspectre-")
20+
set(ze_loader_sources_flags ${ze_loader_sources_flags} /Qspectre-)
1921
elseif(CMAKE_COMPILER_IS_GNUCXX OR OV_COMPILER_IS_CLANG)
20-
ov_add_compiler_flags(-Wno-error)
21-
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} \
22-
-Wno-undef \
23-
-Wno-missing-declarations")
22+
set(ze_loader_sources_flags -Wno-error -Wno-undef -Wno-missing-declarations)
2423
if(UNUSED_BUT_SET_VARIABLE_SUPPORTED)
25-
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-unused-but-set-variable")
24+
set(ze_loader_sources_flags ${ze_loader_sources_flags} -Wno-unused-but-set-variable)
2625
endif()
2726
endif()
2827
set(CMAKE_COMPILE_WARNING_AS_ERROR OFF)
2928
add_subdirectory(level-zero EXCLUDE_FROM_ALL)
3029

30+
# Level-zero thirdparty treats warnings as errors internally, need to set compile options for its source files
31+
get_target_property(ze_loader_sources ze_loader SOURCES)
32+
list(FILTER ze_loader_sources INCLUDE REGEX ".*.cpp")
33+
set_source_files_properties(${ze_loader_sources} TARGET_DIRECTORY ze_loader PROPERTIES COMPILE_OPTIONS "${ze_loader_sources_flags}")
34+
3135
set_property(TARGET ze_loader APPEND PROPERTY INTERFACE_INCLUDE_DIRECTORIES $<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/level-zero/include>)
3236

3337
# This VERSION file created by L0 may cause compilation issue of oneTBB headers, so remove it

0 commit comments

Comments
 (0)