github-actions[bot] wrote:
<!--LLVM CODE LINT COMMENT: clang-tidy-->
:warning: C/C++ code linter, clang-tidy found issues in your code. :warning:
<details>
<summary>
You can test this locally with the following command:
</summary>
```bash
git diff -U0 origin/main...HEAD --
clang-tools-extra/clang-tidy/performance/ExpensiveValueOrCheck.cpp
clang-tools-extra/clang-tidy/performance/ExpensiveValueOrCheck.h
clang-tools-extra/clang-tidy/performance/PerformanceTidyModule.cpp |
python3 clang-tools-extra/clang-tidy/tool/clang-tidy-diff.py -path build -p1
-quiet
```
</details>
<details>
<summary>
View the output from clang-tidy here.
</summary>
```
clang-tools-extra/clang-tidy/performance/ExpensiveValueOrCheck.cpp:52:3:
warning: variable 'ValueType' of type 'QualType' can be declared 'const'
[misc-const-correctness]
52 | QualType ValueType = Call->getType().getCanonicalType();
| ^
| const
clang-tools-extra/clang-tidy/performance/ExpensiveValueOrCheck.cpp:57:3:
warning: variable 'ValueSize' of type 'int64_t' (aka 'long') can be declared
'const' [misc-const-correctness]
57 | int64_t ValueSize = Ctx.getTypeSizeInChars(ValueType).getQuantity();
| ^
| const
clang-tools-extra/clang-tidy/performance/ExpensiveValueOrCheck.cpp:58:3:
warning: variable 'IsExpensive' of type 'bool' can be declared 'const'
[misc-const-correctness]
58 | bool IsExpensive = !ValueType.isTriviallyCopyableType(Ctx) ||
| ^
| const
```
</details>
https://github.com/llvm/llvm-project/pull/200166
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
