-
Notifications
You must be signed in to change notification settings - Fork 9
Open
Labels
Description
I marked it as a bug, but it's more like a conceptual change. EKAT should not take care of enabling any FPEs when the ekat session is started. There are issues for downstream apps, in some cases:
- Case 1: the downstream app wants different FPEs: it has to disable all fpes, and enable the desired ones.
- Case 2: the downstream app does not want fpes, but needs to disable FPEs set by an even more upstream app.
Proposal: remove "default" fpes from ekat. We can still have them in the ekat_test_session minilib, but not in the main ekat library. The cmake option EKAT_ENABLE_FPE
is then simply enabling the "FPE machinery", which allows one to call ekat::enable_fpes(mask)
and the likes.