[FFmpeg-devel] [PATCH] x86: hevc_mc: remove non necessary moves
christophe.gisquet at gmail.com
Tue Feb 3 20:04:39 CET 2015
Le 3 févr. 2015 18:47, "James Almer" <jamrial at gmail.com> a écrit :
> On 02/02/15 2:11 PM, Christophe Gisquet wrote:
> > @@ -87,11 +95,22 @@ QPEL_TABLE 12, 4, w, sse4
> > %elif %1 <= 8
> > movdqa %3,
[%2] ; load data from source2
> > %elif %1 <= 12
> > +%if avx_enabled
> If this is meant for avx2, then the correct check is "cpuflag(avx2)" or
"mmsize > 16",
I initially chose the former but since Michael's comment, I try avoiding
anything but strictly necessary. Not sure what to do then? Do another
> because if at some point we add an avx version (just to take advantage of
> non-destructive three operand format compared to sse4), they will have to
We can probably bench mark this now.
> Indentation for all the stuff bellow is wrong.
Nearly same reason. The original code doesn't check x86_64, which I added.
I didn't want to change the indentation. But it's rather ridiculous here.
More information about the ffmpeg-devel