-
Notifications
You must be signed in to change notification settings - Fork 0
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
Use oneapi compiler #41
Comments
No, it doesn't. |
Yes, this matches correctly. |
Just seen this new issue, great to see some of the things we discussed here, already being addressed! |
I had a quick look through the components, the only place I found this needed setting is in I can make a PR to add '-fp-model precise' to the esmf build when %oneapi (The flag name is unchanged according to oneAPI Programming Guide) I looked in the fms and parallelio packages and there was nothing set specific for %intel, so I think we are in the clear otherwise ? |
setting -fp-model=precise in ACCESS-NRI/spack-packages#203 |
Compiling FMS with %oneapi gives an internal compiler error:
As reported here: This should have been fixed in [email protected] (so don't use %[email protected]) |
For ongoing support and better optimisation on new hardware, we need to move to using the Intel Oneapi / LLVM compilers instead of the current Intel Classic compilers. Initial discussion is here
We expect this change to not be bit for bit.
To-do:
https://github.com/ACCESS-NRI/spack-packages/blob/288ef380159af4536137b40360b2a87c043b7ce1/packages/access-om3-nuopc/package.py#L65
It doesn't really matter if we do this before or after #34 / #35 which uses the new build system (we just need to make sure CMake and spack changes, if required are done in all the related repositories / packages.
I suggest we do this seperately from code changes (e.g. 2025.01.0 could be build with intel-classic and 2025.01.1 could be oneapi with the same source code). This makes it easier to examine any differences in the results.
I may have added extra steps here - let me know if you have done some of these already @micaeljtoliveira
The text was updated successfully, but these errors were encountered: