Adding front-end option to compile Isca using single-precision (only working with Intel compilers)#265
Open
sit23 wants to merge 5 commits intoExeClim:masterfrom
Open
Adding front-end option to compile Isca using single-precision (only working with Intel compilers)#265sit23 wants to merge 5 commits intoExeClim:masterfrom
sit23 wants to merge 5 commits intoExeClim:masterfrom
Conversation
…ses a combination of Iscas in-built macros with overload_c4 etc, whilst maintaining other default flags.
…ountered with the simple cloud fortran files, which had missed a few file-exists imports from fms-mod. Also lcl.f90 explicitly defined variables as double precision, which caused problems when variables were inputted to its functions as single. Now changed to real so that compiler chooses the representation of real.
… precision, but should not be there for single precision. Do not currently have a dynamic way of editing that mkmf template file, so added feature to choose different mkmf template file for single precision. Format is just same name as current mkmf template file with _simple added at the end.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
We want to be able to run Isca using single-precision arithmetic, rather than the default double precision. This p/r mirrors some code I wrote way back in 2020:
sit23@efd0195
This change gives a method on the codebase object to allow the compile flags to be set to do single precision arithmetic, rather then double, but leaves double as the default.
I've added a new held-suarez test case to show how this method is to be used.
N.B. this only works at the moment for Intel compilation.