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

Enable CMake HIP language #434

Merged

Conversation

umfranzw
Copy link
Collaborator

If the HIP language can be used, the HIP_USECXX variable sets hipCUB to use the CMake HIP language rather than CXX.

If the HIP language can be used, the HIP_USECXX variable sets rocThrust to use the CMake HIP language rather than CXX.
@umfranzw umfranzw marked this pull request as ready for review November 28, 2024 18:12
@umfranzw umfranzw requested a review from NguyenNhuDi November 28, 2024 18:16
Copy link
Contributor

@NguyenNhuDi NguyenNhuDi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@umfranzw umfranzw merged commit 7bbde6a into ROCm:release-staging/rocm-rel-6.4 Dec 3, 2024
19 of 20 checks passed
stanleytsang-amd pushed a commit that referenced this pull request Dec 9, 2024
* Include device_copy header (#433)

The device/device_copy.hpp header was missing from the main hipcub.hpp
header file. This change just adds it back.

* [windows][hipCUB] Removed usage of std::unary_function and std::binary_function to prevent syntax error (#435)

* removed ussage of std::unary/binary_function

* updated changelog

* Enable CMake HIP language (#434)

If the HIP language can be used, the HIP_USECXX variable sets rocThrust to use the CMake HIP language rather than CXX.

Co-authored-by: Lauren Wrubleski <[email protected]>

* Add hipcub::AliasTemporaries and some macros (#438)

* Resolve "Add missing macros in HIPCUB API"

* Resolve "Add AliasTemporaries function"

* Added macro names to CHANGELOG

---------

Co-authored-by: Wayne Franz <[email protected]>
Co-authored-by: Lauren Wrubleski <[email protected]>
Co-authored-by: Nick Breed <[email protected]>
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.

4 participants