[Openmp-commits] [openmp] r239912 - Have CMake error out on Windows when user wants OMPT support

[Jonathan Peyton]

Author: jlpeyton
Date: Wed Jun 17 10:43:34 2015
New Revision: 239912

URL: http://llvm.org/viewvc/llvm-project?rev=239912&view=rev
Log:
Have CMake error out on Windows when user wants OMPT support

Currently, OMPT support requires the weak attribute which isn't supported
on Windows.  This patch has CMake error out when LIBOMP_OMPT_SUPPORT=true
and the users is building on Windows.
http://lists.cs.uiuc.edu/pipermail/openmp-dev/2015-June/000692.html

Patch by Jonas Hahnfeld

Modified:
    openmp/trunk/runtime/Build_With_CMake.txt
    openmp/trunk/runtime/CMakeLists.txt

Modified: openmp/trunk/runtime/Build_With_CMake.txt
URL: http://llvm.org/viewvc/llvm-project/openmp/trunk/runtime/Build_With_CMake.txt?rev=239912&r1=239911&r2=239912&view=diff
==============================================================================
--- openmp/trunk/runtime/Build_With_CMake.txt (original)
+++ openmp/trunk/runtime/Build_With_CMake.txt Wed Jun 17 10:43:34 2015
@@ -139,14 +139,14 @@ Should the Fortran modules be created (r
 Should include stats-gathering code?
 
 -DLIBOMP_OMPT_SUPPORT=off|on
-Should OMPT support be turned on?
-If ompt_support is off, then both ompt_blame and ompt_trace are ignored.
+Should OMPT support be turned on? (Not supported on Windows)
+If LIBOMP_OMPT_SUPPORT is off, then both ompt_blame and ompt_trace are ignored.
 
 -DLIBOMP_OMPT_BLAME=on|off
-should OMPT blame functionality be turned on?
+Should OMPT blame functionality be turned on?
 
 -DLIBOMP_OMPT_TRACE=on|off
-should OMPT trace functionality be turned on?
+Should OMPT trace functionality be turned on?
 
 =====================
 ==== Micro tests ====

Modified: openmp/trunk/runtime/CMakeLists.txt
URL: http://llvm.org/viewvc/llvm-project/openmp/trunk/runtime/CMakeLists.txt?rev=239912&r1=239911&r2=239912&view=diff
==============================================================================
--- openmp/trunk/runtime/CMakeLists.txt (original)
+++ openmp/trunk/runtime/CMakeLists.txt Wed Jun 17 10:43:34 2015
@@ -333,6 +333,9 @@ endif()
 if(${LIBOMP_USE_ADAPTIVE_LOCKS} AND NOT(${IA32} OR ${INTEL64}))
     error_say("Adaptive locks (TSX) functionality is only supported on x86 Architecture")
 endif()
+if(${LIBOMP_OMPT_SUPPORT} AND ${WINDOWS})
+    error_say("OpenMP Tools Interface is not supported on Windows")
+endif()
 
 ###############################################
 # - Create the suffix for the export directory