Skip to content

Conversation

@againull
Copy link
Contributor

No description provided.

@againull againull requested a review from a team as a code owner November 13, 2024 06:10
Copy link
Contributor

@cperkinsintel cperkinsintel left a comment

Choose a reason for hiding this comment

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

why is ocloc a requirement? We aren't calling it here or using the ocl kernel_compiler. Isn't just having the UR and a device driver enough?

@againull
Copy link
Contributor Author

why is ocloc a requirement? We aren't calling it here or using the ocl kernel_compiler. Isn't just having the UR and a device driver enough?

We are doing AOT compilation for gpu in this line https://github.com/intel/llvm/pull/16064/files#diff-ec13abf7e006d59a417d6871df6934f0cace740c92c3233be5e2b5b5d382e619L23
and it requires ocloc to be available.

@againull againull merged commit 2636a8b into intel:sycl Nov 14, 2024
14 checks passed
@againull againull deleted the fix_test branch September 29, 2025 21:30
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.

2 participants