Skip to content
This repository was archived by the owner on Mar 28, 2023. It is now read-only.

[SYCL] Adds test for selective building in implicit kernel bundles #726

Conversation

steffenlarsen
Copy link

In select cases the SYCL runtime may create an implicit kernel bundle to for example track specialization constants. These kernel bundles should only build the device image that contains the kernel being launched.
This commit adds a test for checking that only one device image is built by an implicit kernel bundle when only a single kernel is launched and using per-kernel device-code splitting.

Test fix: intel/llvm#5285

In select cases the SYCL runtime may create an implicit kernel bundle to
for example track specialization constants. These kernel bundles should
only build the device image that contains the kernel being launched.
This commit adds a test for checking that only one device image is built
by an implicit kernel bundle when only a single kernel is launched and
using per-kernel device-code splitting.

Signed-off-by: Steffen Larsen <[email protected]>
Signed-off-by: Steffen Larsen <[email protected]>
s-kanaev
s-kanaev previously approved these changes Jan 14, 2022
Copy link

@s-kanaev s-kanaev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@steffenlarsen
Copy link
Author

@v-klochkov & @vladimirlaz - intel/llvm#5285 was merged so this should be ready to merge.

@vladimirlaz
Copy link

@steffenlarsen the test is failing on cuda (I guess it will fail on hip and esimd_emulator as well)
That is caused by no JIT compilation is happening on those platforms. Could you please update the test acordingly?

Signed-off-by: Steffen Larsen <[email protected]>
@vladimirlaz vladimirlaz merged commit fcbb20a into intel:intel Feb 14, 2022
myler pushed a commit to myler/llvm-test-suite that referenced this pull request Apr 12, 2022
…ntel#726)

* In select cases the SYCL runtime may create an implicit kernel bundle to
for example track specialization constants. These kernel bundles should
only build the device image that contains the kernel being launched.
This commit adds a test for checking that only one device image is built
by an implicit kernel bundle when only a single kernel is launched and
using per-kernel device-code splitting.

* Disable on CUDA and HIP

Signed-off-by: Steffen Larsen <[email protected]>
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
…ntel/llvm-test-suite#726)

* In select cases the SYCL runtime may create an implicit kernel bundle to
for example track specialization constants. These kernel bundles should
only build the device image that contains the kernel being launched.
This commit adds a test for checking that only one device image is built
by an implicit kernel bundle when only a single kernel is launched and
using per-kernel device-code splitting.

* Disable on CUDA and HIP

Signed-off-by: Steffen Larsen <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants