Skip to content

Commit bee2efb

Browse files
author
git apple-llvm automerger
committed
Merge commit '7470b1e8773e' from apple/master into swift/master-next
2 parents 1c3abc6 + 7470b1e commit bee2efb

File tree

1 file changed

+3
-8
lines changed

1 file changed

+3
-8
lines changed

llvm/lib/Target/AMDGPU/AMDGPUSubtarget.cpp

Lines changed: 3 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -411,11 +411,8 @@ std::pair<unsigned, unsigned> AMDGPUSubtarget::getWavesPerEU(
411411
unsigned MinImpliedByFlatWorkGroupSize =
412412
getWavesPerEUForWorkGroup(FlatWorkGroupSizes.second);
413413
Default.first = MinImpliedByFlatWorkGroupSize;
414-
bool RequestedFlatWorkGroupSize = false;
415-
416-
if (F.hasFnAttribute("amdgpu-flat-work-group-size")) {
417-
RequestedFlatWorkGroupSize = true;
418-
}
414+
bool RequestedFlatWorkGroupSize =
415+
F.hasFnAttribute("amdgpu-flat-work-group-size");
419416

420417
// Requested minimum/maximum number of waves per execution unit.
421418
std::pair<unsigned, unsigned> Requested = AMDGPU::getIntegerPairAttribute(
@@ -427,9 +424,7 @@ std::pair<unsigned, unsigned> AMDGPUSubtarget::getWavesPerEU(
427424

428425
// Make sure requested values do not violate subtarget's specifications.
429426
if (Requested.first < getMinWavesPerEU() ||
430-
Requested.first > getMaxWavesPerEU())
431-
return Default;
432-
if (Requested.second > getMaxWavesPerEU())
427+
Requested.second > getMaxWavesPerEU())
433428
return Default;
434429

435430
// Make sure requested values are compatible with values implied by requested

0 commit comments

Comments
 (0)