[x265] [PATCH] cleanup: align NR buffer for asm, rearrange member variables to avoid padding

[chen]

At 2015-09-28 17:06:30,dnyaneshwar at multicorewareinc.com wrote:
># HG changeset patch
># User Dnyaneshwar G dnyaneshwar at multicorewareinc.com>
># Date 1443431081 -19800
>#      Mon Sep 28 14:34:41 2015 +0530
># Node ID b5bd2f7a29fdc602b2a9f6c6eee68e3e85a7ea0b
># Parent  f4c267f28487161fa78c43cabb30dc4f4f82570c
>cleanup: align NR buffer for asm, rearrange member variables to avoid padding
>
diff -r f4c267f28487 -r b5bd2f7a29fd source/encoder/encoder.h
>--- a/source/encoder/encoder.h	Mon Sep 28 13:38:33 2015 +0530
>+++ b/source/encoder/encoder.h	Mon Sep 28 14:34:41 2015 +0530
>@@ -45,10 +45,10 @@
>     double        m_psnrSumV;
>     double        m_globalSsim;
>     double        m_totalQp;
>+    double        m_maxFALL;
>     uint64_t      m_accBits;
>     uint32_t      m_numPics;
>     uint16_t      m_maxCLL;
>-    double        m_maxFALL;
> 
>     EncStats()
>     {
>@@ -79,62 +79,58 @@
> {
> public:
> 
>+    ALIGN_VAR_16(uint32_t, m_residualSumEmergency[MAX_NUM_TR_CATEGORIES][MAX_NUM_TR_COEFFS]);
for AVX2 and/or cache line align, I guess ALIGN_VAR_32 or ALIGN_VAR_64 are better.

>+    uint32_t           m_countEmergency[MAX_NUM_TR_CATEGORIES];
suggest use ALIGN_VAR to force alignment


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