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] Add support for CUDA 11.4 Runtime API functions only #366

Merged
merged 1 commit into from
Jul 21, 2021

Conversation

emankov
Copy link
Collaborator

@emankov emankov commented Jul 21, 2021

  • Update hipify-perl accordingly
  • Update CUDA_Runtime_API_functions_supported_by_HIP.md accordingly

+ Update hipify-perl accordingly
+ Update CUDA_Runtime_API_functions_supported_by_HIP.md accordingly
@emankov emankov added sync Syncing update CUDA CUDA related labels Jul 21, 2021
@emankov emankov requested a review from searlmc1 July 21, 2021 12:24
@emankov emankov self-assigned this Jul 21, 2021
@emankov emankov merged commit 2f8eb9f into ROCm:master Jul 21, 2021
emankov added a commit to emankov/HIPIFY that referenced this pull request May 30, 2023
…driven matcher for Data Types substitution

[Synopsis]
+ `hipSPARSE` already uses the common HIP library type `hipDataType`, whereas `hipBLAS`, `rocBLAS`, and `rocSPARSE` don't
+ So, it is needed to hipify `cudaDataType` to `hipDataType` in some cases (like for SPARSE-related sources), and to `hipblasDatatype_t`, `rocblas_datatype`, or `rocsparse_datatype` in other cases

[Solution]
+ There is no single solid solution for sources where `cudaDataType_t` is used for both SPARSE and BLAS
+ [temporary][partial] Introduce the `--use-hip-data-types` option for use `hipDataType` instead of `hipblasDatatype_t` or `rocblas_datatype`; switched off by default
+ Introduced option-driven matcher `DataTypeSelection` for Data Types substitution
+ Check the solution on `cusparseCreateSpVec` to `hipsparseCreateSpVec` hipification, where `hipDataType` is used (the synthetic test `cusparse2hipsparse.cu`)

[ToDo]
+ File tickets to `rocBLAS` and `rocSPARSE` similar to ROCm/hipBLAS#366
+ Take into account all of the supported `cudaDataType_t` enum members and update the tests
+ Remove the workaround after all the HIP libs start to use the single `hipDataType`
emankov added a commit that referenced this pull request May 30, 2023
[HIPIFY][#674][hipBLAS-#366][workaround] Introduce an option-driven matcher for Data Types substitution
emankov added a commit to emankov/HIPIFY that referenced this pull request Nov 15, 2023
+ [Reason] `cusparseGetStream` appeared in CUDA 8.0
+ [fix] Set the missing A-version for `cusparseGetStream`
+ Updated synthetic `SPARSE` tests and the regenerated `SPARSE` documentation
+ [TODO][ROCm#899][HIPIFY][ROCm#895][ROCm#383][hipBLAS-ROCm#366][workaround][temporary] Always convert cudaDataType_t to hipDataType under the `--use-hip-data-types` option
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CUDA CUDA related sync Syncing update
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant