Skip to content

Commit c148ac9

Browse files
committed
self-review: output unbounded instead of UINT_MAX for numDescriptors
1 parent d9714be commit c148ac9

File tree

3 files changed

+9
-5
lines changed

3 files changed

+9
-5
lines changed

clang/test/AST/HLSL/RootSignatures-AST.hlsl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@
4141
// CHECK-SAME: t1, numDescriptors = 8, space = 0, offset = DescriptorTableOffsetAppend, flags = DescriptorsVolatile
4242
// CHECK-SAME: ),
4343
// CHECK-SAME: UAV(
44-
// CHECK-SAME: u1, numDescriptors = 4294967295, space = 0, offset = DescriptorTableOffsetAppend, flags = DescriptorsVolatile
44+
// CHECK-SAME: u1, numDescriptors = unbounded, space = 0, offset = DescriptorTableOffsetAppend, flags = DescriptorsVolatile
4545
// CHECK-SAME: ),
4646
// CHECK-SAME: DescriptorTable(
4747
// CHECK-SAME: numClauses = 3, visibility = All

llvm/lib/Frontend/HLSL/HLSLRootSignatureUtils.cpp

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -288,8 +288,12 @@ raw_ostream &operator<<(raw_ostream &OS, const DescriptorTable &Table) {
288288

289289
raw_ostream &operator<<(raw_ostream &OS, const DescriptorTableClause &Clause) {
290290
OS << Clause.Type << "(" << Clause.Reg
291-
<< ", numDescriptors = " << Clause.NumDescriptors
292-
<< ", space = " << Clause.Space << ", offset = ";
291+
<< ", numDescriptors = ";
292+
if (Clause.NumDescriptors == NumDescriptorsUnbounded)
293+
OS << "unbounded";
294+
else
295+
OS << Clause.NumDescriptors;
296+
OS << ", space = " << Clause.Space << ", offset = ";
293297
if (Clause.Offset == DescriptorTableOffsetAppend)
294298
OS << "DescriptorTableOffsetAppend";
295299
else

llvm/unittests/Frontend/HLSLRootSignatureDumpTest.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ TEST(HLSLRootSignatureTest, DescriptorSRVClauseDump) {
3434
DescriptorTableClause Clause;
3535
Clause.Type = ClauseType::SRV;
3636
Clause.Reg = {RegisterType::TReg, 0};
37-
Clause.NumDescriptors = 2;
37+
Clause.NumDescriptors = NumDescriptorsUnbounded;
3838
Clause.Space = 42;
3939
Clause.Offset = 3;
4040
Clause.Flags = DescriptorRangeFlags::None;
@@ -45,7 +45,7 @@ TEST(HLSLRootSignatureTest, DescriptorSRVClauseDump) {
4545
OS.flush();
4646

4747
std::string Expected =
48-
"SRV(t0, numDescriptors = 2, space = 42, offset = 3, flags = None)";
48+
"SRV(t0, numDescriptors = unbounded, space = 42, offset = 3, flags = None)";
4949
EXPECT_EQ(Out, Expected);
5050
}
5151

0 commit comments

Comments
 (0)