[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