[all-commits] [llvm/llvm-project] 95166f: remove confusing hlsl_unexpected_end_of_params dia...
Finn Plummer via All-commits
all-commits at lists.llvm.org
Wed Jul 9 14:25:33 PDT 2025
Branch: refs/heads/users/inbelic/pr-147800
Home: https://github.com/llvm/llvm-project
Commit: 95166f5e2bfb2816ce39d3abd1823486742109bd
https://github.com/llvm/llvm-project/commit/95166f5e2bfb2816ce39d3abd1823486742109bd
Author: Finn Plummer <canadienfinn at gmail.com>
Date: 2025-07-09 (Wed, 09 Jul 2025)
Changed paths:
M clang/include/clang/Basic/DiagnosticParseKinds.td
M clang/lib/Parse/ParseHLSLRootSignature.cpp
M clang/test/SemaHLSL/RootSignature-err.hlsl
M clang/unittests/Parse/ParseHLSLRootSignatureTest.cpp
Log Message:
-----------
remove confusing hlsl_unexpected_end_of_params diagnostic
- It has been noted more than once that this diagnostic is confusing and
misleading. It can be more concise
Commit: 24a2bea7fa745f3909f632369e8f8aaa7fff0b2a
https://github.com/llvm/llvm-project/commit/24a2bea7fa745f3909f632369e8f8aaa7fff0b2a
Author: Finn Plummer <canadienfinn at gmail.com>
Date: 2025-07-09 (Wed, 09 Jul 2025)
Changed paths:
M clang/include/clang/Basic/DiagnosticParseKinds.td
M clang/include/clang/Parse/ParseHLSLRootSignature.h
M clang/lib/Parse/ParseHLSLRootSignature.cpp
M clang/test/SemaHLSL/RootSignature-err.hlsl
M clang/unittests/Parse/ParseHLSLRootSignatureTest.cpp
Log Message:
-----------
add invalid_param diagnostic to differentiate between missing comma vs invalid token
Commit: 4edb7fa000026935accfd993ff4ca1f4eed68c2e
https://github.com/llvm/llvm-project/commit/4edb7fa000026935accfd993ff4ca1f4eed68c2e
Author: Finn Plummer <canadienfinn at gmail.com>
Date: 2025-07-09 (Wed, 09 Jul 2025)
Changed paths:
M clang/test/SemaHLSL/RootSignature-err.hlsl
Log Message:
-----------
keep test of missing comma
Commit: b17b40c5b6f8e68a83f8ad7cedc3c0a51589c945
https://github.com/llvm/llvm-project/commit/b17b40c5b6f8e68a83f8ad7cedc3c0a51589c945
Author: Finn Plummer <canadienfinn at gmail.com>
Date: 2025-07-09 (Wed, 09 Jul 2025)
Changed paths:
M clang/lib/Parse/ParseHLSLRootSignature.cpp
M clang/test/SemaHLSL/RootSignature-err.hlsl
Log Message:
-----------
audit use of err_expected_after
the order of the parameters is switched as opposed to either and this
was not caught in the unit tests
Commit: bbd8921c962a6982f10881257a3ec37d6f2d2b90
https://github.com/llvm/llvm-project/commit/bbd8921c962a6982f10881257a3ec37d6f2d2b90
Author: Finn Plummer <canadienfinn at gmail.com>
Date: 2025-07-09 (Wed, 09 Jul 2025)
Changed paths:
M clang/include/clang/Basic/DiagnosticParseKinds.td
M clang/lib/Parse/ParseHLSLRootSignature.cpp
M clang/unittests/Parse/ParseHLSLRootSignatureTest.cpp
Log Message:
-----------
nfc: prepare to extend diagnostic to values as well
Commit: c3a4642c5869324a88b70ed867b0ee62da4ff8e7
https://github.com/llvm/llvm-project/commit/c3a4642c5869324a88b70ed867b0ee62da4ff8e7
Author: Finn Plummer <canadienfinn at gmail.com>
Date: 2025-07-09 (Wed, 09 Jul 2025)
Changed paths:
M clang/include/clang/Parse/ParseHLSLRootSignature.h
M clang/lib/Parse/ParseHLSLRootSignature.cpp
M clang/test/SemaHLSL/RootSignature-err.hlsl
M clang/unittests/Parse/ParseHLSLRootSignatureTest.cpp
Log Message:
-----------
extend invalid_token diagnostic to enum parsing
Commit: b32343063a62681e3aefd46d1b21018c84476e26
https://github.com/llvm/llvm-project/commit/b32343063a62681e3aefd46d1b21018c84476e26
Author: Finn Plummer <canadienfinn at gmail.com>
Date: 2025-07-09 (Wed, 09 Jul 2025)
Changed paths:
M clang/include/clang/Parse/ParseHLSLRootSignature.h
M clang/lib/Parse/ParseHLSLRootSignature.cpp
M clang/test/SemaHLSL/RootSignature-err.hlsl
Log Message:
-----------
extend invalid_token diagnostic to flag parsing
Commit: 8187a65789e8776513e9b3ea9756da62fddd4ad7
https://github.com/llvm/llvm-project/commit/8187a65789e8776513e9b3ea9756da62fddd4ad7
Author: Finn Plummer <canadienfinn at gmail.com>
Date: 2025-07-09 (Wed, 09 Jul 2025)
Changed paths:
M clang/lib/Parse/ParseHLSLRootSignature.cpp
M clang/test/SemaHLSL/RootSignature-err.hlsl
Log Message:
-----------
extend invalid_token diagnostic to RootFlags enum
Commit: 6d48044edf53d679d9c6a36d88219b401d05944d
https://github.com/llvm/llvm-project/commit/6d48044edf53d679d9c6a36d88219b401d05944d
Author: Finn Plummer <canadienfinn at gmail.com>
Date: 2025-07-09 (Wed, 09 Jul 2025)
Changed paths:
M clang/lib/Parse/ParseHLSLRootSignature.cpp
M clang/test/SemaHLSL/RootSignature-err.hlsl
M clang/unittests/Parse/ParseHLSLRootSignatureTest.cpp
Log Message:
-----------
improve err_expected to err_expected_either at end of params
Commit: 522b08467c40282feb8566b648b8c16f86232942
https://github.com/llvm/llvm-project/commit/522b08467c40282feb8566b648b8c16f86232942
Author: Finn Plummer <canadienfinn at gmail.com>
Date: 2025-07-09 (Wed, 09 Jul 2025)
Changed paths:
M clang/unittests/Parse/ParseHLSLRootSignatureTest.cpp
Log Message:
-----------
improve unit test coverge of invalid values
Commit: 70448a3e3daae9ebac71df5b995bc93025b7a644
https://github.com/llvm/llvm-project/commit/70448a3e3daae9ebac71df5b995bc93025b7a644
Author: Finn Plummer <canadienfinn at gmail.com>
Date: 2025-07-09 (Wed, 09 Jul 2025)
Changed paths:
M clang/unittests/Parse/ParseHLSLRootSignatureTest.cpp
Log Message:
-----------
improve unit test coverage of invalid parameters
Compare: https://github.com/llvm/llvm-project/compare/95166f5e2bfb%5E...70448a3e3daa
To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications
More information about the All-commits
mailing list