tarantool / tarantool / 18684469233

88%

push

github

perf: fix `perf/tuple.cc` benchmark

The commit fixes several tuple benchmark problems (and potential
unwanted optimization is among them):
1. There was `std::vector<char[...]>` construction - it looks monstrous
   and actually breaks build on some platforms (MacOS, for example)
   because of destructor problems. Let's replace it with `std::array`,
   it has the same semantics and is actually the right way to go because
   it is a C++ class and has a defined destructor.
2. There is an annoying warning complaining that `sprintf` is deprecated,
   so let's simply replace it with `snprintf`.
3. There are warnings regarding `DoNotOptimize` usage - its const-reference
   overload is deprecated because it can be actually optimized by compiler.
   Let's save expression results to local variables so that non-const
   reference overload is used.

NO_TEST=perf
NO_CHANGELOG=perf
NO_DOC=perf

69852 of 117663 branches covered (59.37%)

104831 of 119626 relevant lines covered (87.63%)

1371891.92 hits per line