Re: [libav-devel] [PATCH 1/1] x86: checkasm: check for or handle missing cleanup after MMX instructions

2015-12-22 Thread Janne Grunau
On 2015-12-22 18:21:44 +0100, Henrik Gramner wrote: > On Fri, Dec 11, 2015 at 6:40 PM, Janne Grunau wrote: > > +#define declare_new_emms(cpu_flags, ret, ...) \ > > +ret (*checked_call)(void *, int, int, int, int, int, __VA_ARGS__) = \ > > +((cpu_flags) &

Re: [libav-devel] [libav-commits] checkasm: add fmtconvert tests

2015-12-22 Thread Janne Grunau
On 2015-12-22 18:37:30 +0100, Henrik Gramner wrote: > On Tue, Dec 22, 2015 at 5:41 PM, Janne Grunau wrote: > > I found HTML copy from 1999 of Intel's manual(1) which says that > > cvtpi2ps with a memory location as source doesn't cause a transition to > > MMX state. The

[libav-devel] [PATCH 1/2] x86: zero extend the 32-bit length in int32_to_float_fmul_scalar implicitly

2015-12-22 Thread Janne Grunau
This reverts commit 5dfe4edad63971d669ae456b0bc40ef9364cca80. --- libavcodec/x86/fmtconvert.asm | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/libavcodec/x86/fmtconvert.asm b/libavcodec/x86/fmtconvert.asm index b9a78a5..0383322 100644 --- a/libavcodec/x86/fmtconvert.asm

[libav-devel] [PATCH 2/2] checkasm: x86: post commit review fixes

2015-12-22 Thread Janne Grunau
Check the full FPU tag word instead of only the upper half and simplify the comparison. Use upper-case function base name as macro name to instantiate both checked_call variants. --- tests/checkasm/x86/checkasm.asm | 20 +--- 1 file changed, 9 insertions(+), 11 deletions(-) diff

Re: [libav-devel] [PATCH 5/6] Use avpriv_report_missing_feature() where appropriate

2015-12-22 Thread Diego Biurrun
On Thu, Dec 17, 2015 at 10:28:58AM -0500, Vittorio Giovara wrote: > On Thu, Dec 17, 2015 at 7:28 AM, Diego Biurrun wrote: > > --- a/libavcodec/alac.c > > +++ b/libavcodec/alac.c > > @@ -266,7 +266,7 @@ static int decode_element(AVCodecContext *avctx, > > AVFrame *frame, int

Re: [libav-devel] [libav-commits] checkasm: add fmtconvert tests

2015-12-22 Thread Janne Grunau
On 2015-12-21 19:06:15 +0100, Janne Grunau wrote: > Module: libav > Branch: master > Commit: 489e6add4478b0f5717dbf644234c6f3a3baf02c > > Author:Janne Grunau > Committer: Janne Grunau > Date: Tue Dec 8 16:24:57 2015 +0100 > > checkasm:

Re: [libav-devel] [PATCH 1/1] x86: checkasm: check for or handle missing cleanup after MMX instructions

2015-12-22 Thread Henrik Gramner
On Fri, Dec 11, 2015 at 6:40 PM, Janne Grunau wrote: > +#define declare_new_emms(cpu_flags, ret, ...) \ > +ret (*checked_call)(void *, int, int, int, int, int, __VA_ARGS__) = \ > +((cpu_flags) & av_get_cpu_flags()) ? (void > *)checkasm_checked_call_emms : \ >

Re: [libav-devel] [libav-commits] checkasm: add fmtconvert tests

2015-12-22 Thread Henrik Gramner
On Tue, Dec 22, 2015 at 5:41 PM, Janne Grunau wrote: > I found HTML copy from 1999 of Intel's manual(1) which says that > cvtpi2ps with a memory location as source doesn't cause a transition to > MMX state. The current documentation for cvtpi2pd (packed int to packed >