Skip to content

v2: Fix issue with NVHPC and halting#4110

Merged
mathomp4 merged 2 commits intodevelopfrom
bugfix/nvhpc-ieee
Oct 9, 2025
Merged

v2: Fix issue with NVHPC and halting#4110
mathomp4 merged 2 commits intodevelopfrom
bugfix/nvhpc-ieee

Conversation

@mathomp4
Copy link
Member

@mathomp4 mathomp4 commented Oct 7, 2025

Types of change(s)

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Trivial change (affects only documentation or cleanup)
  • Refactor (no functional changes, no api changes)

Checklist

  • Tested this change with a run of GEOSgcm
  • Ran the Unit Tests (make tests)

Description

In testing with NVHPC 25.9, an issue was found with the set-at-compile-time of set_halting_allowed from #4007. The simple fix is to set set_halting_allowed at run-time.

Related Issue

https://forums.developer.nvidia.com/t/nvfortran-fails-on-valid-fortran/346679/1

@mathomp4 mathomp4 requested a review from a team as a code owner October 7, 2025 12:13
@mathomp4 mathomp4 added 0 Diff The changes in this pull request have verified to be zero-diff with the target branch. 🪲 Bugfix This fixes a bug! 😖 Shutdown This occurred during shutdown labels Oct 7, 2025
@mathomp4 mathomp4 self-assigned this Oct 7, 2025
@mathomp4 mathomp4 merged commit e7facee into develop Oct 9, 2025
39 checks passed
@mathomp4 mathomp4 deleted the bugfix/nvhpc-ieee branch October 9, 2025 14:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

0 Diff The changes in this pull request have verified to be zero-diff with the target branch. 🪲 Bugfix This fixes a bug! 😖 Shutdown This occurred during shutdown

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants