[llvm] [NFC][Offload] Move variables to where they are used (PR #99956)
Shilei Tian via llvm-commits
llvm-commits at lists.llvm.org
Mon Jul 22 14:43:43 PDT 2024
https://github.com/shiltian created https://github.com/llvm/llvm-project/pull/99956
None
>From 05b85168f9cf99226779b548da1bf0b38983d4ac Mon Sep 17 00:00:00 2001
From: Shilei Tian <i at tianshilei.me>
Date: Mon, 22 Jul 2024 17:43:05 -0400
Subject: [PATCH] [NFC][Offload] Move variables to where they are used
---
offload/DeviceRTL/src/Parallelism.cpp | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/offload/DeviceRTL/src/Parallelism.cpp b/offload/DeviceRTL/src/Parallelism.cpp
index 031a5ced25518..15b991f202539 100644
--- a/offload/DeviceRTL/src/Parallelism.cpp
+++ b/offload/DeviceRTL/src/Parallelism.cpp
@@ -166,9 +166,6 @@ __kmpc_parallel_51(IdentTy *ident, int32_t, int32_t if_expr,
// From this point forward we know that there is no thread state used.
ASSERT(state::HasThreadState == false, nullptr);
- uint32_t NumThreads = determineNumberOfThreads(num_threads);
- uint32_t MaxTeamThreads = mapping::getMaxTeamThreads();
- uint32_t PTeamSize = NumThreads == MaxTeamThreads ? 0 : NumThreads;
if (mapping::isSPMDMode()) {
// This was moved to its own routine so it could be called directly
// in certain situations to avoid resource consumption of unused
@@ -178,6 +175,10 @@ __kmpc_parallel_51(IdentTy *ident, int32_t, int32_t if_expr,
return;
}
+ uint32_t NumThreads = determineNumberOfThreads(num_threads);
+ uint32_t MaxTeamThreads = mapping::getMaxTeamThreads();
+ uint32_t PTeamSize = NumThreads == MaxTeamThreads ? 0 : NumThreads;
+
// We do *not* create a new data environment because all threads in the team
// that are active are now running this parallel region. They share the
// TeamState, which has an increase level-var and potentially active-level
More information about the llvm-commits
mailing list