Skip to content

Commit 8fc5dfb

Browse files
committed
Remove unneeded Function Argument from applyDynamicWorkshareLoop
dist_schedule is not supported for dynamically allocated loops, so we can remove the need for the DistScheduleChunkSize from this function.
1 parent 79d96cf commit 8fc5dfb

File tree

2 files changed

+7
-9
lines changed

2 files changed

+7
-9
lines changed

llvm/include/llvm/Frontend/OpenMP/OMPIRBuilder.h

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1168,15 +1168,14 @@ class OpenMPIRBuilder {
11681168
/// the loop.
11691169
/// \param Chunk The size of loop chunk considered as a unit when
11701170
/// scheduling. If \p nullptr, defaults to 1.
1171-
/// \param DistScheduleChunk The size of dist_shcedule chunk considered as
1172-
/// a unit when
1173-
/// scheduling. If \p nullptr, defaults to 1.
11741171
///
11751172
/// \returns Point where to insert code after the workshare construct.
1176-
InsertPointOrErrorTy applyDynamicWorkshareLoop(
1177-
DebugLoc DL, CanonicalLoopInfo *CLI, InsertPointTy AllocaIP,
1178-
omp::OMPScheduleType SchedType, bool NeedsBarrier, Value *Chunk = nullptr,
1179-
Value *DistScheduleChunk = nullptr);
1173+
InsertPointOrErrorTy applyDynamicWorkshareLoop(DebugLoc DL,
1174+
CanonicalLoopInfo *CLI,
1175+
InsertPointTy AllocaIP,
1176+
omp::OMPScheduleType SchedType,
1177+
bool NeedsBarrier,
1178+
Value *Chunk = nullptr);
11801179

11811180
/// Create alternative version of the loop to support if clause
11821181
///

llvm/lib/Frontend/OpenMP/OMPIRBuilder.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5374,8 +5374,7 @@ OpenMPIRBuilder::InsertPointOrErrorTy
53745374
OpenMPIRBuilder::applyDynamicWorkshareLoop(DebugLoc DL, CanonicalLoopInfo *CLI,
53755375
InsertPointTy AllocaIP,
53765376
OMPScheduleType SchedType,
5377-
bool NeedsBarrier, Value *Chunk,
5378-
Value *DistScheduleChunk) {
5377+
bool NeedsBarrier, Value *Chunk) {
53795378
assert(CLI->isValid() && "Requires a valid canonical loop");
53805379
assert(!isConflictIP(AllocaIP, CLI->getPreheaderIP()) &&
53815380
"Require dedicated allocate IP");

0 commit comments

Comments
 (0)