[llvm] [AMDGPU] Expose buffer resource num_records width in TargetParser (PR #220103)
Krzysztof Drewniak via llvm-commits
llvm-commits at lists.llvm.org
Wed Sep 9 23:38:47 PDT 2026
================
@@ -3209,6 +3209,44 @@ TEST(TargetParserTest, testAMDGPUgetMaxHWAddressableLocalMemorySize) {
327680u);
}
+TEST(TargetParserTest, testAMDGPUgetBufferResourceNumRecordsWidth) {
+ EXPECT_EQ(AMDGPU::getBufferResourceNumRecordsWidth(AMDGPU::GK_GFX900), 32u);
+ EXPECT_EQ(AMDGPU::getBufferResourceNumRecordsWidth(AMDGPU::GK_GFX1201), 32u);
+ EXPECT_EQ(AMDGPU::getBufferResourceNumRecordsWidth(AMDGPU::GK_GFX1250), 45u);
+
+ // Generic families that agree on resource width behave.
+ EXPECT_EQ(AMDGPU::getBufferResourceNumRecordsWidth(AMDGPU::GK_GFX9_GENERIC),
+ 32u);
+
+ EXPECT_EQ(AMDGPU::getBufferResourceNumRecordsWidth(AMDGPU::GK_GENERIC),
+ std::nullopt);
+ EXPECT_EQ(AMDGPU::getBufferResourceNumRecordsWidth(AMDGPU::GK_GENERIC_HSA),
+ std::nullopt);
----------------
krzysz00 wrote:
I like optional here - makes it hard to forget about the failure case
https://github.com/llvm/llvm-project/pull/220103
More information about the llvm-commits
mailing list