Skip to content

Commit 78fa980

Browse files
author
Damian Rouson
authored
Merge pull request #706 from lettleli/fix-704-cmake-mpi-compile-flags
Fix MPI compile flags to not include quotation marks (issue-704)
2 parents 5691e7b + 0aff081 commit 78fa980

File tree

1 file changed

+9
-3
lines changed

1 file changed

+9
-3
lines changed

src/mpi/CMakeLists.txt

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,9 @@ set_target_properties(opencoarrays_mod
2222
Fortran_MODULE_DIRECTORY "${CMAKE_BINARY_DIR}/${CMAKE_INSTALL_INCLUDEDIR}/${mod_dir_tail}"
2323
POSITION_INDEPENDENT_CODE TRUE)
2424
target_compile_options(opencoarrays_mod
25-
PRIVATE $<$<COMPILE_LANGUAGE:Fortran>:${MPI_Fortran_COMPILE_FLAGS}>)
25+
PRIVATE $<$<COMPILE_LANGUAGE:Fortran>:${MPI_Fortran_COMPILE_OPTIONS}>)
26+
target_compile_definitions(opencoarrays_mod
27+
PRIVATE $<$<COMPILE_LANGUAGE:Fortran>:${MPI_Fortran_COMPILE_DEFINITIONS}>)
2628
target_link_libraries(opencoarrays_mod
2729
PUBLIC ${MPI_Fortran_LINK_FLAGS}
2830
PUBLIC ${MPI_Fortran_LIBRARIES})
@@ -53,9 +55,13 @@ target_include_directories(caf_mpi_static PUBLIC
5355
$<BUILD_INTERFACE:${CMAKE_SOURCE_DIR}/src>
5456
$<INSTALL_INTERFACE:${CMAKE_INSTALL_INCLUDEDIR}>)
5557
target_compile_options(caf_mpi
56-
PUBLIC $<$<COMPILE_LANGUAGE:C>:${MPI_C_COMPILE_FLAGS}>)
58+
PUBLIC $<$<COMPILE_LANGUAGE:C>:${MPI_C_COMPILE_OPTIONS}>)
59+
target_compile_definitions(caf_mpi
60+
PUBLIC $<$<COMPILE_LANGUAGE:C>:${MPI_C_COMPILE_DEFINITIONS}>)
5761
target_compile_options(caf_mpi_static
58-
PUBLIC $<$<COMPILE_LANGUAGE:C>:${MPI_C_COMPILE_FLAGS}>)
62+
PUBLIC $<$<COMPILE_LANGUAGE:C>:${MPI_C_COMPILE_OPTIONS}>)
63+
target_compile_definitions(caf_mpi_static
64+
PUBLIC $<$<COMPILE_LANGUAGE:C>:${MPI_C_COMPILE_DEFINITIONS}>)
5965

6066
set(CAF_SO_VERSION 0)
6167
if(gfortran_compiler)

0 commit comments

Comments
 (0)