Skip to content

[SYCL] Allow [[sycl::device_has]] attribute on kernel #5503

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 2 commits into from
Feb 9, 2022

Conversation

elizabethandrews
Copy link
Contributor

The original implementation of this attribute in
#5166, did not support
applying the attribute on the kernel itself. This patch
fixes that. The attribute can be applied to the operator
method of kernel functor or on the kernel lambda.

Signed-off-by: Elizabeth Andrews elizabeth.andrews@intel.com

The original implementation of this attribute in
intel#5166, did not support
applying the attribute on the kernel itself. This patch
fixes that. The attribute can be applied to the operator
method of kernel functor or on the kernel lambda.

Signed-off-by: Elizabeth Andrews <elizabeth.andrews@intel.com>
@elizabethandrews elizabethandrews requested a review from a team as a code owner February 8, 2022 00:31
@bader bader changed the title Allow [[sycl::device_has]] attribute on kernel [SYCL] Allow [[sycl::device_has]] attribute on kernel Feb 8, 2022
Copy link
Contributor

@Fznamznon Fznamznon left a comment

Choose a reason for hiding this comment

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

Overall LGTM, only have a question to the test.

Signed-off-by: Elizabeth Andrews <elizabeth.andrews@intel.com>
Copy link
Contributor

@smanna12 smanna12 left a comment

Choose a reason for hiding this comment

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

LGTM.

@smanna
Copy link

smanna commented Feb 8, 2022

could you please stop mentioning me. I'm not involved in this project.

@elizabethandrews
Copy link
Contributor Author

could you please stop mentioning me. I'm not involved in this project.

Oh I'm sorry. I didn't realize I was tagging the wrong person. Thanks for letting me know!

@elizabethandrews
Copy link
Contributor Author

Thanks for the reviews!

@bader bader merged commit aa2162c into intel:sycl Feb 9, 2022
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.

7 participants