AuroraMiddleware/skia2
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

GN: turn on libpng SSE2 filters

Browse Source 
I was profiling and remembered GN hasn't turned on much CPU-specific code.  I looked at zlib, then libpng, then libjpeg-turbo:

  - zlib was easy but not useful.  No routine we use in decoding changes significantly.
  - libpng was easy and useful, and we were already using NEON filters on ARM.
  - libjpeg-turbo requires yasm and is annoying.

BUG=skia:5875

GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=4531

Change-Id: Ie072a457da41ee6538eebacb5eb5dbe5a6eb585e
Reviewed-on: https://skia-review.googlesource.com/4531
Commit-Queue: Mike Klein <mtklein@chromium.org>
Reviewed-by: Matt Sarett <msarett@google.com>
This commit is contained in:
Mike Klein 2016-11-07 16:56:53 -05:00
parent 7d7d7d1946
commit d23341bb5e
1 changed files with 9 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
third_party/libpng/BUILD.gn vendored
View File

@ -17,6 +17,7 @@ if (skia_use_system_libpng) {
third_party("libpng") { third_party("libpng") {
public_include_dirs = [ "." ] public_include_dirs = [ "." ]
defines = []
deps = [ deps = [
"//third_party/zlib", "//third_party/zlib",
] ]
@ -44,5 +45,13 @@ if (skia_use_system_libpng) {
"arm/filter_neon_intrinsics.c", "arm/filter_neon_intrinsics.c",
] ]
} }
if (current_cpu == "x86" || current_cpu == "x64") {
defines += [ "PNG_INTEL_SSE" ]
sources += [
"contrib/intel/filter_sse2_intrinsics.c",
"contrib/intel/intel_init.c",
]
}
} }
} }