[Bug 773073] audioconvert: endian conversion optimization

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Wed Oct 26 11:34:28 UTC 2016


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

--- Comment #45 from Petr Kulhavy <brain at jikos.cz> ---
(In reply to Sebastian Dröge (slomo) from comment #44)

> For the aliasing, actually things are better than expected. There's only a
> problem with ORC here because it uses "restrict" for the arrays (which hints
> at the compiler that they will not alias). For the normal C code there is no
> problem as the compiler is only allowed to assume no aliasing happens if
> there is "restrict", or there are two pointers of a different type (which is
> not the case here).
> 
> I would suggest that for simplicity we get rid of the ORC parts here and
> just do it all in C. Then we only need one function and not a separate one
> for in-place. And the speed difference is probably not big (right?).

Back to the roots? ;-) 
I agree with you. The performance gain is likely minimal and it's a good price
for the simplicity. Let me prepare the patch (on top of this one) then.

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


More information about the gstreamer-bugs mailing list