[x265] [PATCH] count_nonzero primitive, downscaling quantCoeff from int32_t* to int16_t*

[chen]

>-        X265_CHECK((int)numSig == primitives.count_nonzero(coeff, 1 << log2TrSize * 2), "numSig differ\n");
>+        /* This section of code is to safely convert int32_t coefficients to int16_t, once the caller function is
>+         * optimize to take coefficients as int16_t*, it will be cleanse.*/
>+        int numCoeff = (1 << (log2TrSize * 2));
>+        assert(numCoeff <= 1024);
>+        ALIGN_VAR_16(int16_t, qCoeff[32 * 32]);
>+        for (int i = 0; i < numCoeff; i++)
>+        {
>+            qCoeff[i] = (coeff[i] & 0xFFFF);
>+        }
I suggest use clip on it, to avoid value problem (eg: 0x10000 become zero) and asm instruction match to clip
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/x265-devel/attachments/20140812/404b5a97/attachment.html>