[clang-tools-extra] [clang-tidy] Add readability-use-builtin-literals check (PR #76065)
Piotr Zegar via cfe-commits
cfe-commits at lists.llvm.org
Fri Jan 5 03:31:44 PST 2024
https://github.com/PiotrZSL requested changes to this pull request.
Overall what is now is +- working.
Few fixes are still needed to tests, code, maybe some tiny refactoring and code formatting.
Consider adding support for implicit casts:
```
unsigned X = 10;
`-VarDecl <line:1:1, col:14> col:10 X 'unsigned int' cinit
`-ImplicitCastExpr <col:14> 'unsigned int' <IntegralCast>
`-IntegerLiteral <col:14> 'int' 10
```
To:
```
unsigned X = 10U;
`-VarDecl <line:1:1, col:14> col:10 X 'unsigned int' cinit
`-IntegerLiteral <col:14> 'unsigned int' 10
```
I worry also a little bit about integer overflow issues.
https://github.com/llvm/llvm-project/pull/76065
More information about the cfe-commits
mailing list