Remove const from variables with dynamic memory
authorJonas Hahnfeld <hahnjo@hahnjo.de>
Thu, 9 Nov 2017 15:52:25 +0000 (15:52 +0000)
committerJonas Hahnfeld <hahnjo@hahnjo.de>
Thu, 9 Nov 2017 15:52:25 +0000 (15:52 +0000)
commitaeb40adabf18d517fa27085d3e39cb2b93eb7648
treee4c1bb4635181b54f8b07352cccdc8678aaa70f2
parent18ff303bed72782f05cfc9b87ed38d788edbdf07
Remove const from variables with dynamic memory

Allocated memory is typically not 'const' if it needs to be freed.
This patch removes around 50 wrong const attributes, modifies the
corresponding functions and finally gets rid of some const_casts.
These have especially been strange for __kmp_str_fname_free() that
added a 'const' to call __kmp_str_free() which removed it again.

Two minor cleanups that I performed in this process:
 * __kmp_tool_libraries now lives in kmp_settings.cpp as it is
   used nowhere else.
 * __kmp_msg_empty was removed as it was never used and Clang
   now complained that it was assigned a string literal that
   is 'const char *'.

Differential Revision: https://reviews.llvm.org/D39755

llvm-svn: 317797
13 files changed:
openmp/runtime/src/kmp.h
openmp/runtime/src/kmp_dispatch.cpp
openmp/runtime/src/kmp_environment.h
openmp/runtime/src/kmp_error.cpp
openmp/runtime/src/kmp_global.cpp
openmp/runtime/src/kmp_i18n.cpp
openmp/runtime/src/kmp_i18n.h
openmp/runtime/src/kmp_itt.inl
openmp/runtime/src/kmp_sched.cpp
openmp/runtime/src/kmp_settings.cpp
openmp/runtime/src/kmp_str.cpp
openmp/runtime/src/kmp_str.h
openmp/runtime/src/ompt-general.cpp