commit 28744b5c7db284919d46624457e7a4925c15a67d
author Peng Bin <binpengsmail@gmail.com> Mon Feb 12 11:49:11 2018 +0800
committer Zoe Liu <zoeliu@google.com> Wed Feb 14 21:15:28 2018 +0000
tree f46855c80ceb228bfc4d6777ac166a45bc4860fd
parent 043f49640aad8c7e6f3d60da47fb7391d61c4340

Refactor inv_cos_bit for speedup

Replace the last parameter cos_bit for all 1D inv_txfm funcions with
a macro define, as it is actually always equal to 12. By changing it
to const value, compiler can do further optimization.

Change-Id: If8a9fd99c7ac7eb6f485dafbce22b4803efda62e

av1/common/av1_inv_txfm2d.c

diff --git a/av1/common/av1_inv_txfm2d.c b/av1/common/av1_inv_txfm2d.c
index ca178a1..bbbef9f 100644
--- a/av1/common/av1_inv_txfm2d.c
+++ b/av1/common/av1_inv_txfm2d.c
@@ -93,23 +93,25 @@
 #endif  // CONFIG_TX64X64
 };
 
-const int8_t inv_cos_bit_col[MAX_TXWH_IDX /*txw_idx*/]
-                            [MAX_TXWH_IDX /*txh_idx*/] = {
-                              { 12, 12, 12, 0, 0 },
-                              { 12, 12, 12, 12, 0 },
-                              { 12, 12, 12, 12, 12 },
-                              { 0, 12, 12, 12, 12 },
-                              { 0, 0, 12, 12, 12 }
-                            };
+/* clang-format off */
+const int8_t inv_cos_bit_col[MAX_TXWH_IDX]      // txw_idx
+                            [MAX_TXWH_IDX] = {  // txh_idx
+    { INV_COS_BIT, INV_COS_BIT, INV_COS_BIT,           0,           0 },
+    { INV_COS_BIT, INV_COS_BIT, INV_COS_BIT, INV_COS_BIT,           0 },
+    { INV_COS_BIT, INV_COS_BIT, INV_COS_BIT, INV_COS_BIT, INV_COS_BIT },
+    {           0, INV_COS_BIT, INV_COS_BIT, INV_COS_BIT, INV_COS_BIT },
+    {           0,           0, INV_COS_BIT, INV_COS_BIT, INV_COS_BIT }
+  };
 
-const int8_t inv_cos_bit_row[MAX_TXWH_IDX /*txw_idx*/]
-                            [MAX_TXWH_IDX /*txh_idx*/] = {
-                              { 12, 12, 12, 0, 0 },
-                              { 12, 12, 12, 12, 0 },
-                              { 12, 12, 12, 12, 12 },
-                              { 0, 12, 12, 12, 12 },
-                              { 0, 0, 12, 12, 12 }
-                            };
+const int8_t inv_cos_bit_row[MAX_TXWH_IDX]      // txw_idx
+                            [MAX_TXWH_IDX] = {  // txh_idx
+    { INV_COS_BIT, INV_COS_BIT, INV_COS_BIT,           0,           0 },
+    { INV_COS_BIT, INV_COS_BIT, INV_COS_BIT, INV_COS_BIT,           0 },
+    { INV_COS_BIT, INV_COS_BIT, INV_COS_BIT, INV_COS_BIT, INV_COS_BIT },
+    {           0, INV_COS_BIT, INV_COS_BIT, INV_COS_BIT, INV_COS_BIT },
+    {           0,           0, INV_COS_BIT, INV_COS_BIT, INV_COS_BIT }
+  };
+/* clang-format on */
 
 const int8_t iadst4_range[7] = { 0, 1, 0, 0, 0, 0, 0 };