Skip to content

Commit

Permalink
[UR][Spec Constants] Improved handling of invalid/unsupported spec. c…
Browse files Browse the repository at this point in the history
…onstants

Pre-commit MR for oneapi-src/unified-runtime#1469
  • Loading branch information
RossBrunton committed Aug 20, 2024
1 parent ad6ffcd commit caf9a1c
Showing 1 changed file with 19 additions and 8 deletions.
27 changes: 19 additions & 8 deletions sycl/cmake/modules/FetchUnifiedRuntime.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -116,14 +116,25 @@ if(SYCL_UR_USE_FETCH_CONTENT)
CACHE PATH "Path to external '${name}' adapter source dir" FORCE)
endfunction()

set(UNIFIED_RUNTIME_REPO "https://github.com/oneapi-src/unified-runtime.git")
# commit cabf128094eff9ff7b79bdff559640a8a111f0c3
# Merge: a96fcbc5 15bca3b6
# Author: Omar Ahmed <omar.ahmed@codeplay.com>
# Date: Mon Aug 19 16:20:45 2024 +0100
# Merge pull request #1984 from rafbiels/rafbiels/cuda-stream-race-cond
# Fix race condition in CUDA stream creation
set(UNIFIED_RUNTIME_TAG cabf128094eff9ff7b79bdff559640a8a111f0c3)
set(UNIFIED_RUNTIME_REPO "https://github.com/RossBrunton/unified-runtime.git")
# commit put-git-commit-hash-here
# Author: Ross Brunton <ross@codeplay.com>
# Date: Fri Mar 22 17:18:22 2024 +0000
#
# [Spec Constants] Improved handling of invalid/unsupported spec. constants
#
# Two main changes to how `Kernel/ProgramSetSpecializationConstants`
# are handled:
# * They may now output either `INVALID_VALUE` or the new
# `INVALID_SPEC_ID` when the provided list is invalid.
# * The OpenCL and level 0 adapters now respond to
# `UR_DEVICE_INFO_KERNEL_SET_SPECIALIZATION_CONSTANTS` with `false`
# rather than erroring out. This fixes some tests that were
# incorrectly not being skipped.
# * `urKernelSetSpecializationConstants` now "implemented" (as a
# function that returns `UNSUPPORTED_FEATURE` for a number of
# adapters.
set(UNIFIED_RUNTIME_TAG 75382064de0b959d4686cf2505a44f44902a4765)

set(UMF_BUILD_EXAMPLES OFF CACHE INTERNAL "EXAMPLES")
# Due to the use of dependentloadflag and no installer for UMF and hwloc we need
Expand Down

0 comments on commit caf9a1c

Please sign in to comment.