[Openmp-commits] [PATCH] D49105: [OpenMP] Cleanup code

Jonathan Peyton via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Mon Jul 9 15:54:38 PDT 2018


jlpeyton created this revision.
jlpeyton added reviewers: tlwilmar, AndreyChurbanov.
jlpeyton added a project: OpenMP.
Herald added subscribers: guansong, mgorny.

This patch cleans up unused functions, variables, sign compare issues, and addresses some -Warning flags which are now enabled including `-Wcast-qual`.  Not all the warning flags in `LibompHandleFlags.cmake` are enabled, but some are with this patch.

Some `__kmp_gtid_from_*` macros in `kmp.h` are switched to static inline functions which allows us to remove the awkward definition of `KMP_DEBUG_ASSERT()` and `KMP_ASSERT()` macros which used the comma operator.  This had to be done for the innumerable `-Wunused-value` warnings related to `KMP_DEBUG_ASSERT()` (at least for gcc)


Repository:
  rOMP OpenMP

https://reviews.llvm.org/D49105

Files:
  runtime/cmake/LibompHandleFlags.cmake
  runtime/cmake/config-ix.cmake
  runtime/src/kmp.h
  runtime/src/kmp_affinity.cpp
  runtime/src/kmp_affinity.h
  runtime/src/kmp_alloc.cpp
  runtime/src/kmp_atomic.cpp
  runtime/src/kmp_barrier.cpp
  runtime/src/kmp_csupport.cpp
  runtime/src/kmp_debug.h
  runtime/src/kmp_dispatch.cpp
  runtime/src/kmp_dispatch.h
  runtime/src/kmp_gsupport.cpp
  runtime/src/kmp_i18n.cpp
  runtime/src/kmp_lock.cpp
  runtime/src/kmp_lock.h
  runtime/src/kmp_runtime.cpp
  runtime/src/kmp_sched.cpp
  runtime/src/kmp_settings.cpp
  runtime/src/kmp_stats.cpp
  runtime/src/kmp_stats.h
  runtime/src/kmp_stub.cpp
  runtime/src/kmp_tasking.cpp
  runtime/src/kmp_wait_release.h
  runtime/src/ompt-general.cpp
  runtime/src/ompt-specific.h
  runtime/src/thirdparty/ittnotify/ittnotify_static.c

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D49105.154721.patch
Type: text/x-patch
Size: 58750 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/openmp-commits/attachments/20180709/10de8444/attachment-0001.bin>


More information about the Openmp-commits mailing list