Oliver Stöneberg
Oliver Stöneberg
I filed https://github.com/llvm/llvm-project/issues/83732 about the `modernize-use-designated-initializers` false positives.
This needs tickets filed about the `performance-unnecessary-copy-initialization` case which are not detected by us.
`CheckOther::constVariableError()` - https://trac.cppcheck.net/ticket/12925 `ProjectFileDialog::loadFromProjectFile()` - https://trac.cppcheck.net/ticket/12926 / https://trac.cppcheck.net/ticket/12934 The one in `isTemporary()` looks like a variation on this.
Clang 19 release candidate has been released and the apt mirror should also have made the branch switch. As we only utilize the binaries in the CI we can merge...
I filed https://github.com/llvm/llvm-project/issues/101181 upstream about the `performance-unnecessary-value-param` false positive.
@pfultz2 Please have a look. This is the initial approach which works quite well. I already did further functions up to the first more complex case `valueFlowForwardLifetime()` were it gets...
I am reworking this a bit and will fix the matchcompiler and CMake.
Still needs matchcompiler support. Also need to make sure it doesn't impact the performance negatively now the code is no longer completely visible to some of the callers.
> > I will also get rid of the using namespace ValueFlow within the split files since it makes things harder to understand when the increased granularity kicks in. >...
This currently doesn't play well with the matchcompiler. I have it working in CMake but the regular make is still an issue. It also seems there's some code in it...