Conversation
CodSpeed Performance ReportMerging #5483 will not alter performanceComparing Summary
Footnotes
|
anonrig
left a comment
There was a problem hiding this comment.
is there a test that verifies this, so we don't regress?
I found this using Clang Static Analyzer's cplusplus.Move check, but that is very slow and has several false positives in our code base, so not something we can enable in CI. We do have a clang-tidy check enabled that's also supposed to find UaM, but that wasn't sufficient here. Effectively, we don't have a way to avoid this except for good test coverage and careful PR reviews. |
No description provided.