[FFmpeg-devel] [PATCH 3/3] ac3enc: add SIMD-optimized shifting functions for use with the fixed-point AC3 encoder

[Michael Niedermayer]

On Tue, Mar 08, 2011 at 01:59:44PM -0500, Ronald S. Bultje wrote:
> Hi,
> 
> On Tue, Mar 8, 2011 at 1:18 PM, Justin Ruggles <justin.ruggles at gmail.com> wrote:
> > +%macro AC3_LSHIFT_INT16 1
> > +cglobal ac3_lshift_int16_%1, 3,3,5, src, len, shift
> > +    test   shiftd, shiftd
> > +    jz .end
> > +    movd       m0, shiftd
> > +    ALIGN 8
> > +.loop:
> > +    AC3_SHIFT_4MM srcq, psllw, m0
> > +    sub      lend, mmsize*2
> > +    ja .loop
> 
> If it's not a multiple of mmsize*2, this will loop forever. Making
> this jg (= signed) would fix that.

really?
i want that CPU on which ja behaves liek that


> Looks good otherwise.

glad we agree on that


[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The real ebay dictionary, page 2
"100% positive feedback" - "All either got their money back or didnt complain"
"Best seller ever, very honest" - "Seller refunded buyer after failed scam"
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20110308/7f5421a5/attachment.pgp>