[mad-dev] IMDCT MLA patch

Dan Malek dan@mvista.com
Tue, 02 Jan 2001 20:12:12 -0500


Rob Leslie wrote:
> 
> FWIW, here's a patch to use the multiply/accumulate macros in the Layer III
> IMDCT.

Cool!  I would like to request one change.  The lowest level MLA
inline functions assume a pointer to memory locations, and this isn't
the most efficient for the PowerPC implementation.  I can shave about
another 5 percent or so off the decoding time if I let the compiler
actually think these are registers, and not just memory locations.

What I would like to do is have the code in layer3.c and synth.c
do something like: mad_f_mla(hi, lo, x, y), which I can use
directly, then:
	#define mad_f_mla(hi, lo, x, y) mad_f_amla(&(hi), &(lo), x, y)
and change the names of the processor accelerated inlines to be
"mad_f_amla" instead of "mad_f_mla".

Does this make sense?  Sorry to be a pain about this........


	-- Dan