Merge pull request #1210 from AllanZyne/sanitizer-pr-cpu-local

[DeviceSanitizer] Support detecting out-of-bounds error on CPU Device & Static Local Memory
oneapi-src · Feb 1, 2024 · c6b0a24 · c6b0a24
2 parents 5d5c810 + 0631330
commit c6b0a24
Show file tree

Hide file tree

Showing 7 changed files with 401 additions and 76 deletions.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -126,7 +126,15 @@ if(UR_ENABLE_TRACING)
 endif()
 
 if(UR_ENABLE_SANITIZER)
-    add_compile_definitions(UR_ENABLE_SANITIZER)
+    if(APPLE)
+        message(WARNING "Sanitizer layer isn't supported on macOS")
+        set(UR_ENABLE_SANITIZER OFF)
+    elseif(WIN32)
+        message(WARNING "Sanitizer layer isn't supported on Windows")
+        set(UR_ENABLE_SANITIZER OFF)
+    else()
+        add_compile_definitions(UR_ENABLE_SANITIZER)
+    endif()
 endif()
 
 if(UR_USE_ASAN)

diff --git a/source/loader/CMakeLists.txt b/source/loader/CMakeLists.txt
@@ -114,7 +114,14 @@ if(UR_ENABLE_SANITIZER)
         ${CMAKE_CURRENT_SOURCE_DIR}/layers/sanitizer/ur_sanitizer_layer.hpp
         ${CMAKE_CURRENT_SOURCE_DIR}/layers/sanitizer/ur_sanddi.cpp
     )
+
+    target_sources(ur_loader
+        PRIVATE
+        ${CMAKE_CURRENT_SOURCE_DIR}/layers/sanitizer/linux/san_utils.cpp
+    )
+
     target_include_directories(ur_loader PRIVATE
+        "${CMAKE_CURRENT_SOURCE_DIR}/layers/sanitizer"
         "${CMAKE_CURRENT_SOURCE_DIR}/../"
     )
 endif()