Skip to content

fix(rocrand): resolve issues with nvcc compilation #1047

Merged
stanleytsang-amd merged 4 commits into
ROCm:developfrom
StreamHPC:users/naraenda/fix-nvcc-compilation
Aug 18, 2025
Merged

fix(rocrand): resolve issues with nvcc compilation #1047
stanleytsang-amd merged 4 commits into
ROCm:developfrom
StreamHPC:users/naraenda/fix-nvcc-compilation

Conversation

@Naraenda
Copy link
Copy Markdown
Member

@Naraenda Naraenda commented Aug 4, 2025

  • Resolves compilation issues when building with nvcc.
    • Resolve a bunch of instances where a device function was used that was defined in a host function. This is not allowed with NVCC.
  • Remove .clangd-file which was accidentally added in a previous PR.

@Naraenda Naraenda self-assigned this Aug 4, 2025
@Naraenda Naraenda requested a review from a team as a code owner August 4, 2025 10:49
@Naraenda Naraenda added project: rocrand organization: streamhpc contributors from streamhpc labels Aug 4, 2025
Copy link
Copy Markdown
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, mci failure seems unrelated to these changes. Triggered a new build to see if issue persists

@stanleytsang-amd stanleytsang-amd merged commit 1fceace into ROCm:develop Aug 18, 2025
10 of 14 checks passed
assistant-librarian Bot pushed a commit to ROCm/rocRAND that referenced this pull request Aug 18, 2025
fix(rocrand): resolve issues with nvcc compilation

- Resolves compilation issues when building with nvcc.
- Resolve a bunch of instances where a __device__ function was used that
was defined in a __host__ function. This is not allowed with NVCC.
- Remove .clangd-file which was accidentally added in a previous PR.
amd-jnovotny pushed a commit to amd-jnovotny/rocm-libraries that referenced this pull request Aug 26, 2025
CP merge pull request ROCm#1038 from estewart08/openmp-cmake-config (ROCm#1043)
amd-jnovotny pushed a commit to amd-jnovotny/rocm-libraries that referenced this pull request Aug 26, 2025
CP merge pull request ROCm#1038 from estewart08/openmp-cmake-config (ROCm#1043)

[ROCm/hipBLAS commit: 8d7ecf9]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants