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

Add support for Intel LLVM-based compilers #2361

Closed
uturuncoglu opened this issue Jul 9, 2024 · 3 comments · Fixed by NOAA-EMC/fv3atm#861
Closed

Add support for Intel LLVM-based compilers #2361

uturuncoglu opened this issue Jul 9, 2024 · 3 comments · Fixed by NOAA-EMC/fv3atm#861
Labels
enhancement New feature or request

Comments

@uturuncoglu
Copy link
Collaborator

Description

I was experimenting UFS Weather Model under TACC Frontera as a part of UFS coastal project. I could able to run model configurations using spack-stack 1.5.1 along with intel/19.1.1 and impi/19.0.9 but when I try to upgrade the compiler and MPI to their newer versions (intel/23.1.0 and impi/21.9.0 - intel-oneapi), I started to get following error,

CMake Error at CMakeLists.txt:149 (include):
  include could not find requested file:

    cmake/IntelLLVM.cmake

I think that the reason is the Intel compiler. In my case, ${CMAKE_Fortran_COMPILER_ID} is returning IntelLLVM and model build is failing since there is no cmake/IntelLLVM.cmake (

include(cmake/${CMAKE_Fortran_COMPILER_ID}.cmake)
). I am seeing usage of IntelLLVM along with Intel in some places of the code but it is not a standard and used in everywhere.

Solution

I think, the top level CMake file as well as CMake interfaces of the components needs to be extended to allow IntelLLVM.

Alternatives

Not sure

Related to

N/A

@uturuncoglu uturuncoglu added the enhancement New feature or request label Jul 9, 2024
@DusanJovic-NOAA
Copy link
Collaborator

See #2224

@uturuncoglu
Copy link
Collaborator Author

@DusanJovic-NOAA Thanks. I was not aware of it. Do you want me to close this issue? or you prefer to keep it open.

@DusanJovic-NOAA
Copy link
Collaborator

@DusanJovic-NOAA Thanks. I was not aware of it. Do you want me to close this issue? or you prefer to keep it open.

Keep it open. We can use this issue to track the progress.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants