[FFmpeg-soc] [soc]: r5127 - als/alsdec.c

[Reimar Döffinger]

On Sat, Aug 15, 2009 at 03:46:57PM +0200, Thilo Borgmann wrote:
> 
> >> smp is the loop counter and can therefore not be put outside the loop.
> >> For the convert assignment, this could have been true but since there
> >> have to be two if's (if smp==progressive  and if smp < progressive), it
> >> has shown to be even slower than the two loop version.
> > 
> > Not quite, you could move the convert = 1 and only that outside the
> > loop.
> > Maybe not really worth it though, it might use up one more register in
> > the loop...
> 
> I don't have an explanation, but that way benchmarked worse.

It obfuscates things a bit for the compiler and probably it misses some
optimization opportunities.
You might see if this is better, but the variable "convert" should
probably be named differently then
convert = smp < progressive;
max = convert ? smp : progressive;