Skip to content

[SYCL] get kernel info with free functions #18866

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 11 commits into from
Jun 17, 2025
Merged

[SYCL] get kernel info with free functions #18866

merged 11 commits into from
Jun 17, 2025

Conversation

dklochkov-emb
Copy link
Contributor

This PR

  1. adds get_kernel_info functions for kernel free functions, docs
  2. fixes bug when more than one property added with add_ir_attributes_funcion, free function was not recognized as a kernel function

@dklochkov-emb dklochkov-emb requested review from a team as code owners June 9, 2025 16:40
@dklochkov-emb dklochkov-emb self-assigned this Jun 9, 2025
sergey-semenov
sergey-semenov requested changes Jun 10, 2025
View reviewed changes
dyniols
dyniols reviewed Jun 10, 2025
View reviewed changes
@lbushi25 lbushi25 mentioned this pull request Jun 11, 2025
Merged
lbushi25
lbushi25 reviewed Jun 11, 2025
View reviewed changes
Fznamznon
Fznamznon reviewed Jun 12, 2025
View reviewed changes
Fznamznon
Fznamznon approved these changes Jun 12, 2025
View reviewed changes
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.

CI fails seems to be related though.

@dklochkov-emb
[SYCL] add xfail for cpu device
ae6c323 
[SYCL] check if compile time wg size is more than runtime
@dklochkov-emb
Copy link
Contributor Author

It was added XFAIL tracker to track problems with cpu device, RT call returns empty string, need separate investigation.

@dklochkov-emb
Copy link
Contributor Author

@intel/llvm-gatekeepers please, merge it

@kbenzie
Copy link
Contributor

kbenzie commented Jun 17, 2025

@sergey-semenov requested changes so merging is blocked until those are approved

@aelovikov-intel aelovikov-intel merged commit dd7ac1a into intel:sycl Jun 17, 2025
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Fznamznon Fznamznon Fznamznon approved these changes

@sergey-semenov sergey-semenov sergey-semenov approved these changes

@dyniols dyniols dyniols approved these changes

@lbushi25 lbushi25 Awaiting requested review from lbushi25

Assignees

@dklochkov-emb dklochkov-emb

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@dklochkov-emb @kbenzie @Fznamznon @sergey-semenov @lbushi25 @dyniols @aelovikov-intel