Skip to content

Conversation

@firewave
Copy link
Collaborator

@firewave firewave commented Sep 27, 2024

  • added CMake target run-clang-tidy-csa to run Clang Static Analyzer
  • added CMake option ENABLE_CSA_ALPHA to enable the alpha checkers

@firewave
Copy link
Collaborator Author

As it wrecks havoc on the performance (I did not remember it being so bad) we should limit this to the CI so it won't kill the usability IDEs which support clang-tidy.
of

[ 59/240][305.7s] clang-tidy-19 -p=/home/runner/work/cppcheck/cppcheck/cmake.output -quiet /home/runner/work/cppcheck/cppcheck/test/testother.cpp

OTOH it is a bit of relieve to see that other dataflow implementations also seem to struggle with some "heavy" patterns.

@firewave
Copy link
Collaborator Author

OTOH it is a bit of relieve to see that other dataflow implementations also seem to struggle with some "heavy" patterns.

It appears they are also slow with big if-else chains.

@firewave
Copy link
Collaborator Author

firewave commented Oct 18, 2024

Some of the times are misleading because some of the files are analyzed more than once because of llvm/llvm-project#73357 (comment).

@firewave

This comment was marked as outdated.

@firewave firewave force-pushed the ci-tidy-static branch 2 times, most recently from 5f60dc7 to 0828a83 Compare January 28, 2025 21:46
@firewave firewave changed the title enabled clang-analyzer-* clang-tidy warnings perform scheduled clang-tidy analysis with clang-analyzer-* warnings enabled Jan 28, 2025
@firewave firewave force-pushed the ci-tidy-static branch 4 times, most recently from 240a115 to 1af19f2 Compare January 29, 2025 08:24
@firewave
Copy link
Collaborator Author

I filed llvm/llvm-project#132001 about the clang-analyzer-optin.cplusplus.UninitializedObject false positives with unnamed fields used for explicit padding.

@firewave

This comment was marked as resolved.

@firewave firewave force-pushed the ci-tidy-static branch 4 times, most recently from 39bddd1 to e2c66e7 Compare March 19, 2025 11:36
@firewave firewave changed the title perform scheduled clang-tidy analysis with clang-analyzer-* warnings enabled perform scheduled Clang Static Analyzer run Mar 19, 2025
@firewave
Copy link
Collaborator Author

It would be great to have a badge for this run on the main page but I have no idea how to differentiate the scheduled run from a regular one.

@firewave firewave changed the title perform scheduled Clang Static Analyzer run added scheduled Clang Static Analyzer run Mar 19, 2025
@firewave firewave marked this pull request as ready for review March 19, 2025 22:57
@firewave firewave force-pushed the ci-tidy-static branch 2 times, most recently from 3c2722d to 00a7c95 Compare March 23, 2025 23:42
@firewave firewave merged commit 7583df8 into danmar:main Mar 28, 2025
60 checks passed
@firewave firewave deleted the ci-tidy-static branch March 28, 2025 10:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant