Skip to content

Conversation

@firewave
Copy link
Collaborator

No description provided.

@firewave
Copy link
Collaborator Author

firewave commented Jan 21, 2026

This also gets rid of the Clang-only hack I introduced a while ago.

Using the code from https://trac.cppcheck.net/ticket/10765#comment:4:
Clang 21 - 2,713,071,190 -> 2,616,154,402
GCC 15 - 2,918,649,065 -> 2,718,303,437

Running it on https://github.com/firewave/mame_regtest/blob/master/mame_regtest.c :
Clang 21 - 20,959,594,045-> 20,008,068,513
GCC 15 - 22,603,210,883 -> 20,663,260,243

There is newly introduced overhead in Token::isMutableExpr() (about 2% of Ir).

@firewave
Copy link
Collaborator Author

Appears to have a massive impact on the CI.

Before:
Check time: cli/cmdlineparser.cpp: 40m 55.757s

After:
Check time: cli/cmdlineparser.cpp: 23m 17.996s

@firewave
Copy link
Collaborator Author

No differences with #7800.

@firewave firewave marked this pull request as ready for review January 21, 2026 20:12
@sonarqubecloud
Copy link

@chrchr-github chrchr-github merged commit 31da8a2 into danmar:main Jan 26, 2026
66 checks passed
@firewave firewave deleted the mutexpr branch January 26, 2026 12:23
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