[PATCH] D13802: [OPENMP] Make -fopenmp to turn on OpenMP support by default.

Alexey Bataev via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 28 11:37:11 PDT 2015


ABataev added a comment.

In http://reviews.llvm.org/D13802#277270, @jhowarth wrote:

> The proposed patches here to change the default behavior of -fopenmp from -fopenmp=libgomp to -fopenmp=libomp seem to only handle the configure-based build. The following change required to switch over the crake-based build to default -fopenmp to libomp  is missing.
>
>   Index: CMakeLists.txt
>   ===================================================================
>   --- CMakeLists.txt	(revision 251539)
>   +++ CMakeLists.txt	(working copy)
>   @@ -196,7 +196,7 @@ set(GCC_INSTALL_PREFIX "" CACHE PATH "Di
>    set(DEFAULT_SYSROOT "" CACHE PATH
>      "Default <path> to all compiler invocations for --sysroot=<path>." )
>   
>   -set(CLANG_DEFAULT_OPENMP_RUNTIME "libgomp" CACHE STRING
>   +set(CLANG_DEFAULT_OPENMP_RUNTIME "libomp" CACHE STRING
>      "Default OpenMP runtime used by -fopenmp.")
>   
>    set(CLANG_VENDOR "" CACHE STRING
>


This patch is for clang and it is accepted already. But I'm going to land all patches for -fopenmp only when llvm and libomp changes are accepted.


http://reviews.llvm.org/D13802





More information about the llvm-commits mailing list