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

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