aboutsummaryrefslogtreecommitdiffstats
path: root/tests/ref/fate/matroska-zlib-decompression
diff options
context:
space:
mode:
authorAndreas Rheinhardt <andreas.rheinhardt@outlook.com>2021-04-14 20:00:12 +0200
committerAndreas Rheinhardt <andreas.rheinhardt@outlook.com>2021-04-18 04:27:19 +0200
commit5627da3555ecbf6fbfd98d045abbc7b63bc98fce (patch)
tree0da683d23e549754594b2655318fca8e29166a7f /tests/ref/fate/matroska-zlib-decompression
parente1f41b4fdc5d3e8813d8f9d4d81899bd4e6df31b (diff)
downloadffmpeg-5627da3555ecbf6fbfd98d045abbc7b63bc98fce.tar.gz
avformat/matroskaenc: Remove unnecessary function calls
ffio_fill() is used when initially writing unknown length elements; yet it can happen that the amount of bytes written by it is zero in which case it is of course unnecessary to ever call it. Whether it is possible to know this during compiletime depends upon how aggressively the compiler inlines function calls (i.e. if it inlines calls to start_ebml_master() where the upper bound for the size of the element implies that the size will be written on one byte) and this depends upon optimization settings. It is not the aim of this patch to inline all calls where it is known that ffio_fill() will be unnecessary, but merely to make compilers that inline such calls aware of the fact that writing zero bytes with ffio_fill() is unnecessary. To this end av_builtin_constant_p() is used to check whether the size is a compiletime constant. For GCC 10 this made a difference at -O3 only: The size of .text decreased from 0x747F (with 29 calls to ffio_fill(), eight of which use size zero) to 0x7337 (with 21 calls to ffio_fill(), zero of which use size zero). For Clang 11 it made a difference at -O2 and -O3: At -O2, the size of .text decreased from 0x879C to 0x871C (with eight calls to ffio_fill() eliminated); at -O3 the size of .text decreased from 0xAF2F to 0xAEBF. Once again, eight calls to ffio_fill() with size zero have been eliminated. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
Diffstat (limited to 'tests/ref/fate/matroska-zlib-decompression')
0 files changed, 0 insertions, 0 deletions