[FFmpeg-devel] [PATCH 02/10] x86: dcadsp: implement SSE lfe_dir
michaelni at gmx.at
Sun Apr 6 21:10:30 CEST 2014
On Sun, Apr 06, 2014 at 06:46:30PM +0200, Christophe Gisquet wrote:
> 2014-04-06 18:35 GMT+02:00 James Almer <jamrial at gmail.com>:
> > Ideally, this should be fixed on x86inc so movh expands to movq or movlps
> > depending on the requested instruction set, so this doesn't happen again.
> Exactly. Actually, I had already made this request (during my work on
> he-aac), but it was rejected, as well as this patch:
i dont remember seeing this patch before, maybe ive forgotten though
> The basic reason was "we don't want to bother with SSE in our code",
> which also meant "just drop your SSE version".
IMHO, SSE is fine if someone bothers writing the code and is willing
to maintain it
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Rewriting code that is poorly written but fully understood is good.
Rewriting code that one doesnt understand is a sign that one is less smart
then the original author, trying to rewrite it will not make it better.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 198 bytes
Desc: Digital signature
More information about the ffmpeg-devel