Skip to content

Fixes in routing model for gcc 15 - #1479

Merged
mathomp4 merged 4 commits into
developfrom
bugfix/rreichle/routing_gcc15
Jul 16, 2026
Merged

Fixes in routing model for gcc 15#1479
mathomp4 merged 4 commits into
developfrom
bugfix/rreichle/routing_gcc15

Conversation

@gmao-rreichle

@gmao-rreichle gmao-rreichle commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Fixes for routing model needed by GNU compiler gcc15.

  • Remove GNU optimization override for routing_model.F90 (no longer needed with gcc15 and/or the present fix).
  • Avoid calculating 0 to the power of something (which trips up gcc15).

Trivially zero-diff for GCM simulations (b/c routing model is not yet used in GCM tests)

Zero-diff for GEOSldas when using Intel compiler (confirmed by @gmao-rreichle on 16 July 2027; this test evaluated the changes in the present PR in isolation, i.e., without GEOS-ESM/GEOSldas#864; adding GEOSldas PR#864 is not zero-diff).

For GNU, testing was only done in conjunction with the changes in GEOS-ESM/GEOSldas#864, with the expected non-zero diff ("roundoff") result.


Related PRs:
GEOS-ESM/GEOSldas#864


cc: @biljanaorescanin @weiyuan-jiang

@gmao-rreichle gmao-rreichle added 0 diff The changes in this pull request have verified to be zero-diff with the target branch. 0 diff trivial The changes in this pull request are trivially zero-diff (documentation, build failure, &c.) labels Jul 15, 2026
@gmao-rreichle
gmao-rreichle marked this pull request as ready for review July 16, 2026 17:18
@gmao-rreichle
gmao-rreichle requested review from a team as code owners July 16, 2026 17:18
@mathomp4
mathomp4 self-requested a review July 16, 2026 17:33
@mathomp4
mathomp4 merged commit aff169b into develop Jul 16, 2026
15 checks passed
@gmao-rreichle
gmao-rreichle deleted the bugfix/rreichle/routing_gcc15 branch July 16, 2026 17:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

0 diff trivial The changes in this pull request are trivially zero-diff (documentation, build failure, &c.) 0 diff The changes in this pull request have verified to be zero-diff with the target branch.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants