[Openmp-dev] r239337 - Remove unused variables '__kmp_build_check_*'

Jack Howarth howarth.mailing.lists at gmail.com
Mon Jun 8 16:54:33 PDT 2015


Jonathan,
     The commit...

http://lists.cs.uiuc.edu/pipermail/openmp-commits/2015-June/000339.html

only converts the unused-variable warnings for __kmp_build_check_*
into unused typedef warnings. My initial inclination was to test
adding...

#ifdef KMP_USE_ASSERT
...
#endif

preprocessor wrappers around all of the calls to the KMP_BUILD_ASSERT
macro. However this doesn't work since the openmp build passes  -D
KMP_USE_ASSERT to the compiler flags in the cmake build of openmp in
the llvm tree. This occurs for a cmake build which pass passed
-DLLVM_ENABLE_ASSERTIONS=OFF for the top-level cmake options.
Shouldn't KMP_USE_ASSERT be disabled in that case or are those two
usages of asserts unrelated?
              Jack



More information about the Openmp-dev mailing list