[llvm] This patch reverts the fix added in #85218 in favour of a better approach (PR #88118)

Akash Banerjee via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 9 05:31:34 PDT 2024


https://github.com/TIFitis created https://github.com/llvm/llvm-project/pull/88118

This patch reverts the fix added in #85218 in favour of a better approach.

>From 6d1b173f3f8da866766bbb1703f7c38b6ab25607 Mon Sep 17 00:00:00 2001
From: Akash Banerjee <Akash.Banerjee at amd.com>
Date: Tue, 9 Apr 2024 13:29:18 +0100
Subject: [PATCH] This patch reverts the fix added in #85218 in favour of a
 better approach.

---
 llvm/lib/Frontend/OpenMP/OMPIRBuilder.cpp | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/llvm/lib/Frontend/OpenMP/OMPIRBuilder.cpp b/llvm/lib/Frontend/OpenMP/OMPIRBuilder.cpp
index 7fd8474c2ec890..6b5446182a772f 100644
--- a/llvm/lib/Frontend/OpenMP/OMPIRBuilder.cpp
+++ b/llvm/lib/Frontend/OpenMP/OMPIRBuilder.cpp
@@ -4886,11 +4886,8 @@ OpenMPIRBuilder::InsertPointTy OpenMPIRBuilder::createTargetData(
     return InsertPointTy();
 
   // Disable TargetData CodeGen on Device pass.
-  if (Config.IsTargetDevice.value_or(false)) {
-    if (BodyGenCB)
-      Builder.restoreIP(BodyGenCB(Builder.saveIP(), BodyGenTy::NoPriv));
+  if (Config.IsTargetDevice.value_or(false))
     return Builder.saveIP();
-  }
 
   Builder.restoreIP(CodeGenIP);
   bool IsStandAlone = !BodyGenCB;



More information about the llvm-commits mailing list