[Openmp-commits] [openmp] e27ce28 - [OpenMP][libomp] Make LIBOMP_CONFIGURED_LIBFLAGS a list instead of string

Jonathan Peyton via Openmp-commits openmp-commits at lists.llvm.org
Thu Jun 2 08:51:10 PDT 2022


Author: Mikael Simberg
Date: 2022-06-02T10:50:21-05:00
New Revision: e27ce281399dca8b08b6ca593172a1bd5dbdd5c1

URL: https://github.com/llvm/llvm-project/commit/e27ce281399dca8b08b6ca593172a1bd5dbdd5c1
DIFF: https://github.com/llvm/llvm-project/commit/e27ce281399dca8b08b6ca593172a1bd5dbdd5c1.diff

LOG: [OpenMP][libomp] Make LIBOMP_CONFIGURED_LIBFLAGS a list instead of string

When configuring llvm with the openmp subproject, the build for the omp
target fails if LIBOMP_CONFIGURED_LIBFLAGS contains more than one item.
LIBOMP_CONFIGURED_LIBFLAGS should be a semicolon-separated list instead
of a string with items separated by spaces.

Differential Revision: https://reviews.llvm.org/D125370

Added: 
    

Modified: 
    openmp/runtime/cmake/LibompHandleFlags.cmake

Removed: 
    


################################################################################
diff  --git a/openmp/runtime/cmake/LibompHandleFlags.cmake b/openmp/runtime/cmake/LibompHandleFlags.cmake
index d725ece02be42..6643ea392f80a 100644
--- a/openmp/runtime/cmake/LibompHandleFlags.cmake
+++ b/openmp/runtime/cmake/LibompHandleFlags.cmake
@@ -144,7 +144,8 @@ function(libomp_get_libflags libflags)
   endif()
   set(libflags_local ${libflags_local} ${LIBOMP_LIBFLAGS})
   libomp_setup_flags(libflags_local)
-  set(${libflags} ${libflags_local} PARENT_SCOPE)
+  libomp_string_to_list("${libflags_local}" libflags_local_list)
+  set(${libflags} ${libflags_local_list} PARENT_SCOPE)
 endfunction()
 
 # Fortran flags


        


More information about the Openmp-commits mailing list