Upgrade default PGI from 15.7 to 17.5 on Titan #1619
Conversation
The results of the acme developer test suite using PGI/17.5 on Titan are exactly same as those reported in cdash using PGI/15.7. So the default PGI version was upgraded from 15.7 to 17.5 on Titan. [Non-BFB]
|
@minxu74 and @mrnorman , I can understand changing the defaults to one of Dave Norton's local installations for pgiacc, but this is not officially supported by the OLCF. Will pgi/17.5.0 be an officially installed module "soon"? Can we wait until then? Pointing at Dave Norton's space has been fragile in the past. |
|
Given that pgi/17.3 is already installed, I don't think it will be long, no. Plus, I've explicitly requested that it be installed on Titan. I agree that relying on Dave's installs can be fragile. We can wait to make it default for "pgi" until it's officially installed by OLCF if you want, but "pgiacc" should be set to 17.5 at least. |
|
If it is imminent and if pgi/17.5.0 does not require commenting out the intent(in) on the pointer variables in the atmospheric physics, I'm fine with this PR. Will need to update it as soon as the official OLCF release is out. |
|
Pat, that's a good point. 17.5 isn't supposed to require commenting out the intent(in) for those pointers. And 17.5 should come pretty quick. |
|
@worleyph and @mrnorman Thank you both for the useful discussions. Since it is necessary to upgrade to pgi/17.5 for PGIACC and the upgrade can also avoid changing source codes (commenting out the intent), I am going to merge the branch to master now. I will update it once the official version is released by OLCF. |
The results of the acme developer test suite using PGI/17.5 on Titan are exactly same as those reported in cdash using PGI/15.7. So the default PGI version was upgraded from 15.7 to 17.5 on Titan. It is necessary to upgrade to pgi/17.5 for PGIACC and moreover the upgrade can also avoid changing source codes (commenting out the intent) required by the pgi version 16 and early versions of 17. [Non-BFB] * minxu74/titan/upgrade-default-pgi: Upgrade default PGI from 15.7 to 17.5 on Titan
|
@minxu74 did you merge this to next? |
|
@rljacob Yes. it was in next. |
|
I see it now. Don't forget to add the PR # to the title. Then it will show up in the history here. |
The results of the acme developer test suite using PGI/17.5 on Titan
are exactly same as those reported in cdash using PGI/15.7. So the
default PGI version was upgraded from 15.7 to 17.5 on Titan.
[Non-BFB]