[x265] [PATCH 1 of 2] primitives: added C primitives to compute SSIM

[Steve Borho]

On Fri, Oct 4, 2013 at 11:07 PM, Aarthi Priya Thirumalai <
aarthi at multicorewareinc.com> wrote:

> I understand. If High_Bit)Depth = 1, you could still choose uint32 version
> for the primitives  but in ssim_end_1(int..) you might get an overflow
> when bitdepth > 9, as mentioned in the comments there. I could use a 64bit
> int there for all cases to store intermediary calculations , if you want to
> remove those if conditions.
>

If uint64 is big enough for 12bit pixels, then that would be preferable
over floats, I think.

But we would still want a 32bit int version for the 8bit pixel case, so you
would still need a run-time decision to pick the right version of the
function.

Am I misunderstanding the need for the float version? Is it just for higher
pixel bit depths?

--
Steve
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/x265-devel/attachments/20131005/c38192a3/attachment.html>