Skip to content

Commit ebab0ca

Browse files
author
Finn Plummer
committed
self-review: use consumeExpectedToken api to report unexpected_end_of_params diag
1 parent 180c33c commit ebab0ca

File tree

1 file changed

+9
-11
lines changed

1 file changed

+9
-11
lines changed

clang/lib/Parse/ParseHLSLRootSignature.cpp

Lines changed: 9 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -43,12 +43,11 @@ bool RootSignatureParser::parse() {
4343
break;
4444
}
4545

46-
if (!tryConsumeExpectedToken(TokenKind::end_of_stream)) {
47-
getDiags().Report(CurToken.TokLoc, diag::err_hlsl_unexpected_end_of_params)
48-
<< /*expected=*/TokenKind::end_of_stream
49-
<< /*param of=*/TokenKind::kw_RootSignature;
46+
if (consumeExpectedToken(TokenKind::end_of_stream,
47+
diag::err_hlsl_unexpected_end_of_params,
48+
/*param of=*/TokenKind::kw_RootSignature))
5049
return true;
51-
}
50+
5251
return false;
5352
}
5453

@@ -74,12 +73,10 @@ bool RootSignatureParser::parseDescriptorTable() {
7473
break;
7574
}
7675

77-
if (!tryConsumeExpectedToken(TokenKind::pu_r_paren)) {
78-
getDiags().Report(CurToken.TokLoc, diag::err_hlsl_unexpected_end_of_params)
79-
<< /*expected=*/TokenKind::pu_r_paren
80-
<< /*param of=*/TokenKind::kw_DescriptorTable;
76+
if (consumeExpectedToken(TokenKind::pu_r_paren,
77+
diag::err_hlsl_unexpected_end_of_params,
78+
/*param of=*/TokenKind::kw_DescriptorTable))
8179
return true;
82-
}
8380

8481
Elements.push_back(Table);
8582
return false;
@@ -132,7 +129,7 @@ bool RootSignatureParser::parseDescriptorTableClause() {
132129
return true;
133130

134131
if (consumeExpectedToken(TokenKind::pu_r_paren, diag::err_hlsl_unexpected_end_of_params,
135-
ParamKind))
132+
/*param of=*/ParamKind))
136133
return true;
137134

138135
Elements.push_back(Clause);
@@ -279,6 +276,7 @@ bool RootSignatureParser::consumeExpectedToken(TokenKind Expected,
279276
case diag::err_expected:
280277
DB << Expected;
281278
break;
279+
case diag::err_hlsl_unexpected_end_of_params:
282280
case diag::err_expected_either:
283281
case diag::err_expected_after:
284282
DB << Expected << Context;

0 commit comments

Comments
 (0)