Commit b78ac50
Add missing diagnostic argument (microsoft#7426)
Two instances of the err_integer_literal_too_large diagnostic in HLSL
specific code within Sema::ActOnNumericConstant() had a missing
argument. When these diagnostics were raised this caused an assert in an
assert enabled DXC, and random corruption of the diagnostic text in a
non-assert enabled DXC.
The trivial fix is to supply the required argument.
Fixes microsoft#7425
Co-authored-by: Tim Corringham <[email protected]>1 parent 5aec1ec commit b78ac50
2 files changed
Lines changed: 18 additions & 2 deletions
File tree
- tools/clang
- lib/Sema
- test/HLSLFileCheck/hlsl/diagnostics/errors
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
3504 | 3504 | | |
3505 | 3505 | | |
3506 | 3506 | | |
3507 | | - | |
| 3507 | + | |
| 3508 | + | |
3508 | 3509 | | |
3509 | 3510 | | |
3510 | 3511 | | |
3511 | 3512 | | |
3512 | | - | |
| 3513 | + | |
| 3514 | + | |
3513 | 3515 | | |
3514 | 3516 | | |
3515 | 3517 | | |
| |||
Lines changed: 14 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
0 commit comments