Skip to content

aocc: help flang find gcc#59

Merged
bernhardkaindl merged 1 commit into
spack:developfrom
nolta:spack-pr-50227
Aug 22, 2025
Merged

aocc: help flang find gcc#59
bernhardkaindl merged 1 commit into
spack:developfrom
nolta:spack-pr-50227

Conversation

@nolta
Copy link
Copy Markdown
Contributor

@nolta nolta commented Jun 6, 2025

Note

This PR was migrated from spack/spack#50227.

AOCC flang can't find gcc if gcc is installed in a non-standard location.

This commit:

  • Patches flang1 & flang2 with the rpath for libquadmath.
  • Creates a wrapper script to pass options to flang, since flang.cfg is currently ignored.
  • Passes --gcc-toolchain.
  • Passes -Wl,-rpath so executables can find libquadmath.
  • Passes -Wno-unused-command-line-argument to silence warnings generated by -Wl,-rpath, since they can cause problems.

flang can't find gcc if gcc is installed in a non-standard location.

This commit:

- Patches flang1 & flang2 with the rpath for libquadmath.
- Creates a wrapper script to pass options to flang, since flang.cfg
  is currently ignored.
- Passes `--gcc-toolchain`.
- Passes `-Wl,-rpath` so executables can find libquadmath.
- Passes `-Wno-unused-command-line-argument` to silence warnings
  generated by `-Wl,-rpath`, since they can cause problems.
@tldahlgren tldahlgren added the waiting-on-maintainer Waiting on a review from the package maintainer label Jun 10, 2025
Comment thread repos/spack_repo/builtin/packages/aocc/package.py
Copy link
Copy Markdown
Contributor

@bernhardkaindl bernhardkaindl left a comment

Choose a reason for hiding this comment

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

@amd-toolchain-support, see the comment by @nolta, in the resolved discussion thread.

As you're maintaining aocc, is this good to merge by you?

Copy link
Copy Markdown
Contributor

@amd-toolchain-support amd-toolchain-support left a comment

Choose a reason for hiding this comment

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

These changes looks good,

@amd-toolchain-support
Copy link
Copy Markdown
Contributor

@amd-toolchain-support, see the comment by @nolta, in the resolved discussion thread.

As you're maintaining aocc, is this good to merge by you?

Apologies for delay,
In my testing, did not encountered any issue with these changes as such. Approving it for merge.

Copy link
Copy Markdown
Contributor

@bernhardkaindl bernhardkaindl left a comment

Choose a reason for hiding this comment

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

Thanks @amd-toolchain-support, will merge.

@bernhardkaindl bernhardkaindl merged commit b8a64c3 into spack:develop Aug 22, 2025
16 checks passed
paulmelis pushed a commit to paulmelis/spack-packages that referenced this pull request Aug 26, 2025
flang can't find gcc if gcc is installed in a non-standard location.

This commit:

- Patches flang1 & flang2 with the rpath for libquadmath.
- Creates a wrapper script to pass options to flang, since flang.cfg
  is currently ignored.
- Passes `--gcc-toolchain`.
- Passes `-Wl,-rpath` so executables can find libquadmath.
- Passes `-Wno-unused-command-line-argument` to silence warnings
  generated by `-Wl,-rpath`, since they can cause problems.
mladenivkovic pushed a commit to mladenivkovic/spack-packages that referenced this pull request Aug 31, 2025
flang can't find gcc if gcc is installed in a non-standard location.

This commit:

- Patches flang1 & flang2 with the rpath for libquadmath.
- Creates a wrapper script to pass options to flang, since flang.cfg
  is currently ignored.
- Passes `--gcc-toolchain`.
- Passes `-Wl,-rpath` so executables can find libquadmath.
- Passes `-Wno-unused-command-line-argument` to silence warnings
  generated by `-Wl,-rpath`, since they can cause problems.
@tldahlgren tldahlgren removed the waiting-on-maintainer Waiting on a review from the package maintainer label Jan 9, 2026
climbfuji added a commit to climbfuji/spack-packages that referenced this pull request May 7, 2026
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