tarantool / tarantool / 18749378732 / 1

88%

master: 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

(cherry picked from commit a854a6519)

68719 of 116259 branches covered (59.11%)

103443 of 118119 relevant lines covered (87.58%)

1602439.68 hits per line