diff options
| author | Jonathan Peyton <jonathan.l.peyton@intel.com> | 2015-06-11 17:36:16 +0000 |
|---|---|---|
| committer | Jonathan Peyton <jonathan.l.peyton@intel.com> | 2015-06-11 17:36:16 +0000 |
| commit | 9919dfc3b62cb5e51f32e3ac1e5fc8941e80dda1 (patch) | |
| tree | 14b1ae51283840c4548689a557536918d778c5a3 /openmp/runtime/cmake | |
| parent | 2d221406fa5967b663993f55ba7262f78fce061a (diff) | |
| download | bcm5719-llvm-9919dfc3b62cb5e51f32e3ac1e5fc8941e80dda1.tar.gz bcm5719-llvm-9919dfc3b62cb5e51f32e3ac1e5fc8941e80dda1.zip | |
Remove unused variables '__kmp_build_check_*' for non assert builds.
Add new LIBOMP_ENABLE_ASSERTIONS macro which can be set in a standalone build
or takes the value of LLVM_ENABLE_ASSERTIONS when inside llvm/projects. This
change also defines the KMP_BUILD_ASSERT() macro to do nothing when ENABLE_ASSERTIONS
is off. This means the __kmp_build_check_* types won't be defined and thus, no warnings.
http://lists.cs.uiuc.edu/pipermail/openmp-dev/2015-June/000719.html
Patch by Jack Howarth and Jonathan Peyton
llvm-svn: 239546
Diffstat (limited to 'openmp/runtime/cmake')
| -rw-r--r-- | openmp/runtime/cmake/Definitions.cmake | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/openmp/runtime/cmake/Definitions.cmake b/openmp/runtime/cmake/Definitions.cmake index b797e04a7d0..fe367d3520e 100644 --- a/openmp/runtime/cmake/Definitions.cmake +++ b/openmp/runtime/cmake/Definitions.cmake @@ -83,7 +83,9 @@ function(append_cpp_flags input_cpp_flags) ################################## # Other conditional definitions - append_definitions("-D KMP_USE_ASSERT") + if(${LIBOMP_ENABLE_ASSERTIONS}) + append_definitions("-D KMP_USE_ASSERT") + endif() append_definitions("-D KMP_DYNAMIC_LIB") if(${STUBS_LIBRARY}) append_definitions("-D KMP_STUB") |
