Skip to content

[SYCL][UR][L0 v2] add missing offset calculation to #18355

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
merged 1 commit into from
May 12, 2025

Conversation

igchor
Copy link
Member

@igchor igchor commented May 7, 2025

getDevicePtr and mapHostPtr for ur_usm_handle_t.

We never actually use the handle with non-zero offset but we may change that in future.

@igchor igchor requested a review from a team as a code owner May 7, 2025 18:26
getDevicePtr and mapHostPtr for ur_usm_handle_t.

We never actually use the handle with non-zero offset
but we may change that in future.
@igchor igchor force-pushed the non_zero_offset branch from 5d86a2c to e32cf99 Compare May 7, 2025 18:30
@igchor igchor temporarily deployed to WindowsCILock May 7, 2025 18:31 — with GitHub Actions Inactive
@igchor igchor temporarily deployed to WindowsCILock May 7, 2025 18:50 — with GitHub Actions Inactive
@igchor igchor temporarily deployed to WindowsCILock May 7, 2025 18:50 — with GitHub Actions Inactive
@pbalcer
Copy link
Contributor

pbalcer commented May 12, 2025

CI failure is unrelated:

Failed Tests (1):
  SYCL :: ThreadSanitizer/check_device_global.cpp

#18349

@pbalcer
Copy link
Contributor

pbalcer commented May 12, 2025

@intel/llvm-gatekeepers please merge

@steffenlarsen steffenlarsen merged commit cda5a47 into intel:sycl May 12, 2025
18 of 19 checks passed
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