Force more inlining in Skia ARM NEON optimizations

skia/skia_library_opts.gyp

 # Copyright 2013 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 
 # This gyp file contains the platform-specific optimizations for Skia
 {
   'targets': [
     # Due to an unfortunate intersection of lameness between gcc and gyp,
     # we have to build the *_SSE2.cpp files in a separate target.  The
@@ skipping 227 matching lines @@
           'type': 'static_library',
           'includes': [
             'skia_common.gypi',
           ],
           'include_dirs': [
             '../third_party/skia/include/core',
             '../third_party/skia/include/effects',
             '../third_party/skia/src/core',
             '../third_party/skia/src/opts',
           ],
+          # The special cases are performance critical and suffer from lack

[mtklein, 2014/05/07 14:49:42]

Is there any way we can do this with SK_ALWAYS_INLINE (==
__attribute__((always_inline)) ) tagged on particular functions in source?  My
concerns about doing in the build are:
  1) Next time we come to fiddle with this, we won't know which functions this
was meant to help.
  2) We're only doing this in Chrome, so Android loses out, and we're not
testing it on our Skia bots.

+          # of inlining. Keep below in sync with -finline-limit=XX usage.
           'cflags!': [
+            '-Os',
+            '-finline-limit=64',
             '-fno-omit-frame-pointer',
             '-mfpu=vfp',  # remove them all, just in case.
             '-mfpu=vfpv3',
             '-mfpu=vfpv3-d16',
           ],
           'cflags': [
+            '-O2',
             '-mfpu=neon',
             '-fomit-frame-pointer',
           ],
           'ldflags': [
             '-march=armv7-a',
             '-Wl,--fix-cortex-a8',
           ],
           'sources': [
             '../third_party/skia/src/opts/memset16_neon.S',
             '../third_party/skia/src/opts/memset32_neon.S',
@@ skipping 17 matching lines @@
               'defines': [
                 '__ARM_HAVE_OPTIONAL_NEON_SUPPORT',
               ],
             }],
           ],
         },
       ],
     }],
   ],
 }