Hi, Le 3 févr. 2015 18:47, "James Almer" <jamr...@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 "cleanup" commit? > because if at some point we add an avx version (just to take advantage of the > non-destructive three operand format compared to sse4), they will have to be replaced. 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. _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
