Skip to content

[SYCL][ext] Add host impl for bf16 conversion #6492

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
Closed
wants to merge 1 commit into from
Closed

[SYCL][ext] Add host impl for bf16 conversion #6492

wants to merge 1 commit into from

Conversation

hdelan
Copy link
Contributor

@hdelan hdelan commented Jul 29, 2022 •
edited
Loading

Instead of throwing an error, it would be convenient if bfloat16 conversions could be done on host as well as device.

cc @JackAKirk

@hdelan hdelan requested a review from a team as a code owner July 29, 2022 14:23
@hdelan hdelan requested a review from smaslov-intel July 29, 2022 14:23
@hdelan hdelan changed the title Add host impl for bf16 conversion [SYCL][ext] Add host impl for bf16 conversion Jul 29, 2022
@JackAKirk JackAKirk mentioned this pull request Aug 4, 2022
Merged
@steffenlarsen
Copy link
Contributor

I think this conflicts with #5954. Tagging @MrSidims to discuss the best solution.

@MrSidims
Copy link
Contributor

I think this conflicts with #5954. Tagging @MrSidims to discuss the best solution.

I don't have a strong opinion, but #5954 has a dummy test for the feature.

@hdelan hdelan closed this Aug 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@smaslov-intel smaslov-intel Awaiting requested review from smaslov-intel smaslov-intel is a code owner automatically assigned from intel/llvm-reviewers-runtime

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
@hdelan @steffenlarsen @MrSidims