Skip to content

Commit 8bdc206

Browse files
author
joaosaffran
committed
address comments
1 parent c4b78d8 commit 8bdc206

File tree

3 files changed

+3
-5
lines changed

3 files changed

+3
-5
lines changed

llvm/lib/Target/DirectX/DXILRootSignature.cpp

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -192,9 +192,8 @@ static bool validate(LLVMContext *Ctx, const mcdxbc::RootSignatureDesc &RSD) {
192192
return reportValueError(Ctx, "ShaderVisibility",
193193
(uint32_t)P.Header.ShaderVisibility);
194194

195-
if (!verifyParameterType(P.Header.ParameterType))
196-
return reportValueError(Ctx, "ParameterType",
197-
(uint32_t)P.Header.ParameterType);
195+
assert(verifyParameterType(P.Header.ParameterType) &&
196+
"Invalid value for ParameterType");
198197
}
199198

200199
return false;

llvm/test/CodeGen/DirectX/ContainerData/RootSignature-Flags-Validation-Error.ll renamed to llvm/test/CodeGen/DirectX/ContainerData/RootSignature-Parameters-Validation-Error.ll

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,5 @@ attributes #0 = { "hlsl.numthreads"="1,1,1" "hlsl.shader"="compute" }
1616

1717
!dx.rootsignatures = !{!2} ; list of function/root signature pairs
1818
!2 = !{ ptr @main, !3 } ; function, root signature
19-
!3 = !{ !4, !5 } ; list of root signature elements
20-
!4 = !{ !"RootFlags", i32 1 } ; 1 = allow_input_assembler_input_layout
19+
!3 = !{ !5 } ; list of root signature elements
2120
!5 = !{ !"RootConstants", i32 255, i32 1, i32 2, i32 3 }

0 commit comments

Comments
 (0)