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

Uses the Fortran linker for the Cray compiler on Frontier #6367

Merged
merged 1 commit into from
Apr 24, 2024

Conversation

grnydawn
Copy link
Contributor

@grnydawn grnydawn commented Apr 24, 2024

  • add "set(E3SM_LINK_WITH_FORTRAN "TRUE")" in crayclang_frontier.cmake

Some ELM external source files (such as sbetr/src/betr/betr_rxns/Tracer1beckBGCReactionsType.F90) use Fortran-compiler-specific intrinsic functions such as 'erfc', which require linking with the Fortran compiler.

[No baseline for Frontier yet]

@grnydawn grnydawn requested a review from jgfouca April 24, 2024 17:16
@grnydawn grnydawn self-assigned this Apr 24, 2024
@grnydawn grnydawn added Cray Cray compiler related issues Frontier labels Apr 24, 2024
grnydawn added a commit that referenced this pull request Apr 24, 2024
Uses the Fortran linker for the Cray compiler on Frontier

* add "set(E3SM_LINK_WITH_FORTRAN "TRUE")" in crayclang_frontier.cmake

Some ELM external source files (such as sbetr/src/betr/betr_rxns/Tracer1beckBGCReactionsType.F90) use Fortran-compiler-specific intrinsic functions such as 'erfc', which require linking with the Fortran compiler.

[No baseline for Frontier yet]
Copy link

PR Preview Action v1.4.7
🚀 Deployed preview to https://E3SM-Project.github.io/E3SM/pr-preview/pr-6367/
on branch gh-pages at 2024-04-24 20:37 UTC

@grnydawn grnydawn merged commit 3f4b72c into master Apr 24, 2024
21 checks passed
@grnydawn grnydawn deleted the ykim/frontier/crayclang-update branch April 24, 2024 23:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Cray Cray compiler related issues Frontier
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants