[x265] Fwd: [PATCH] refactorizaton of the transform/quant path

[Praveen Tiwari]

---------- Forwarded message ----------
From: Steve Borho <steve at borho.org>
Date: Tue, Nov 18, 2014 at 11:35 PM
Subject: Re: [x265] [PATCH] refactorizaton of the transform/quant path
To: Development for x265 <x265-devel at videolan.org>


On 11/18, praveen at multicorewareinc.com wrote:
> # HG changeset patch
> # User Praveen Tiwari
> # Date 1416299427 -19800
> # Node ID 706fa4af912bc1610478de8f09a651ae3e58624c
> # Parent  2f0062f0791b822fa932712a56e6b0a14e976d91
> refactorizaton of the transform/quant path.
> This patch involves scaling down the DCT/IDCT coefficients from int32_t
to int16_t
> as they can be accommodated on int16_t without any introduction of encode
error,
> this allows us to clean up lots of DCT/IDCT intermediated buffers,
optimize enode efficiency for different
> cli options including noise reduction by reducing data movement
operations, accommodating more number of
> coefficients in a single register for SIMD operations. This patch include
all necessary
> changes for the transfor/quant path including unit test code.

Testbench failure with this patch applied:

$ ./test/TestBench
Using random seed 546B89D8 8bpp
Testing primitives: SSE2
Testing primitives: SSE3
Testing primitives: SSSE3
Testing primitives: SSE4
denoiseDct: Failed!

Mac OS X x86_64 8bpp

>>I'm going to hold this patch until you can send a new patch to resolve
>>this issue.

[MC] Can we disable this single assembly code and push the patches so that
this and other patches don't have to wait, once we done with this issue we
can enable denoise asm code.

--
Steve Borho
_______________________________________________
x265-devel mailing list
x265-devel at videolan.org
https://mailman.videolan.org/listinfo/x265-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/x265-devel/attachments/20141119/dffa57e1/attachment.html>