simplecpp icon indicating copy to clipboard operation
simplecpp copied to clipboard
verified publisher icon danmar

enabled and fixed `modernize-pass-by-value` clang-tidy warnings / avoid some unnecessary copies

Open firewave opened this issue 4 months ago • 2 comments

firewave avatar Sep 01 '25 06:09 firewave

callgrind in CI

g++ - 74,990,727 -> 76,190,727 clang++ - 70,436,671 -> 71,500,835

firewave avatar Sep 01 '25 06:09 firewave

Trying to mitigate the regression makes things worse although I am able to reduce the amount of std::string copies. There is still memcpy() calls I do not even understand where they are coming from.

firewave avatar Sep 01 '25 07:09 firewave