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

Floating point exceptions when using -fpe0 #194

Closed
nichannah opened this issue Jul 9, 2015 · 3 comments
Closed

Floating point exceptions when using -fpe0 #194

nichannah opened this issue Jul 9, 2015 · 3 comments

Comments

@nichannah
Copy link
Collaborator

When the model is built with -fpe0 (intel) or ffpe-trap=invalid,zero,overflow (gnu) a couple of FPEs are triggered. This issue will fix those.

In addition, should we consider making these options standard? I can't find any mention of a down-side on intel/amd procs.

@jwdGFDL
Copy link

jwdGFDL commented Jul 9, 2015

In general, setting FPE exception handling will slow performance. We
haven't tested it recently. But historically the performance loss is
quite signficant.

On 07/09/2015 10:23 AM, Nic Hannah wrote:

When the model is built with -fpe0 (intel) or
ffpe-trap=invalid,zero,overflow (gnu) a couple of FPEs are triggered.
This issue will fix those.

In addition, should we consider making these options standard? I can't
find any mention of a down-side on intel/amd procs.


Reply to this email directly or view it on GitHub
NOAA-GFDL#194.

Jeff Durachta
Engineering Lead for Modeling Services
NOAA Geophysical Fluid Dynamics Lab
Forrestal Campus, Princeton University
201 Forrestal Road
Princeton, NJ 08540
Office: +1-609-987-5054

nichannah added a commit to nichannah/MOM6 that referenced this issue Jul 9, 2015
nichannah added a commit to nichannah/MOM6 that referenced this issue Jul 9, 2015
@adcroft
Copy link
Collaborator

adcroft commented Jul 9, 2015

I think we could (should?) use these copmiler options in the DEBUG=1 mode. Right now, the FFLAGS_DEBUG in ncrc-gnu.mk just turns off optimization and turns on bounds checking. I agree with @mom6jwd that we would not worry about catching these exceptions for production executables.

@nichannah
Copy link
Collaborator Author

I have created an mkmf issue NOAA-GFDL/mkmf#6 to add -fpe0 for debug builds.

nichannah added a commit to nichannah/MOM6 that referenced this issue Jul 17, 2015
Avoids touching halos which may not be initialized. mom-ocean#194
nichannah added a commit to nichannah/MOM6 that referenced this issue Jul 17, 2015
Avoids touching halos which may not be initialized. mom-ocean#194
nichannah added a commit to nichannah/MOM6 that referenced this issue Jul 17, 2015
Avoids touching halos which may not be initialized. mom-ocean#194
gustavo-marques added a commit to gustavo-marques/MOM6 that referenced this issue Aug 30, 2021
Introduce the legacy CESM WW3 coupling option (VR12-MA)
Hallberg-NOAA pushed a commit to marshallward/MOM6 that referenced this issue Aug 29, 2022
* Fix nudged OBCs for tracers
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

No branches or pull requests

3 participants