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

Fixes for HIP detection for recent ROCm and CMake #1804

Merged
merged 23 commits into from
Sep 26, 2024

Conversation

havogt
Copy link
Contributor

@havogt havogt commented Sep 19, 2024

  • New minimum CMake version requirement 3.21.0 for HIP language support
  • Hip detection via check_language instead of compiler binary name
  • Enable workarounds for Clang 17 (which where already needed for Clang 16 and earlier)
  • Fix for deprecated ptrAttributes.memoryType (was already changed in CUDA since a while)
  • Add cuda error checks for functions that return an error with [[no_discard]] in hip.

@havogt havogt marked this pull request as draft September 19, 2024 09:23
@havogt
Copy link
Contributor Author

havogt commented Sep 24, 2024

launch jenkins

@havogt havogt requested a review from fthaler September 24, 2024 09:20
@havogt havogt marked this pull request as ready for review September 24, 2024 09:20
@havogt
Copy link
Contributor Author

havogt commented Sep 24, 2024

launch jenkins

CMakeLists.txt Show resolved Hide resolved
CMakeLists.txt Outdated Show resolved Hide resolved
cmake/public/detect_features.cmake Outdated Show resolved Hide resolved
cmake/public/gridtools_setup_targets.cmake Outdated Show resolved Hide resolved
include/gridtools/common/cuda_util.hpp Outdated Show resolved Hide resolved
Co-authored-by: Felix Thaler <felix.thaler@nummi.ch>
@havogt
Copy link
Contributor Author

havogt commented Sep 25, 2024

launch jenkins

@havogt havogt requested a review from fthaler September 25, 2024 08:41
@havogt
Copy link
Contributor Author

havogt commented Sep 25, 2024

launch jenkins

@havogt havogt merged commit fde30fc into GridTools:master Sep 26, 2024
69 checks passed
@havogt havogt deleted the update_hip_detection branch September 26, 2024 06:58
havogt added a commit that referenced this pull request Sep 30, 2024
- New minimum CMake version requirement 3.21.0 for HIP language support
- Hip detection via `check_language` instead of compiler binary name
- Enable workarounds for Clang 17 (which where already needed for Clang
16 and earlier)
- Fix for deprecated `ptrAttributes.memoryType` (was already changed in
CUDA since a while)
- Add cuda error checks for functions that return an error with
`[[no_discard]]` in hip.

---------

Co-authored-by: Felix Thaler <felix.thaler@nummi.ch>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants