[llvm] [OpenMPIRBuilder] NFC: Improve description of IsTargetDevice and IsGPU (PR #79322)
Sergio Afonso via llvm-commits
llvm-commits at lists.llvm.org
Wed Jan 24 08:29:00 PST 2024
https://github.com/skatrak created https://github.com/llvm/llvm-project/pull/79322
This patch tries to better explain the differences between the `IsTargetDevice` and `IsGPU` flags of the `OpenMPIRBuilderConfig`.
>From 01ff2a3a5599af72a038bac99ebbbfeceec28035 Mon Sep 17 00:00:00 2001
From: Sergio Afonso <safonsof at amd.com>
Date: Wed, 24 Jan 2024 16:25:10 +0000
Subject: [PATCH] [OpenMPIRBuilder] NFC: Improve description of IsTargetDevice
and IsGPU
This patch tries to better explain the differences between the `IsTargetDevice`
and `IsGPU` flags of the `OpenMPIRBuilderConfig`.
---
llvm/include/llvm/Frontend/OpenMP/OMPIRBuilder.h | 15 ++++++++++++---
1 file changed, 12 insertions(+), 3 deletions(-)
diff --git a/llvm/include/llvm/Frontend/OpenMP/OMPIRBuilder.h b/llvm/include/llvm/Frontend/OpenMP/OMPIRBuilder.h
index 669104307fa0e2..2288969ecc95c4 100644
--- a/llvm/include/llvm/Frontend/OpenMP/OMPIRBuilder.h
+++ b/llvm/include/llvm/Frontend/OpenMP/OMPIRBuilder.h
@@ -83,11 +83,20 @@ BasicBlock *splitBBWithSuffix(IRBuilderBase &Builder, bool CreateBranch,
/// are ones that are not dependent on the configuration.
class OpenMPIRBuilderConfig {
public:
- /// Flag for specifying if the compilation is done for embedded device code
- /// or host code.
+ /// Flag to define whether to generate code for the role of the OpenMP host
+ /// (if set to false) or device (if set to true) in an offloading context. It
+ /// is set when the -fopenmp-is-target-device compiler frontend option is
+ /// specified.
std::optional<bool> IsTargetDevice;
- /// Flag for specifying if the compilation is done for an accelerator.
+ /// Flag for specifying if the compilation is done for an accelerator. It is
+ /// set according to the architecture of the target triple and currently only
+ /// true when targeting AMDGPU or NVPTX. Today, these targets can only perform
+ /// the role of an OpenMP target device, so `IsTargetDevice` must also be true
+ /// if `IsGPU` is true. This restriction might be lifted if an accelerator-
+ /// like target with the ability to work as the OpenMP host is added, or if
+ /// the capabilities of the currently supported GPU architectures are
+ /// expanded.
std::optional<bool> IsGPU;
// Flag for specifying if offloading is mandatory.
More information about the llvm-commits
mailing list