[llvm-branch-commits] [llvm] [openmp] [OpenMP][Offload] Add offload runtime support for dyn_groupprivate clause (PR #152831)
via llvm-branch-commits
llvm-branch-commits at lists.llvm.org
Sat Aug 9 22:55:13 PDT 2025
github-actions[bot] wrote:
<!--LLVM CODE FORMAT COMMENT: {clang-format}-->
:warning: C/C++ code formatter, clang-format found issues in your code. :warning:
<details>
<summary>
You can test this locally with the following command:
</summary>
``````````bash
git-clang-format --diff HEAD~1 HEAD --extensions cpp,h -- offload/test/offloading/dyn_groupprivate_strict.cpp offload/DeviceRTL/include/DeviceTypes.h offload/DeviceRTL/include/Interface.h offload/DeviceRTL/include/State.h offload/DeviceRTL/src/Kernel.cpp offload/DeviceRTL/src/State.cpp offload/include/Shared/APITypes.h offload/include/Shared/Environment.h offload/include/device.h offload/include/omptarget.h offload/libomptarget/OpenMP/API.cpp offload/libomptarget/device.cpp offload/plugins-nextgen/amdgpu/dynamic_hsa/hsa_ext_amd.h offload/plugins-nextgen/amdgpu/src/rtl.cpp offload/plugins-nextgen/common/include/PluginInterface.h offload/plugins-nextgen/common/src/PluginInterface.cpp offload/plugins-nextgen/cuda/dynamic_cuda/cuda.h offload/plugins-nextgen/cuda/src/rtl.cpp offload/plugins-nextgen/host/src/rtl.cpp openmp/runtime/src/kmp_csupport.cpp openmp/runtime/src/kmp_stub.cpp
``````````
</details>
<details>
<summary>
View the diff from clang-format here.
</summary>
``````````diff
diff --git a/offload/plugins-nextgen/amdgpu/src/rtl.cpp b/offload/plugins-nextgen/amdgpu/src/rtl.cpp
index 48e677d06..9751169b0 100644
--- a/offload/plugins-nextgen/amdgpu/src/rtl.cpp
+++ b/offload/plugins-nextgen/amdgpu/src/rtl.cpp
@@ -3442,7 +3442,8 @@ Error AMDGPUKernelTy::launchImpl(GenericDeviceTy &GenericDevice,
}
// Increase to the requested dynamic memory size for the device if needed.
- DynBlockMemSize = std::max(DynBlockMemSize, GenericDevice.getDynamicMemorySize());
+ DynBlockMemSize =
+ std::max(DynBlockMemSize, GenericDevice.getDynamicMemorySize());
// Push the kernel launch into the stream.
return Stream->pushKernelLaunch(*this, AllArgs, NumThreads, NumBlocks,
diff --git a/offload/plugins-nextgen/cuda/src/rtl.cpp b/offload/plugins-nextgen/cuda/src/rtl.cpp
index bd1cedf56..b052197e2 100644
--- a/offload/plugins-nextgen/cuda/src/rtl.cpp
+++ b/offload/plugins-nextgen/cuda/src/rtl.cpp
@@ -1323,7 +1323,8 @@ Error CUDAKernelTy::launchImpl(GenericDeviceTy &GenericDevice,
GenericDevice.Plugin.getRPCServer().Thread->notify();
// Increase to the requested dynamic memory size for the device if needed.
- DynBlockMemSize = std::max(DynBlockMemSize, GenericDevice.getDynamicMemorySize());
+ DynBlockMemSize =
+ std::max(DynBlockMemSize, GenericDevice.getDynamicMemorySize());
// In case we require more memory than the current limit.
if (DynBlockMemSize >= MaxDynBlockMemSize) {
``````````
</details>
https://github.com/llvm/llvm-project/pull/152831
More information about the llvm-branch-commits
mailing list