Skip to content
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

Upgrade temporary __buffer infrastructure to use USM where possible #1906

Open
danhoeflinger opened this issue Oct 15, 2024 · 0 comments
Open
Assignees

Comments

@danhoeflinger
Copy link
Contributor

It would be good to upgrade __buffer_impl within oneDPL to use USM when it is able to for performance reasons.
We should be able to just utilize __result_and_scratch_storage inside of __buffer_impl in parallel_backend_sycl_utils.h, but there may be some nuance and complications when handling arbitrary typed value types.

This has the potential to provide a performance boost to all APIs which use temporary storage internally in the form of a sycl buffer currently.

This issue stems from conversation in #1879 (comment)_

@danhoeflinger danhoeflinger self-assigned this Oct 15, 2024
@danhoeflinger danhoeflinger changed the title Upgrade __buffer infrastructure to usm USM where possible Upgrade __buffer infrastructure to use USM where possible Oct 15, 2024
@danhoeflinger danhoeflinger changed the title Upgrade __buffer infrastructure to use USM where possible Upgrade temporary __buffer infrastructure to use USM where possible Oct 15, 2024
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

No branches or pull requests

1 participant