Skip to content

[CI][NFC] Add clarification for ext_oneapi_can_compile change #19764

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
steffenlarsen merged 2 commits into from
Aug 13, 2025
Merged

[CI][NFC] Add clarification for ext_oneapi_can_compile change #19764

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
a39d39e
[CI][NFC] Add clarification for ext_oneapi_can_compile change
steffenlarsen Aug 12, 2025
c018b34
Move clarification
steffenlarsen Aug 13, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 10 additions & 5 deletions devops/compat_ci_exclude.sycl-rel-6_2
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,16 @@ DiscardEvents/invalid_event_exceptions.cpp
# Throw exception instead of returning garbage
Basic/info.cpp

# https://github.com/intel/llvm/pull/17442 changed the behavior of
# ext_oneapi_can_compile to refer to whether a source-based kernel_bundle can
# be used with compile rather than build. This makes the OpenCL kernel compiler
# tests from the previous release fail as they expect that it returns true for
# OpenCL-based kernel_bundles. The APIs are experimental, so this change in
# behavior is allowed and is not considered an API/ABI break.
KernelCompiler/opencl.cpp
KernelCompiler/opencl_cache_eviction.cpp
KernelCompiler/opencl_queries.cpp

# Likely OK, but need author to provide justification, get approval/confirmation
# from someone:

Expand Down Expand Up @@ -48,11 +58,6 @@ Assert/assert_in_simultaneous_kernels.cpp
Assert/assert_in_simultaneously_multiple_tus.cpp
Assert/assert_in_simultaneously_multiple_tus_one_ndebug.cpp

# https://github.com/intel/llvm/pull/17442
KernelCompiler/opencl.cpp
KernelCompiler/opencl_cache_eviction.cpp
KernelCompiler/opencl_queries.cpp

# https://github.com/intel/llvm/pull/18403 (pulldown, so probably offload-tools
# team should be looking into this)
DeviceImageDependencies/NewOffloadDriver/dynamic.cpp
Expand Down
Loading