[Openmp-commits] [openmp] 0e7e426 - [OMPT] Fix debug prefix not being defined

Joseph Huber via Openmp-commits openmp-commits at lists.llvm.org
Wed Nov 16 05:53:50 PST 2022


Author: Joseph Huber
Date: 2022-11-16T07:53:16-06:00
New Revision: 0e7e426c0c4a42416da6e0d30da4826575ecedd0

URL: https://github.com/llvm/llvm-project/commit/0e7e426c0c4a42416da6e0d30da4826575ecedd0
DIFF: https://github.com/llvm/llvm-project/commit/0e7e426c0c4a42416da6e0d30da4826575ecedd0.diff

LOG: [OMPT] Fix debug prefix not being defined

Summary:
This header file uses the `DP` prefixes but does not define
`DEBUG_PREFIX`. This patch adds a simple fix, but realistically the `DP`
system isn't ideal. Now that we have access to LLVM libraries and other
utilities we should consider rewriting all of the debugging and error
handling glue.

Added: 
    

Modified: 
    openmp/libomptarget/include/ompt_connector.h

Removed: 
    


################################################################################
diff  --git a/openmp/libomptarget/include/ompt_connector.h b/openmp/libomptarget/include/ompt_connector.h
index f39607106d68d..5d8648d694d31 100644
--- a/openmp/libomptarget/include/ompt_connector.h
+++ b/openmp/libomptarget/include/ompt_connector.h
@@ -26,6 +26,8 @@
 #include "omp-tools.h"
 #include "omptarget.h"
 
+#define DEBUG_PREFIX "OMPT"
+
 #define LIBOMPTARGET_STRINGIFY(s) #s
 
 /// Type for the function to be invoked for connecting two libraries.
@@ -99,6 +101,8 @@ class OmptLibraryConnectorTy {
   std::string LibIdent;
 };
 
+#undef DEBUG_PREFIX
+
 #endif // OMPT_SUPPORT
 
 #endif // _OMPT_CONNECTOR_H


        


More information about the Openmp-commits mailing list