[x264-devel] commit: Replace some divisions with shifts (Henrik Gramner )

[Jan Willamowius]

git at videolan.org wrote:
> x264 | branch: master | Henrik Gramner <hengar-6 at student.ltu.se> | Tue Jun  8 16:29:16 2010 +0200| [59a9e0337cf5f75046b7031c61fdd2d88fc9021c] | committer: Jason Garrett-Glaser 
> 
> Replace some divisions with shifts
[...]
> -    return i_satd / 2;
> +    return i_satd >> 1;

Did anybody check if this really makes any difference in the generated
code at all ?

I would suspect most compilers to do these optimizations for constant
values automatically. I personally would favor code readybility
over such optimizations.

Just a though,
Jan

-- 
Jan Willamowius, jan at willamowius.de, http://www.gnugk.org/