Skip to content
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.

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

[HIPIFY][6.0.0][hipBLAS#366] Support for ROCm HIP 6.0.0 - Step 4 - hipblasDatatype_t -> hipDataType #1065

Merged
merged 1 commit into from
Oct 13, 2023

Conversation

emankov
Copy link
Collaborator

@emankov emankov commented Oct 13, 2023

[IMP]

[TODO]

…ipblasDatatype_t` -> `hipDataType`

[IMP]
+ In hipBLAS 6.0.0, ROCm/hipBLAS#366 is finally fixed, thus HIPIFY can use `hipDataType` instead of `hipblasDatatype_t`
+ `hipDataType` can be found in `hip/library_types.h` (also mapped to CUDA's `library_types.h`)
+ `rocblas_datatype` is left unchanged

[TODO]
+ Revise all the hipBLAS functions which use `hipblasDatatype_t` or `hipDataType`
+ Remove now the unnecessary option `-use-hip-data-types`
+ Close ROCm/hipBLAS/issues/366 as implemented with the releasing of hipBLAS 6.0.0
+ [long-term] Move `hipDataType` from BLAS to Runtime, when other than hipBLAS libs will start using `hipDataType`
@emankov emankov added feature Feature request or implementation HIP sync Syncing update BLAS BLAS related labels Oct 13, 2023
@emankov emankov requested a review from searlmc1 October 13, 2023 20:17
@emankov emankov self-assigned this Oct 13, 2023
@emankov emankov merged commit 9bb62de into ROCm:amd-staging Oct 13, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BLAS BLAS related feature Feature request or implementation HIP sync Syncing update
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[5.5.X] hipblasDatatype_t should be replaced with HIP's hipDataType
1 participant