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

Remove workaround to PGI compiler bug #327

Open
wants to merge 2 commits into
base: development
Choose a base branch
from

Conversation

mnlevy1981
Copy link
Collaborator

As of 18.10, put_input_file_line() does not need the pgi_bugfix_var optional
argument to be recognized as a unique interface for put_setting(). Once this is
accepted onto development, PGI 18.4 (and earlier) will not be able to compile
MARBL. It will fail to build the stand-alone driver with the error

PGF90-S-0155-cannot access PRIVATE type bound procedure put_input_file_line

(Presumably GCMs that use put_setting() will also run into this error)

As of 18.10, put_input_file_line() does not need the pgi_bugfix_var optional
argument to be recognized as a unique interface for put_setting(). Once this is
accepted onto development, PGI 18.4 (and earlier) will not be able to compile
MARBL. It will fail to build the stand-alone driver with the error

PGF90-S-0155-cannot access PRIVATE type bound procedure put_input_file_line

(Presumably GCMs that use put_setting() will also run into this error)
@mnlevy1981 mnlevy1981 changed the base branch from stable to development November 15, 2018 23:43
@mnlevy1981
Copy link
Collaborator Author

I've asked ISG to install the newest PGI compiler on hobart, at which point I'll make a CIME pull request to update the version used by CESM... after CESM is building with 18.10, we can bring this onto development and stable (assuming the CIME change goes into both CESM 2.1 and 2.2)

Without the pgi bug workaround, the stand-alone test suite needs to
build with the version 18.10 of the compiler.
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

Successfully merging this pull request may close these issues.

1 participant