From fa43df7e57c14ed46765556dcf8b68457df78a69 Mon Sep 17 00:00:00 2001 From: Shilei Tian Date: Mon, 25 Aug 2025 00:05:13 -0400 Subject: [PATCH] [NFC][AMDGPU] Remove redundant code in `AMDGPUSubtarget::getWavesPerEU` --- llvm/lib/Target/AMDGPU/AMDGPUSubtarget.cpp | 13 +++++-------- 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/llvm/lib/Target/AMDGPU/AMDGPUSubtarget.cpp b/llvm/lib/Target/AMDGPU/AMDGPUSubtarget.cpp index d095fc6cf9549..a1d82128423bb 100644 --- a/llvm/lib/Target/AMDGPU/AMDGPUSubtarget.cpp +++ b/llvm/lib/Target/AMDGPU/AMDGPUSubtarget.cpp @@ -209,19 +209,16 @@ std::pair AMDGPUSubtarget::getWavesPerEU(const Function &F) const { // Default/requested minimum/maximum flat work group sizes. std::pair FlatWorkGroupSizes = getFlatWorkGroupSizes(F); - // Minimum number of bytes allocated in the LDS. - unsigned LDSBytes = AMDGPU::getIntegerPairAttribute(F, "amdgpu-lds-size", - {0, UINT32_MAX}, true) - .first; - return getWavesPerEU(FlatWorkGroupSizes, LDSBytes, F); + return getWavesPerEU(F, FlatWorkGroupSizes); } std::pair AMDGPUSubtarget::getWavesPerEU( const Function &F, std::pair FlatWorkGroupSizes) const { // Minimum number of bytes allocated in the LDS. - unsigned LDSBytes = AMDGPU::getIntegerPairAttribute(F, "amdgpu-lds-size", - {0, UINT32_MAX}, true) - .first; + unsigned LDSBytes = + AMDGPU::getIntegerPairAttribute(F, "amdgpu-lds-size", {0, UINT32_MAX}, + /*OnlyFirstRequired=*/true) + .first; return getWavesPerEU(FlatWorkGroupSizes, LDSBytes, F); }