Oliver Stöneberg

Results 1147 comments of Oliver Stöneberg

> nice speedups! I am curious.. do you have any theory why there is speedup it just looks like a refactoring to me? The `Token::Match()` calls were not matchcompiled because...

@pfultz2 does this look okay to you?

Thanks. I think I addressed all of it.

This essentially eliminates any meaningful impact by `followAllReferences()` at all. `-D__GNUC__ --check-level=exhaustive ../lib/utils.cpp` Clang 19 `652,987,030` -> ~`624,476,510`~ `618,089,977` `followAllReferences()` calls from `isAliasOf()` - `350,100` -> `1,581` The example from...

Something really weird is going on here in the UBSAN job: ``` Check time: cli/threadexecutor.cpp: 0.53017s Check time: cli/processexecutor.cpp: 1.41327s Check time: lib/addoninfo.cpp: 0.172107s Check time: lib/analyzerinfo.cpp: 0.636273s ``` The...

Before ``` Check time: cli/cmdlineparser.cpp: 1091.73s [...] Check time: lib/checkio.cpp: 219.069s [...] Check time: lib/symboldatabase.cpp: 191.785s [...] Check time: lib/tokenize.cpp: 290.026s ``` After ``` Check time: cli/cmdlineparser.cpp: 760.299s [...] Check...

Could we merge this without the debug integration for now (I will file a ticket about doing that)? I would like to clean up and test the existing debug output...

Thanks for the feedback. Some of the shortcomings were intentional since it seemed to make things simpler and I wanted to get the performance improvement in. But I remembered the...

@pfultz2 How should the information be presented in the debug output? I also assume it should be a separate section.

> This should be done in the symboldatabase either before or during exprids. In the limited cases I tested that produced considerably more calls to `followAllReferences()` but did not affect...