Skip to content

[build_supp] Fixes to derive_rocm_path for ucc build#2169

Merged
dpalermo merged 1 commit intoaomp-devfrom
dp_build_supp_ucc
Apr 17, 2026
Merged

[build_supp] Fixes to derive_rocm_path for ucc build#2169
dpalermo merged 1 commit intoaomp-devfrom
dp_build_supp_ucc

Conversation

@dpalermo
Copy link
Copy Markdown
Contributor

  • Fixes failures seen in the nightly build
  • Use AOMPHIP if provided
  • Use realpath/../.. since llvm is a symlink
  • Also set HIPCC as ucc otherwise defaults to /usr/bin/amdclang
  • Add checkversion for rocmopenmpi, ucx, ucc (avoids unnecessary rebuilds)

   - Fixes failures seen in the nightly build
   - Use AOMPHIP if provided
   - Use realpath/../.. since llvm is a symlink
   - Also set HIPCC as ucc otherwise defaults to /usr/bin/amdclang
   - Add checkversion for rocmopenmpi, ucx, ucc (avoids unnecessary rebuilds)
@dpalermo
Copy link
Copy Markdown
Contributor Author

Fixes needed after #2140

@dpalermo
Copy link
Copy Markdown
Contributor Author

Also no longer exits if HIP install not found...just skip the components that needed HIP.
build_supp.sh is also used with trunk builds which don't have HIP

Copy link
Copy Markdown
Member

@saiislam saiislam left a comment

Choose a reason for hiding this comment

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

Thanks for the fixes.
LGTM!

@dpalermo dpalermo merged commit fa40a6b into aomp-dev Apr 17, 2026
1 check passed
ronlieb pushed a commit that referenced this pull request Apr 18, 2026
- Fixes failures seen in the nightly build
   - Use AOMPHIP if provided
   - Use realpath/../.. since llvm is a symlink
   - Also set HIPCC as ucc otherwise defaults to /usr/bin/amdclang
   - Add checkversion for rocmopenmpi, ucx, ucc (avoids unnecessary rebuilds)
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