[Openmp-commits] [openmp] bdecfeb - [OpenMP][NFC] Rename OmptCallback.cpp into OpenMP/OMPT/Callback.cpp (#73813)

[via Openmp-commits]

Author: Johannes Doerfert
Date: 2023-11-29T08:44:07-08:00
New Revision: bdecfebce4b0af94b3afa3e20add0b9d232f21ad

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

LOG: [OpenMP][NFC] Rename OmptCallback.cpp into OpenMP/OMPT/Callback.cpp (#73813)

Also revert the ifdef OMPT_SUPPORT order to have the short fallback
first and not after 400 lines.

Added: 
    openmp/libomptarget/src/OpenMP/OMPT/Callback.cpp

Modified: 
    openmp/libomptarget/src/CMakeLists.txt

Removed: 
    openmp/libomptarget/src/OmptCallback.cpp


################################################################################
diff  --git a/openmp/libomptarget/src/CMakeLists.txt b/openmp/libomptarget/src/CMakeLists.txt
index a83965f59b3eb04..8d85662888090a8 100644
--- a/openmp/libomptarget/src/CMakeLists.txt
+++ b/openmp/libomptarget/src/CMakeLists.txt
@@ -19,10 +19,10 @@ add_llvm_library(omptarget
   device.cpp
   interface.cpp
   omptarget.cpp
-  OmptCallback.cpp
   rtl.cpp
   LegacyAPI.cpp
   OpenMP/InteropAPI.cpp
+  OpenMP/OMPT/Callback.cpp
 
   ADDITIONAL_HEADER_DIRS
   ${LIBOMPTARGET_INCLUDE_DIR}

diff  --git a/openmp/libomptarget/src/OmptCallback.cpp b/openmp/libomptarget/src/OpenMP/OMPT/Callback.cpp
similarity index 99%
rename from openmp/libomptarget/src/OmptCallback.cpp
rename to openmp/libomptarget/src/OpenMP/OMPT/Callback.cpp
index a45d1a0f282ac02..f1a8ad0e3cafb7b 100644
--- a/openmp/libomptarget/src/OmptCallback.cpp
+++ b/openmp/libomptarget/src/OpenMP/OMPT/Callback.cpp
@@ -1,4 +1,4 @@
-//===-- OmptCallback.cpp - Target independent OpenMP target RTL --- C++ -*-===//
+//===-- OpenMP/OMPT/Callback.cpp - OpenMP Tooling Callback implementation -===//
 //
 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
 // See https://llvm.org/LICENSE.txt for license information.
@@ -10,9 +10,14 @@
 //
 //===----------------------------------------------------------------------===//
 
-#ifdef OMPT_SUPPORT
+#ifndef OMPT_SUPPORT
 
-#include "llvm/Support/DynamicLibrary.h"
+extern "C" {
+/// Dummy definition when OMPT is disabled
+void ompt_libomptarget_connect() {}
+}
+
+#else // OMPT_SUPPORT is set
 
 #include <cstdlib>
 #include <cstring>
@@ -24,6 +29,8 @@
 #include "OpenMP/OMPT/Connector.h"
 #include "OpenMP/OMPT/Interface.h"
 
+#include "llvm/Support/DynamicLibrary.h"
+
 #undef DEBUG_PREFIX
 #define DEBUG_PREFIX "OMPT"
 
@@ -491,9 +498,4 @@ void ompt_libomptarget_connect(ompt_start_tool_result_t *result) {
   DP("Leave ompt_libomptarget_connect\n");
 }
 }
-#else
-extern "C" {
-/// Dummy definition when OMPT is disabled
-void ompt_libomptarget_connect() {}
-}
 #endif // OMPT_SUPPORT