Don't use NEON optimized code in lpng_v163
authorSam Clegg <sbc@chromium.org>
Wed, 15 Apr 2015 17:53:07 +0000 (10:53 -0700)
committerSam Clegg <sbc@chromium.org>
Wed, 15 Apr 2015 17:53:07 +0000 (10:53 -0700)
The source files required to use the NEON function are not
included so we should not try to reference those symbols.

BUG=477162
TEST=ninja -C out_arm/Release/ pdfium_diff
R=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1085023003

AUTHORS
samples/fx_lpng/lpng_v163/pngpriv.h

diff --git a/AUTHORS b/AUTHORS
index cb225ab..c57cb51 100644 (file)
--- a/AUTHORS
+++ b/AUTHORS
@@ -13,6 +13,7 @@ Brett Wilson <brettw@chromium.org>
 Bruce Dawson <brucedawson@chromium.org>
 Chris Palmer <palmer@chromium.org>
 Finnur Thorarinsson <finnur@chromium.org>
+Jiang Jiang <jiangj@opera.com>
 Jochen Eisinger <jochen@chromium.org>
 John Abd-El-Malek <jam@chromium.org>
 Julien Tinnes <jln@chromium.org>
@@ -26,8 +27,8 @@ Peter Kasting <pkasting@chromium.org>
 Raymes Khoury <raymes@chromium.org>
 Reid Kleckner <rnk@chromium.org>
 Robert Sesek <rsesek@chromium.org>
+Sam Clegg <sbc@chromium.org>
 Thomas Sepez <tsepez@chromium.org>
-Jiang Jiang <jiangj@opera.com>
 
 Foxit Software Inc <*@foxitsoftware.com>
 Google Inc. <*@google.com>
index 7fa3d60..1896538 100644 (file)
  * this in $(CC), e.g. "CC=gcc -mfpu=neon", but people who build libpng rarely\r
  * do this.\r
  */\r
+/*\r
+ * pdfium: This version of libpng does not contain the arm neon optimized code\r
+ * so we should never try to reference it.\r
+ */\r
+#if 0\r
 #ifndef PNG_ARM_NEON_OPT\r
    /* ARM NEON optimizations are being controlled by the compiler settings,\r
     * typically the target FPU.  If the FPU has been set to NEON (-mfpu=neon\r
     */\r
 #  define PNG_FILTER_OPTIMIZATIONS png_init_filter_functions_neon\r
 #endif\r
+#endif\r
 \r
 /* Is this a build of a DLL where compilation of the object modules requires\r
  * different preprocessor settings to those required for a simple library?  If\r