[opus] [Opus] [Patch]01-Add ARM5E macros

Timothy B. Terriberry tterribe at xiph.org
Sun May 19 20:21:45 PDT 2013


Timothy B. Terriberry wrote:
> Aurélien Zanelli wrote:
>> Also, if you have any question or comment about it, feel free to contact
>> me.
>
> A lot of these macros consist of a single ARM instruction, yet specify
> an early clobber constraint for the output. Is that intentional?

Apparently it was Jean-Marc who added those constraints in the original 
file in libspeex (though he couldn't remember why). In any case, I 
committed a modified version of your patch:

https://git.xiph.org/?p=opus.git;a=commitdiff;h=972a34ec

Total speed up was 22.3% on test_opus_encoder on a 600 MHz Cortex A8 
using gcc 4.2.1 (there are probably more representative tests and 
certainly better compilers, but that should give an idea of the ballpark).


More information about the opus mailing list