Skip to content

Conversation

@Leon-Degel-Koehn
Copy link

Provides the necessary changes to use LFortran as the compiler for this project to the extend that it currently can. Please note the corresponding issue to gain further information on how to use this. It provides a compilation script and references the required mpi wrapper library. #98

Provides the necessary changes to use LFortran as the compiler for
this project to the extend that it currently can. Please note the
corresponding issue to gain further information on how to use this.
It provides a compilation script and references the required mpi
wrapper library.
CACHE STRING "")
INCLUDE_DIRECTORIES(${TIMING_INCLUDE_DIRS})

ADD_DEFINITIONS(-DINCLUDE_CMAKE_FCI -DHAVE_MPI)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it necessary to remove -DHAVE_MPI to make LFortran work? This option might be used by other compilers.

set(CMAKE_Fortran_FLAGS_RELEASE "-O2 -debug minimal")
elseif(COMPILER STREQUAL "LFortran")
add_compile_definitions(CPRLFORTRAN)
# There is an issue, some modules require --cpp as a flag in their compilation
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please add a comment that shows the link to #98

#endif
#if (defined AIX)
call flush_(unit)
#endif
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I suggest keeping AIX if it's not required to make LFortran work

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.

2 participants