[Bug 773073] audioconvert: endian conversion optimization

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=773073

--- Comment #32 from Sebastian Dröge (slomo) <slomo at coaxion.net> ---

(In reply to Petr Kulhavy from comment #28)
> (In reply to Sebastian Dröge (slomo) from comment #25)
> > No, see comment 21 related to this :)
> 
> That's a pity. 24-bit format is used in professional audio and 24-bit
> operations could be also useful for RGB video transformations (channel
> swapping).

Usually people use xRGB or variants of that exactly because of that reason.
It's generally faster to handle as there are instructions for doing various
things on 32 bit values, but not many for 24 bits.

> Shall I use inline assembly then? If yes is there a way to detect the target
> platform and in particular NEON with preprocessor?

You can ask ORC if NEON is supported at runtime, and for the compile-time part
take a look at what the resampler does. It IIRC has some NEON specific assembly
too already.

(In reply to Tim-Philipp Müller from comment #31)
> Cool! Out of curiosity, did you benchmark the ORC variants? Are they faster
> than what the compiler does by default with the C functions? (I guess the
> compiler won't do anything fancy here if it's not told to do so for a
> specific extension?)

Also if the ORC backup C code compiles to something as fast as the manual
implementation in C.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.