refs #10765/#11262 - Token: introduced cache for isAliasOf() calls

[firewave]

No description provided.

[firewave]

Using the code from https://trac.cppcheck.net/ticket/10765#comment:4:
Clang 21 - 2,741,990,149 -> 2,438,803,169
GCC 15 - 2,842,020,801 -> 2,487,141,306

The actual amount of isAliasOf() calls is reduced from 2,593,432 (16% of Ir) to 23,464 (0,2% of Ir).

There is newly introduced overhead in Token::isAliasOf() (over 5% of Ir).

[firewave]

Running it on https://github.com/firewave/mame_regtest/blob/master/mame_regtest.c provides better results:
Clang 21 - 20,959,710,351-> 15,716,108,542
GCC 15 - 20,784,491,178 -> 15,732,199,035

[firewave]

This causes false positives in the selfcheck:

lib/symboldatabase.cpp:890:17: style: Variable 'scope' can be declared as reference to const [constVariableReference]
for (Scope& scope : scopeList) {
^
lib/symboldatabase.cpp:945:17: style: Variable 'scope' can be declared as reference to const [constVariableReference]
for (Scope& scope : scopeList) {
^

[firewave]

No differences with #7800.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[firewave]

Unfortunately it does not seem to help much with the runtime of the CI.

Looks better now - CI seemed to be a tad slower in the previous case.

Before:
Check time: cli/cmdlineparser.cpp: 23m 50.534s

After:
Check time: cli/cmdlineparser.cpp: 18m 17.478s

[firewave]

This causes false positives in the selfcheck:

The problem is that it is called in ValueFlow creation but also depends on the ValueFlow values - so it cannot be cached. Similar to #7573 .