[PATCH] D13802: [OPENMP] Make -fopenmp to turn on OpenMP support by default.
Alexey Bataev via cfe-commits
cfe-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 cfe-commits
mailing list