From: Henrik Gramner Date: Sat, 20 Feb 2016 15:53:35 +0000 (+0100) Subject: checkasm: Fix idct_dequant_2x4_(dc|dconly) tests X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=dbbf1dd2836a21b65178442c1fb7a00ea089d7ec;p=libx264 checkasm: Fix idct_dequant_2x4_(dc|dconly) tests They used the wrong qp values and the dconly test had the wrong name. This was undetected before because there wasn't any assembly implementations. --- diff --git a/tools/checkasm.c b/tools/checkasm.c index 7aa960d4..ab009912 100644 --- a/tools/checkasm.c +++ b/tools/checkasm.c @@ -2090,7 +2090,7 @@ static int check_quant( int cpu_ref, int cpu_new ) { set_func_name( "idct_dequant_2x4_dc_%s", i_cqm?"cqm":"flat" ); used_asms[1] = 1; - for( int qp = h->param.rc.i_qp_max; qp >= h->param.rc.i_qp_min; qp-- ) + for( int qp = h->chroma_qp_table[h->param.rc.i_qp_max]; qp >= h->chroma_qp_table[h->param.rc.i_qp_min]; qp-- ) { for( int i = 0; i < 8; i++ ) dct1[i] = rand()%(PIXEL_MAX*16*2+1) - PIXEL_MAX*16; @@ -2110,9 +2110,9 @@ static int check_quant( int cpu_ref, int cpu_new ) if( qf_a.idct_dequant_2x4_dconly != qf_ref.idct_dequant_2x4_dconly ) { - set_func_name( "idct_dequant_2x4_dc_%s", i_cqm?"cqm":"flat" ); + set_func_name( "idct_dequant_2x4_dconly_%s", i_cqm?"cqm":"flat" ); used_asms[1] = 1; - for( int qp = h->param.rc.i_qp_max; qp >= h->param.rc.i_qp_min; qp-- ) + for( int qp = h->chroma_qp_table[h->param.rc.i_qp_max]; qp >= h->chroma_qp_table[h->param.rc.i_qp_min]; qp-- ) { for( int i = 0; i < 8; i++ ) dct1[i] = rand()%(PIXEL_MAX*16*2+1) - PIXEL_MAX*16;