On Mon, May 14, 2012 at 04:27:56PM -0400, Justin Ruggles wrote:
> On 05/14/2012 04:18 PM, Måns Rullgård wrote:
> > Justin Ruggles <[email protected]> writes:
> > 
> >> This also fixes the invalid use of the pshufw operation in the MMX-only
> >> function.
> >> ---
> >>  libavcodec/x86/ac3dsp.asm   |   59 
> >> +++++++++++++++++++++++++++---------------
> >>  libavcodec/x86/ac3dsp_mmx.c |   10 +++---
> >>  2 files changed, 43 insertions(+), 26 deletions(-)
> > 
> > Please mention x86 in the subject for changes like this.
> 
> will do. amended commit message locally to:
> 
> ac3dsp: simplify x86 versions of ac3_max_msb_abs_int16
> 
> Simplifies the code by using cpuflags and a new macro.
> Also fixes the invalid use of the pshufw operation in the MMX-only
> function.

Please change this to

  Also fixes the invalid use of the MMX2 pshufw operation in the
  MMX-only function.

I tested the patch on my K6-III, the sigill is fixed.  Thanks,
please push anytime.

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to