On Sat, Aug 02, 2014 at 08:30:39PM +0200, Lukasz Marek wrote:
> >+static av_always_inline int sad_wxh(const uint8_t *src1, ptrdiff_t stride1,
> >+                                    const uint8_t *src2, ptrdiff_t stride2,
> >+                                    int w, int h)
> >+{
> >+    int x, y, sum = 0;
> >+
> >+    for (y = 0; y < h; y++) {
> >+        for (x = 0; x < w; x++)
> >+            sum += abs(src1[x] - src2[x]);
> 
> minor: I came out of nowhere to this patch, but maybe FFABS? :)
> 

No difference in generated ASM here

-- 
Clément B.

Attachment: pgpFDI_gArcbX.pgp
Description: PGP signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to