[Openmp-commits] [openmp] 18a6cce - [Libomptarget] Fix misused macro name preventing printing of library name

Joseph Huber via Openmp-commits openmp-commits at lists.llvm.org
Tue Jul 4 06:00:40 PDT 2023


Author: Joseph Huber
Date: 2023-07-04T08:00:27-05:00
New Revision: 18a6ccea3ae35a0de4bea7e9a603e9b83d48b084

URL: https://github.com/llvm/llvm-project/commit/18a6ccea3ae35a0de4bea7e9a603e9b83d48b084
DIFF: https://github.com/llvm/llvm-project/commit/18a6ccea3ae35a0de4bea7e9a603e9b83d48b084.diff

LOG: [Libomptarget] Fix misused macro name preventing printing of library name

Summary:
This code used `LIBOMPTARGET_DEBUG` which is not the macro name, but the
environment variable. This caused this portion to always be disabled. In
the long run we should aim for this to always be availible as it's
useful for other diagnostic message.

Added: 
    

Modified: 
    openmp/libomptarget/src/rtl.cpp

Removed: 
    


################################################################################
diff  --git a/openmp/libomptarget/src/rtl.cpp b/openmp/libomptarget/src/rtl.cpp
index 5360d9784cb628..fb3754110efa48 100644
--- a/openmp/libomptarget/src/rtl.cpp
+++ b/openmp/libomptarget/src/rtl.cpp
@@ -196,7 +196,7 @@ bool RTLsTy::attemptLoadRTL(const std::string &RTLName, RTLInfoTy &RTL) {
     return false;
   }
 
-#ifdef LIBOMPTARGET_DEBUG
+#ifdef OMPTARGET_DEBUG
   RTL.RTLName = Name;
 #endif
 


        


More information about the Openmp-commits mailing list