[Speex-dev] [PATCH] compute_weighted_codebook a little bit faster

Jean-Marc Valin Jean-Marc.Valin at USherbrooke.ca
Fri Dec 9 18:14:32 PST 2005


> here is a patch making the function compute_weighted_codebook a little
> bit faster. Not so impressive but avoid a loop and is really faster on
> small platforms like the MIPS I'm working on.

I see your patch does two things. First, it defined 
sbv_size = i*subvect_size;
which shouldn't change anything with any compiler that can do CSE. The
second is to not copy  shape_cb[] into shape[]. This is wrong because
MAC16_16() requires 16-bit inputs. Also, most DSPs are slower at
loading/multiplying chars than shorts.

BTW, have you tested your LSP patch on fixed-point yet?

	Jean-Marc


More information about the Speex-dev mailing list