[Openmp-commits] [openmp] r368559 - Cleanup unused variable.
Andrey Churbanov via Openmp-commits
openmp-commits at lists.llvm.org
Mon Aug 12 05:37:30 PDT 2019
Author: achurbanov
Date: Mon Aug 12 05:37:30 2019
New Revision: 368559
URL: http://llvm.org/viewvc/llvm-project?rev=368559&view=rev
Log:
Cleanup unused variable.
This patch fixes problem raised in post-review comments of the
https://reviews.llvm.org/D65285. Developers of ittnotify confirmed
that dll_path_ptr field of the __itt_global structure is never used
by ittnotify library, so it is safe to remove the dll_path array.
Differential Revision: https://reviews.llvm.org/D65885
Modified:
openmp/trunk/runtime/src/thirdparty/ittnotify/ittnotify_static.cpp
Modified: openmp/trunk/runtime/src/thirdparty/ittnotify/ittnotify_static.cpp
URL: http://llvm.org/viewvc/llvm-project/openmp/trunk/runtime/src/thirdparty/ittnotify/ittnotify_static.cpp?rev=368559&r1=368558&r2=368559&view=diff
==============================================================================
--- openmp/trunk/runtime/src/thirdparty/ittnotify/ittnotify_static.cpp (original)
+++ openmp/trunk/runtime/src/thirdparty/ittnotify/ittnotify_static.cpp Mon Aug 12 05:37:30 2019
@@ -226,8 +226,6 @@ static __itt_api_info api_list[] = {
#pragma warning(pop)
#endif /* ITT_PLATFORM==ITT_PLATFORM_WIN */
-static char dll_path[PATH_MAX] = { 0 };
-
/* static part descriptor which handles. all notification api attributes. */
__itt_global _N_(_ittapi_global) = {
ITT_MAGIC, /* identification info */
@@ -238,7 +236,7 @@ __itt_global _N_(_ittapi_global) = {
MUTEX_INITIALIZER, /* mutex */
NULL, /* dynamic library handle */
NULL, /* error_handler */
- (const char**)&dll_path, /* dll_path_ptr */
+ NULL, /* dll_path_ptr */
(__itt_api_info*)&api_list, /* api_list_ptr */
NULL, /* next __itt_global */
NULL, /* thread_list */
More information about the Openmp-commits
mailing list