Skip to content

Conversation

MartinWehking
Copy link

Needed for intel/llvm#11626

@MartinWehking MartinWehking marked this pull request as ready for review October 23, 2023 15:36
@MartinWehking MartinWehking requested a review from a team as a code owner October 23, 2023 15:36
@ldrumm
Copy link
Contributor

ldrumm commented Oct 23, 2023

Does this feature need to be enabled conditionally depending on the arch properties?

@MartinWehking
Copy link
Author

@ldrumm The only problematic one is Tahiti, which doesn't support fp16, but we decided to ignore Tahiti in this case.
There's unfortunately no possibility to query fp16 support dynamically (for fp64 there's a member inside 'hipDeviceArch_t', rocm/include/hip/hip_runtime_api.h, but not for fp16)

@JackAKirk
Copy link
Contributor

JackAKirk commented Oct 24, 2023

Tahiti

Tahiti is very old and not officially supported by ROCm anyway. This should be fine.

againull pushed a commit to intel/llvm that referenced this pull request Oct 26, 2023
Enables fp16 support for AMD GPUs.
Based on Zheming's previous work:
[8488](#8488)
Some test cases for e.g. images were disabled since they aren't
supported

Draft PR for enabling fp16 support in the unified-runtime:
oneapi-src/unified-runtime#988

---------

Co-authored-by: Jin Z <[email protected]>
@zjin-lcf
Copy link

zjin-lcf commented Dec 1, 2023

Thanks.

@kbenzie kbenzie added the ready to merge Added to PR's which are ready to merge label Dec 1, 2023
@fabiomestre fabiomestre changed the base branch from adapters to main December 5, 2023 16:53
@fabiomestre
Copy link
Contributor

I have updated the target branch of this PR from the adapters branch to the main branch.
Development in UR is moving back to main. The adapters branch will soon be deleted.

@kbenzie kbenzie merged commit 364a299 into oneapi-src:main Dec 6, 2023
kbenzie added a commit to kbenzie/unified-runtime that referenced this pull request Dec 6, 2023
kbenzie added a commit to kbenzie/intel-llvm that referenced this pull request Dec 6, 2023
steffenlarsen pushed a commit to intel/llvm that referenced this pull request Dec 8, 2023
kbenzie added a commit to kbenzie/unified-runtime that referenced this pull request Dec 15, 2023
callumfare pushed a commit to kbenzie/intel-llvm that referenced this pull request Dec 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready to merge Added to PR's which are ready to merge v0.8.x Include in the v0.8.x release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants