[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