Skip to content

Commit dfbbbf3

Browse files
committed
Don't build libdevice sanitizer if UR_SANITIZER_INCLUDE_DIR is empty
1 parent 691d786 commit dfbbbf3

File tree

2 files changed

+9
-6
lines changed

2 files changed

+9
-6
lines changed

libdevice/CMakeLists.txt

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,8 @@ elseif(EXISTS ${SYCL_UR_SOURCE_DIR})
1515
set(UR_SANITIZER_INCLUDE_DIR
1616
${SYCL_UR_SOURCE_DIR}/source/loader/layers/sanitizer)
1717
else()
18-
message(FATAL_ERROR "Unified Runtime source directory is not provided")
18+
message(WARNING "Unified Runtime source directory is not provided, \
19+
will not build libdevice sanitizer")
1920
endif()
2021

2122
# Build libdevice for SYCL.

libdevice/cmake/modules/SYCLLibdevice.cmake

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -200,7 +200,7 @@ set(cmath_obj_deps device_math.h device.h sycl-compiler)
200200
set(imf_obj_deps device_imf.hpp imf_half.hpp imf_bf16.hpp imf_rounding_op.hpp imf_impl_utils.hpp device.h sycl-compiler)
201201
set(itt_obj_deps device_itt.h spirv_vars.h device.h sycl-compiler)
202202
set(bfloat16_obj_deps sycl-headers sycl-compiler)
203-
if (NOT MSVC)
203+
if (NOT MSVC AND UR_SANITIZER_INCLUDE_DIR)
204204
set(sanitizer_obj_deps
205205
device.h atomic.hpp spirv_vars.h
206206
${UR_SANITIZER_INCLUDE_DIR}/asan_libdevice.hpp
@@ -268,10 +268,12 @@ if(MSVC)
268268
SRC msvc_math.cpp
269269
DEPENDENCIES ${cmath_obj_deps})
270270
else()
271-
add_devicelibs(libsycl-sanitizer
272-
SRC sanitizer_utils.cpp
273-
DEPENDENCIES ${sanitizer_obj_deps}
274-
EXTRA_OPTS -fno-sycl-instrument-device-code -I${UR_SANITIZER_INCLUDE_DIR})
271+
if(UR_SANITIZER_INCLUDE_DIR)
272+
add_devicelibs(libsycl-sanitizer
273+
SRC sanitizer_utils.cpp
274+
DEPENDENCIES ${sanitizer_obj_deps}
275+
EXTRA_OPTS -fno-sycl-instrument-device-code -I${UR_SANITIZER_INCLUDE_DIR})
276+
endif()
275277
endif()
276278

277279
add_devicelibs(libsycl-fallback-cassert

0 commit comments

Comments
 (0)