<div style="line-height:1.7;color:#000000;font-size:14px;font-family:arial"><pre><br>At 2014-08-30 13:34:16,"Satoshi Nakagawa" <nakagawa424@oki.com> wrote:
># HG changeset patch
># User Satoshi Nakagawa <nakagawa424@oki.com>
># Date 1409376721 -32400
># Sat Aug 30 14:32:01 2014 +0900
># Node ID eb94972d65473fc8e6fa42ebd95a72cfc727edf5
># Parent 4e2d9ac6d489e82e70544d626c89964ee653c452
>asm: fix dequant_normal
>
>diff -r 4e2d9ac6d489 -r eb94972d6547 source/common/dct.cpp
>--- a/source/common/dct.cpp Fri Aug 29 11:12:49 2014 +0200
>+++ b/source/common/dct.cpp Sat Aug 30 14:32:01 2014 +0900
>@@ -720,7 +720,9 @@
>
> void dequant_normal_c(const int32_t* quantCoef, int32_t* coef, int num, int scale, int shift)
> {
>-#if !HIGH_BIT_DEPTH
>+#if HIGH_BIT_DEPTH
>+ X265_CHECK(scale < 32768 || ((scale & 3) == 0 && shift > 2), "dequant invalid scale %d\n", scale);
>+#else
> // NOTE: maximum of scale is (72 * 256)
> X265_CHECK(scale < 32768, "dequant invalid scale %d\n", scale);
> #endif
</pre><pre>How about remove '#if...'?</pre><pre>The asm code didn't check it.</pre><pre>Thanks</pre><pre> </pre></div>