Skip to content

Commit 662c3a8

Browse files
author
joaosaffran
committed
addressing comments
1 parent ef14638 commit 662c3a8

8 files changed

+8
-8
lines changed

llvm/include/llvm/Frontend/HLSL/RootSignatureValidations.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -179,7 +179,7 @@ class RootSignatureBindingValidation {
179179
}
180180
};
181181
llvm::SmallVector<RangeInfo>
182-
findUnboundRanges(const llvm::SmallVectorImpl<RangeInfo> &Ranges,
182+
findUnboundRanges(const llvm::ArrayRef<RangeInfo> &Ranges,
183183
const llvm::ArrayRef<RangeInfo> &Bindings);
184184
} // namespace rootsig
185185
} // namespace hlsl

llvm/lib/Frontend/HLSL/RootSignatureValidations.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -317,7 +317,7 @@ findOverlappingRanges(llvm::SmallVector<RangeInfo> &Infos) {
317317
}
318318

319319
llvm::SmallVector<RangeInfo>
320-
findUnboundRanges(const llvm::SmallVectorImpl<RangeInfo> &Ranges,
320+
findUnboundRanges(const llvm::ArrayRef<RangeInfo> &Ranges,
321321
const llvm::ArrayRef<RangeInfo> &Bindings) {
322322
llvm::SmallVector<RangeInfo> Unbounds;
323323
for (const auto &Range : Ranges) {

llvm/lib/Target/DirectX/DXILPostOptimizationValidation.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,7 @@ static void reportRegNotBound(Module &M,
135135
OS << "register " << ResourceClassToString(Unbound.Class)
136136
<< " (space=" << Unbound.Space << ", register=" << Unbound.LowerBound
137137
<< ")"
138-
<< " is not defined in Root Signature";
138+
<< " does not have a binding in the Root Signature";
139139
M.getContext().diagnose(DiagnosticInfoGeneric(Message));
140140
}
141141

llvm/test/CodeGen/DirectX/rootsignature-validation-fail-cbuffer.ll

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
; RUN: not opt -S -passes='dxil-post-optimization-validation' -mtriple=dxil-pc-shadermodel6.6-compute %s 2>&1 | FileCheck %s
22

3-
; CHECK: error: register CBuffer (space=665, register=3) is not defined in Root Signature
3+
; CHECK: error: register CBuffer (space=665, register=3) does not have a binding in the Root Signature
44

55
; Root Signature(
66
; CBV(b3, space=666, visibility=SHADER_VISIBILITY_ALL)

llvm/test/CodeGen/DirectX/rootsignature-validation-fail-constants.ll

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
; RUN: not opt -S -passes='dxil-post-optimization-validation' -mtriple=dxil-pc-shadermodel6.6-compute %s 2>&1 | FileCheck %s
2-
; CHECK: error: register CBuffer (space=666, register=2) is not defined in Root Signature
2+
; CHECK: error: register CBuffer (space=666, register=2) does not have a binding in the Root Signature
33
; Root Signature(RootConstants(num32BitConstants=4, b2))
44

55
%__cblayout_CB = type <{ float }>

llvm/test/CodeGen/DirectX/rootsignature-validation-fail-sampler.ll

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
; RUN: not opt -S -passes='dxil-post-optimization-validation' -mtriple=dxil-pc-shadermodel6.6-compute %s 2>&1 | FileCheck %s
22

3-
; CHECK: error: register Sampler (space=2, register=3) is not defined in Root Signature
3+
; CHECK: error: register Sampler (space=2, register=3) does not have a binding in the Root Signature
44

55
; Root Signature(
66
; CBV(b3, space=666, visibility=SHADER_VISIBILITY_ALL)

llvm/test/CodeGen/DirectX/rootsignature-validation-fail-srv.ll

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
; RUN: not opt -S -passes='dxil-post-optimization-validation' -mtriple=dxil-pc-shadermodel6.6-compute %s 2>&1 | FileCheck %s
22

3-
; CHECK: error: register SRV (space=0, register=0) is not defined in Root Signature
3+
; CHECK: error: register SRV (space=0, register=0) does not have a binding in the Root Signature
44

55
; Root Signature(
66
; CBV(b3, space=666, visibility=SHADER_VISIBILITY_ALL)

llvm/test/CodeGen/DirectX/rootsignature-validation-fail-uav.ll

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
; RUN: not opt -S -passes='dxil-post-optimization-validation' -mtriple=dxil-pc-shadermodel6.6-compute %s 2>&1 | FileCheck %s
22

3-
; CHECK: error: register UAV (space=0, register=4294967295) is not defined in Root Signature
3+
; CHECK: error: register UAV (space=0, register=4294967295) does not have a binding in the Root Signature
44

55
; Root Signature(
66
; CBV(b3, space=666, visibility=SHADER_VISIBILITY_ALL)

0 commit comments

Comments
 (0)