[SYCL] Fix kernel bundles don't really carry kernel IDs#5121
[SYCL] Fix kernel bundles don't really carry kernel IDs#5121bader merged 7 commits intointel:syclfrom
Conversation
|
PR with test |
v-klochkov
left a comment
v-klochkov
left a comment
There was a problem hiding this comment.
This PR does not change/remove any global symbols, and only creates some, which is non-breaking allowed ABI change.
|
@denis-kabanov, please, fix pre-commit test failures for intel/llvm-test-suite#616. |
@bader, according to verify with #5121, my test passed all checks, whereas failed tests (Assert, DeviceLib) are a common problem for each pr to llvm-test-suite. |
|
@denis-kabanov, @intel/llvm-reviewers-runtime, I would like patches to pass all tests in pre-commit. Please, address common problems first. If it takes time, we can disable failing tests while working on them. |
Disable Basic/get_backend test due to sporadic failures in prs: intel/llvm#5191 intel/llvm#5121
|
A failed test is irrelevant to these changes. |
Check for carrying real kernel IDs passed Jenkins/llvm-test-suite. |
Disable Basic/get_backend test due to sporadic failures in prs: intel/llvm#5191 intel/llvm#5121
* upstream/sycl: (1382 commits) [SYCL][XPTI] Report memory allocation info from SYCL runtime (intel#5172) [CI] Switch labels for OCL x64 job (intel#5185) [SYCL] Add basic support for the generic_space address space (intel#5148) [CI] Update CODEOWNERS for SYCL printf support passes (intel#5199) [SYCL][Matrix] Enable wi_slice for joint_matrix (intel#4979) [SYCL][Group algorithms] Move group sort extension to experimental (intel#5169) [SYCL] Fix kernel bundles don't really carry kernel IDs (intel#5121) [SYCL] Initial printf support for non-constant AS format strings (intel#5069) [SYCL][NFC] Fix static code analysis concerns (intel#5189) [SYCL][Doc] Fix typos to fix doc build (intel#5190) [Driver][SYCL] Turn on -fsycl-dead-args-optimization by default (intel#3004) [SYCL][L0][Plugin] Add support for batching copy commands (intel#5155) [CI] Add cache checkout script to docker containers (intel#5184) [SYCL][Doc] Add HIP backend to the filter selector (intel#5176) [Doc] Add documentation for Docker images (intel#4778) [LIBCLC] Add functionality for in-kernel asserts for CUDA backend (intel#5174) Force opt to use new pass manager in exponential-deferred-inlining test after a8c2ba1 [SYCL] Add vec and marray support to known_identity type trait (intel#5163) Correctly resolve merge conflicts Update SPV_INTEL_hw_thread_queries to rev 2 ...
Disable Basic/get_backend test due to sporadic failures in prs: intel#5191 intel#5121
5 participants
Fix that interop kernel bundles don't really carry kernel IDs for contained kernels. This change adds In enqueueImpKernel another condition to taking the path using the associated kernel_bundle, namely that interop bundles can't be used in the first branch. (This is because the kernels in interop kernel bundles (if any) do not have kernel_id and can therefore not be looked up, but since they are self-contained they can simply be launched directly.)