[FFmpeg-devel] [PATCH] SSE2 version of vf_idet's filter_line()

[Pascal Massimino]

Hi,


updated patch, sorry for the broken format in the previous one. Hope it's
ok now.


On Wed, Sep 3, 2014 at 1:08 AM, James Darnley <james.darnley at gmail.com>
wrote:

> On 2014-09-02 17:13, Pascal Massimino wrote:
> > %macro DECLARE_VAR 2
> >     %define  %1 %2
> >     %define  %1d %2d
> > %endmacro
> >
> > ;; Mappings for common variables
> > DECLARE_VAR index, r5
> > DECLARE_VAR total, r4
> > DECLARE_VAR parallel_width, r6
>
> I suggest you leave these up to the cglobal macro
>
> > DECLARE_VAR tmp1, r7
> > DECLARE_VAR tmp2, r6
>
> Perhaps these two should use DECLARE_REG_TMP
>

Ok, i axed the left-over thing and make used for the c-version for
left-overs. Simpler.
x8inc.asm is quite hairy. Hope i used 'cglobal' right now.

/skal


> > movsxd
>
> Use x264asm's movsxifnidn macro, then it will only do something on 64-bit.
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-MMX-MMXEXT-SSE2-implementation-of-idet-s-filter_line.patch
Type: text/x-patch
Size: 3760 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140903/904d9bea/attachment.bin>