Skip to content

[SYCL] Add support for ZEBIN format in persistent cache #6311

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.

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

Conversation

steffenlarsen
Copy link
Contributor

Following the changes in #6256 this commit allows executable device binaries with ZEBIN format to be cached persistently.

Following the changes in intel#6256 this
commit allows executable device binaries with ZEBIN format to be cached
persistently.

Signed-off-by: Larsen, Steffen <[email protected]>
@pvchupin pvchupin merged commit 34dcf83 into intel:sycl Jun 15, 2022
steffenlarsen added a commit to steffenlarsen/llvm that referenced this pull request Jun 22, 2022
intel#6311 added support for ZEBIN
executable binaries in the persistent device code cache by identifying
them through their ELF header type. However, newer ZEBIN binaries do
not have this header type and thus must be identified in another way.
This commit adds identification of this format by looking for the
.ze_info section that must be present in ZEBIN binaries.

Signed-off-by: Larsen, Steffen <[email protected]>
pvchupin pushed a commit that referenced this pull request Jun 30, 2022
#6311 added support for ZEBIN
executable binaries in the persistent device code cache by identifying
them through their ELF header type. However, newer ZEBIN binaries do
not have this header type and thus must be identified in another way.
This commit adds identification of this format by looking for the
.ze_info section that must be present in ZEBIN binaries.

Signed-off-by: Larsen, Steffen <[email protected]>
@steffenlarsen steffenlarsen deleted the steffen/zebin_persistent_cache_support branch December 6, 2023 11:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants