shithub: libvpx

Info  •  Files  •  Log  •  Branches

Don't inline cost_coeffs.

It was tiny when it was orginally marked INLINE. Forcing this function
to be inlined prevents the compiler from inlining its much smaller
callers.

No measurable speed impact, 28320 byte smaller libvpx.a

Change-Id: I6bf4c917157d15cbadb3cd3e20a9e82d35dc7d6f

--- a/vp9/encoder/vp9_rdopt.c

+++ b/vp9/encoder/vp9_rdopt.c

@@ -328,12 +328,12 @@

   { 1, 2, 3, 4, 11,  256 - 21, 0 },

   { 1, 2, 3, 4, 11, 1024 - 21, 0 },

 };

-static INLINE int cost_coeffs(MACROBLOCK *x,

-                              int plane, int block,

-                              ENTROPY_CONTEXT *A, ENTROPY_CONTEXT *L,

-                              TX_SIZE tx_size,

-                              const int16_t *scan, const int16_t *nb,

-                              int use_fast_coef_costing) {

+static int cost_coeffs(MACROBLOCK *x,

+                       int plane, int block,

+                       ENTROPY_CONTEXT *A, ENTROPY_CONTEXT *L,

+                       TX_SIZE tx_size,

+                       const int16_t *scan, const int16_t *nb,

+                       int use_fast_coef_costing) {

   MACROBLOCKD *const xd = &x->e_mbd;

   MB_MODE_INFO *mbmi = &xd->mi[0].src_mi->mbmi;

   const struct macroblock_plane *p = &x->plane[plane];