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

[SYCL][Bindless] Refactor existing bindless aspects for missing L0 bindless functionality #14323

Merged
merged 21 commits into from
Jul 23, 2024

Conversation

DBDuncan
Copy link
Contributor

Add device aspects query if a device supports image arrays, unique addressing per dimension and support for sampling 1D usm images.

Remove export semaphore, export image memory and fetch 3D usm image aspects as they are not supported on any backend.

Update Level Zero and CUDA aspect results to reflect current support status with latest drivers.

…ndless functionality

Add device aspects query if a device supports image arrays, unique addressing per dimension and support for sampling 1D usm images.

Remove export semaphore, export image memory and fetch 3D usm image aspects as they are not supported on any backend.

Update Level Zero and CUDA aspect results to reflect current support status with latest drivers.
@DBDuncan
Copy link
Contributor Author

Corresponding UR PR: oneapi-src/unified-runtime#1795

@DBDuncan DBDuncan marked this pull request as ready for review July 8, 2024 15:35
@DBDuncan DBDuncan requested review from a team as code owners July 8, 2024 15:35
@DBDuncan DBDuncan requested a review from againull July 8, 2024 15:35
@DBDuncan
Copy link
Contributor Author

DBDuncan commented Jul 9, 2024

Friendly ping @intel/dpcpp-tools-reviewers @intel/unified-runtime-reviewers @intel/llvm-reviewers-runtime @againull

@DBDuncan
Copy link
Contributor Author

Friendly ping @intel/dpcpp-tools-reviewers

@DBDuncan
Copy link
Contributor Author

@intel/dpcpp-tools-reviewers Review please.

@DBDuncan
Copy link
Contributor Author

@sarnex @intel/dpcpp-tools-reviewers Please review. Need approval as soon as you can.

Copy link
Contributor

@sarnex sarnex left a comment

Choose a reason for hiding this comment

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

lgtm, sorry for delay

Copy link
Contributor

@kbenzie kbenzie left a comment

Choose a reason for hiding this comment

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

oneapi-src/unified-runtime#1795 has been merged, please update the UR repo/tag as suggested and pull in the latest changes from the sycl branch.

sycl/plugins/unified_runtime/CMakeLists.txt Outdated Show resolved Hide resolved
Copy link
Contributor

@kbenzie kbenzie left a comment

Choose a reason for hiding this comment

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

UR LGTM

@DBDuncan
Copy link
Contributor Author

@intel/llvm-gatekeepers LGTM

@sarnex sarnex merged commit 67fa084 into intel:sycl Jul 23, 2024
15 checks passed
hdelan pushed a commit to hdelan/llvm that referenced this pull request Jul 26, 2024
…ndless functionality (intel#14323)

Add device aspects query if a device supports image arrays, unique
addressing per dimension and support for sampling 1D usm images.

Remove export semaphore, export image memory and fetch 3D usm image
aspects as they are not supported on any backend.

Update Level Zero and CUDA aspect results to reflect current support
status with latest drivers.
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.

5 participants