[Openmp-commits] [openmp] 85184b4 - [OpenMP] Fix libomptarget build issue (#74067)

via Openmp-commits openmp-commits at lists.llvm.org
Fri Dec 1 04:33:19 PST 2023


Author: Dominik Adamski
Date: 2023-12-01T13:33:11+01:00
New Revision: 85184b4aefbd01afd6e7be57bc6c1c404b3c13ce

URL: https://github.com/llvm/llvm-project/commit/85184b4aefbd01afd6e7be57bc6c1c404b3c13ce
DIFF: https://github.com/llvm/llvm-project/commit/85184b4aefbd01afd6e7be57bc6c1c404b3c13ce.diff

LOG: [OpenMP] Fix libomptarget build issue (#74067)

Libomptarget cannot be build because of the recent refactoring
introduced in patch 148dec9fa43b :
[OpenMP][NFC] Separate Envar (environment variable) handling (#73994)

That patch moved handling of environment variables from libomptarget
library. That's why we don't need usage of "llvm::omp::target" namespace
if we handle environment variables.

Added: 
    

Modified: 
    openmp/libomptarget/src/interface.cpp

Removed: 
    


################################################################################
diff  --git a/openmp/libomptarget/src/interface.cpp b/openmp/libomptarget/src/interface.cpp
index a2f713459e1d0c9..ee1bd4932442b1f 100644
--- a/openmp/libomptarget/src/interface.cpp
+++ b/openmp/libomptarget/src/interface.cpp
@@ -454,7 +454,6 @@ EXTERN void __tgt_target_nowait_query(void **AsyncHandle) {
   // for the device operations (work/spin wait on them) or block until they are
   // completed (use device side blocking mechanism). This allows the runtime to
   // adapt itself when there are a lot of long-running target regions in-flight.
-  using namespace llvm::omp::target;
   static thread_local utils::ExponentialBackoff QueryCounter(
       Int64Envar("OMPTARGET_QUERY_COUNT_MAX", 10),
       Int64Envar("OMPTARGET_QUERY_COUNT_THRESHOLD", 5),


        


More information about the Openmp-commits mailing list