search

MCSclimate / MCT

Model Coupling Tookit
Other
43 stars 18 forks source link

Support new Intel GPU-enabled compilers #61

Closed naromero77 closed 3 years ago

naromero77 commented 4 years ago

Strips out -loopopt=0 flag in main configure and mpi-serial configure. configure will not complete without this change.

naromero77 commented 4 years ago

@rljacob Is there some CI enable for this repo? I know the E3SM project is very good about testing, and I guess I was expecting my PR to trigger some kind of testing automatically that would be shown as part of this PR.

rljacob commented 4 years ago

No this repo does not have CI. It gets so few PRs that I haven't tried to set it up.

naromero77 commented 4 years ago

Let me know what needs to get done in order to merge the change.

rljacob commented 4 years ago

@naromero77 I'm working on this. Although we include configure in the repo, that is for convenience because it is a generated file made by autoconf from configure.in. But I've forgotten all my autoconf so don't know how to make this change the "right" way.

Also mpi-serial is included in MCT as a subtree. So can you make a separate PR for that change to https://github.com/MCSclimate/mpi-serial ? Then close this PR and make a new one the only changes MCT. I can do all that if you're busy.

naromero77 commented 4 years ago

I am busy today, but can do that next week.

naromero77 commented 3 years ago

This PR now only contains the commit for the MPI code.

naromero77 commented 3 years ago

Here is the second PR: https://github.com/MCSclimate/mpi-serial/pull/15

rljacob commented 3 years ago

Ok I'm on iris with oneAPI. What is your normal configure command?

naromero77 commented 3 years ago

Here is how you load the module:

module use /soft/restricted/CNDA/modulefiles
module load  oneapi

Here is how I run configure:

./configure FC=ifx CC=icx
rljacob commented 3 years ago

Replaced by #66