Skip to content

Commit 23069ab

Browse files
author
joaosaffran
committed
addressing comments
1 parent 68c7513 commit 23069ab

File tree

1 file changed

+5
-8
lines changed

1 file changed

+5
-8
lines changed

llvm/lib/MC/DXContainerRootSignature.cpp

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -83,13 +83,12 @@ Error RootSignatureDesc::write(raw_ostream &OS) const {
8383
ParamsOffset.push_back(Offset);
8484
}
8585

86-
size_t It = 0;
87-
for (const auto &P : Parameters) {
88-
89-
auto Offset = ParamsOffset[It];
90-
if (Error Err = rewriteOffset(Writer, Offset))
86+
assert(NumParameters == ParamsOffset.size());
87+
for (size_t I = 0; I < NumParameters; ++I) {
88+
if (Error Err = rewriteOffset(Writer, ParamsOffset[I]))
9189
return Err;
92-
It++;
90+
91+
const auto &P = Parameters[I];
9392

9493
switch (P.ParameterType) {
9594
case dxbc::RootParameterType::Constants32Bit: {
@@ -105,8 +104,6 @@ Error RootSignatureDesc::write(raw_ostream &OS) const {
105104
}
106105
}
107106

108-
assert(It == NumParameters);
109-
110107
llvm::ArrayRef<char> BufferRef(reinterpret_cast<char *>(Buffer.data()),
111108
Buffer.size());
112109
OS.write(BufferRef.data(), BufferRef.size());

0 commit comments

Comments
 (0)