Rewriting MatrixConvolution image filter with SSE and AVX2

gyp/opts.gypi

Index: gyp/opts.gypi
diff --git a/gyp/opts.gypi b/gyp/opts.gypi
index f3f61efd425899b4182b269dd06eb40003574345..f821d783de3158d7ffd4cab05570b84858141b7a 100644
--- a/gyp/opts.gypi
+++ b/gyp/opts.gypi
@@ -43,6 +43,8 @@
             '<(skia_src_path)/opts/SkBitmapProcState_opts_SSE2.cpp',
             '<(skia_src_path)/opts/SkBlitRow_opts_SSE2.cpp',
             '<(skia_src_path)/opts/opts_check_x86.cpp',
+            '<(skia_src_path)/opts/SkOpts_sse_sse2.cpp',

[Stephen White, 2016/04/12 20:01:58]

It seems strange to add a file just for this one function. Mike would know
better than I the current way to do these integrations, in Skia, but perhaps the
SSE path should just be integrated directly into
SkMatrixConvolutionImageFilter.cpp (protected by a compile-time flag), and we
just switch between the SSE-or-not version and the AVX2 version at runtime?

+            '<(skia_src_path)/opts/SkMatrixConvolutionImageFilter_opts_SSE.cpp',
         ],
         'ssse3_sources': [
             '<(skia_src_path)/opts/SkBitmapProcState_opts_SSSE3.cpp',
@@ -60,5 +62,7 @@
         ],
         'avx2_sources': [
             '<(skia_src_path)/core/SkForceCPlusPlusLinking.cpp',
+            '<(skia_src_path)/opts/SkMatrixConvolutionImageFilter_opts_AVX2.cpp',
+            '<(skia_src_path)/opts/SkOpts_avx2.cpp',
         ],
 }