Skip to content

[ESIMD] Enable oneapi::experimental::printf support for ESIMD #6432

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 1 commit into from
Jul 13, 2022
Merged

[ESIMD] Enable oneapi::experimental::printf support for ESIMD #6432

merged 1 commit into from
Jul 13, 2022

Conversation

v-klochkov
Copy link
Contributor

Signed-off-by: Vyacheslav N Klochkov [email protected]

@v-klochkov v-klochkov requested a review from a team as a code owner July 13, 2022 00:17
@v-klochkov v-klochkov mentioned this pull request Jul 13, 2022
Merged
@spencerpatty
Copy link

are things like sycl::min or sycl::max allowed in esimd context ?

@v-klochkov v-klochkov requested a review from kychendev July 13, 2022 06:13
@v-klochkov
Copy link
Contributor Author

v-klochkov commented Jul 13, 2022 •
edited
Loading

are things like sycl::min or sycl::max allowed in esimd context ?

Quick answer: Yes, sycl::minimum is supported in ESIMD context. I checked that quickly, and sycl::atan() too.
TODO here: I am not sure why sycl::minimum and funcs like sycl::atan, etc. work well, Need to debug why they do not emit an error.

@v-klochkov v-klochkov merged commit 9ba8238 into intel:sycl Jul 13, 2022
@v-klochkov v-klochkov deleted the esimd_printf branch July 13, 2022 06:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kychendev kychendev kychendev approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@v-klochkov @spencerpatty @kychendev