Skip to content

Conversation

firewave
Copy link
Collaborator

No description provided.

@firewave
Copy link
Collaborator Author

firewave commented Sep 14, 2025

Analyzing the code from https://trac.cppcheck.net/ticket/12861:

Clang 20 - 11,437,539,974 -> 3,423,215,226 -> 737,254,030

This needs more testing as it might be detrimental in cases which are not affected by excessive astTop() calls.

@chrchr-github
Copy link
Collaborator

Can we reduce the timeout in the #12847 test with this?

@firewave
Copy link
Collaborator Author

Can we reduce the timeout in the #12847 test with this?

Yes. I forgot to add the reference for the code I tested it with.

@firewave
Copy link
Collaborator Author

Clang 20 - 11,437,539,974 -> 3,423,215,226 -> 737,254,030

The second drop does not occur. It was a mistake in an intermediate version.

@firewave
Copy link
Collaborator Author

Clang 20 - 11,437,539,974 -> 3,423,215,226 -> 737,254,030

The second drop does not occur. It was a mistake in an intermediate version.

There's a way to address it anyways but let's do that in a separate PR since it also might have detrimental effects on other cases.

Copy link

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.

2 participants