astutils.cpp: always inline isMutableExpression() with Clang

[firewave]

The function is too big to automatically be inlined. By forcing the inlining we mainly save the overhead associated with calling the function.

[firewave]

I forgot to publish the comment with the profiling data and I deleted the data. Will add it later.

[firewave]

-D__GNUC__ --check-level=exhaustive ../lib/utils.cpp

Clang 17 883,924,504 -> 862,654,217
GCC 14 887,771,241 -> 883,432,056

The example from https://trac.cppcheck.net/ticket/10765#comment:4:

Clang 17 8,155,725,690 -> 7,984,163,813
GCC 14 8,034,731,082 -> 8,002,352,248

[chrchr-github]

MSVC has __forceinline...

[firewave]

MSVC has __forceinline...

Profiling Visual Studio has been difficult for me because it is time-based and has some confusing output which is not good if you are looking at short-running examples. Also I have not set up a build utilizing Boost yet.

But it has been a while and I should give it a spin again.