diff options
author | Haibo Huang <hhb@google.com> | 2021-01-30 00:21:28 +0000 |
---|---|---|
committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | 2021-01-30 00:21:28 +0000 |
commit | 68227155668eb3fb4e5bb51df24a68c99b3a221d (patch) | |
tree | f616cbf164716ee566a2b186cfc40a4d23540fdf | |
parent | 4a15aa96eba4db5b6660553b30bb8f2e2569f5e9 (diff) | |
parent | 80dea9fe815a3098e509e5b8aee654255aad95d6 (diff) | |
download | libjpeg-turbo-68227155668eb3fb4e5bb51df24a68c99b3a221d.tar.gz |
Upgrade libjpeg-turbo to fa0de07678c9828cc57b3eb086c03771912ba527 am: f7bb80dc5e am: 4f66d2ef51 am: 71852861f1 am: 80dea9fe81
Original change: https://android-review.googlesource.com/c/platform/external/libjpeg-turbo/+/1564663
MUST ONLY BE SUBMITTED BY AUTOMERGER
Change-Id: I773ad167e86c48298d6a252f163bbbb5cbdc9d86
-rw-r--r-- | BUILD.gn | 56 | ||||
-rw-r--r-- | METADATA | 8 | ||||
-rw-r--r-- | README.chromium | 7 | ||||
-rw-r--r-- | jchuff.c | 8 | ||||
-rw-r--r-- | jcphuff.c | 9 | ||||
-rw-r--r-- | simd/arm/aarch64/jchuff-neon.c | 6 | ||||
-rw-r--r-- | simd/arm/aarch64/jsimd.c | 3 | ||||
-rw-r--r-- | simd/arm/jcphuff-neon.c | 4 | ||||
-rw-r--r-- | simd/arm/neon-compat.h | 6 | ||||
-rw-r--r-- | simd/arm/neon-compat.h.in | 6 |
10 files changed, 52 insertions, 61 deletions
@@ -6,7 +6,7 @@ # implementation. Use the meta target //third_party:jpeg instead. import("//build/config/sanitizers/sanitizers.gni") -if (current_cpu == "arm") { +if (current_cpu == "arm" || current_cpu == "arm64") { import("//build/config/arm.gni") } @@ -150,36 +150,11 @@ static_library("simd") { sources = [ "simd/x86_64/jsimd.c", ] - } else if (current_cpu == "arm" && arm_version >= 7 && - (arm_use_neon || arm_optionally_use_neon)) { - include_dirs += [ "simd/arm/" ] - sources = [ - "simd/arm/aarch32/jchuff-neon.c", - "simd/arm/aarch32/jsimd.c", - "simd/arm/jccolor-neon.c", - "simd/arm/jcgray-neon.c", - "simd/arm/jcphuff-neon.c", - "simd/arm/jcsample-neon.c", - "simd/arm/jdcolor-neon.c", - "simd/arm/jdmerge-neon.c", - "simd/arm/jdsample-neon.c", - "simd/arm/jfdctfst-neon.c", - "simd/arm/jfdctint-neon.c", - "simd/arm/jidctfst-neon.c", - "simd/arm/jidctint-neon.c", - "simd/arm/jidctred-neon.c", - "simd/arm/jquanti-neon.c", - ] - defines = [ - "NEON_INTRINSICS" - ] - configs -= [ "//build/config/compiler:default_optimization" ] - configs += [ "//build/config/compiler:optimize_speed" ] - } else if (current_cpu == "arm64") { + } else if ((current_cpu == "arm" || current_cpu == "arm64") && + arm_use_neon) { include_dirs += [ "simd/arm/" ] + sources = [ - "simd/arm/aarch64/jchuff-neon.c", - "simd/arm/aarch64/jsimd.c", "simd/arm/jccolor-neon.c", "simd/arm/jcgray-neon.c", "simd/arm/jcphuff-neon.c", @@ -194,9 +169,22 @@ static_library("simd") { "simd/arm/jidctred-neon.c", "simd/arm/jquanti-neon.c", ] + if (current_cpu == "arm") { + sources += [ + "simd/arm/aarch32/jchuff-neon.c", + "simd/arm/aarch32/jsimd.c", + ] + } else if (current_cpu == "arm64"){ + sources += [ + "simd/arm/aarch64/jchuff-neon.c", + "simd/arm/aarch64/jsimd.c", + ] + } + defines = [ "NEON_INTRINSICS" ] + configs -= [ "//build/config/compiler:default_optimization" ] configs += [ "//build/config/compiler:optimize_speed" ] } else { @@ -279,16 +267,14 @@ static_library("libjpeg") { ":libjpeg_headers", ] - # MemorySanitizer doesn't support assembly code, so keep it disabled in - # MSan builds for now. - if (is_msan) { + # MemorySanitizer doesn't support assembly code, so keep it disabled in x86 + # and x64 MSan builds for now. + if (is_msan && (current_cpu == "x86" || current_cpu == "x64")) { sources += [ "jsimd_none.c" ] } else { public_deps += [ ":simd" ] - if ((current_cpu == "arm" && arm_version >= 7 && - (arm_use_neon || arm_optionally_use_neon)) || - current_cpu == "arm64") { + if ((current_cpu == "arm" || current_cpu == "arm64") && arm_use_neon) { defines += [ "NEON_INTRINSICS", ] } } @@ -5,11 +5,11 @@ third_party { type: GIT value: "https://chromium.googlesource.com/chromium/deps/libjpeg_turbo" } - version: "09efc26aff7983f4377a1743a197ca3d74796d7d" + version: "fa0de07678c9828cc57b3eb086c03771912ba527" license_type: NOTICE last_upgrade_date { - year: 2020 - month: 12 - day: 29 + year: 2021 + month: 1 + day: 28 } } diff --git a/README.chromium b/README.chromium index b0a16238..e8b0bc67 100644 --- a/README.chromium +++ b/README.chromium @@ -14,11 +14,12 @@ This consists of the components: * An OWNERS file * A codereview.settings file * Patched header files used by Chromium -* Cherry-picked three additional patches from upstream master to fix bugs found - by fuzzers: +* Additional patches cherry-picked from upstream master to fix various bugs: https://github.com/libjpeg-turbo/libjpeg-turbo/commit/ccaba5d7894ecfb5a8f11e48d3f86e1f14d5a469 https://github.com/libjpeg-turbo/libjpeg-turbo/commit/c7ca521bc85b57d41d3ad4963c13fc0100481084 https://github.com/libjpeg-turbo/libjpeg-turbo/commit/110d8d6dcafaed517e8f77a6253169535ee3a20e + https://github.com/libjpeg-turbo/libjpeg-turbo/commit/d2c407995992be1f128704ae2479adfd7906c158 + https://github.com/libjpeg-turbo/libjpeg-turbo/commit/74e6ea45e3547ae85cd43efcdfc24a6907a2154e * Deleted unused directories: ci, cmakescripts, doc, java, release, sharedlib, simd/loongson, simd/mips, simd/powerpc, and win * Deleted unused files: appveyor.yml, CMakeLists.txt, doxygen.config, @@ -73,8 +74,6 @@ following changes which are not merged to upstream: - Refactor djpeg.c to provide test interface A new gtest directory contains GTest wrappers (and associated utilities) for each of tjunittest, tjbench, cjpeg, djpeg and jpegtran. -* Disable Neon SIMD path for Huffman encoding when compiling for Windows on Arm - using Clang-cl: http://crbug.com/1160249 Refer to working-with-nested-repos [1] for details of how to setup your git svn client to update the code (for making local changes, cherry picking from @@ -44,15 +44,19 @@ * flags (this defines __thumb__). */ -/* NOTE: Both GCC and Clang define __GNUC__ */ -#if defined(__GNUC__) && (defined(__arm__) || defined(__aarch64__)) +#if defined(__arm__) || defined(__aarch64__) || defined(_M_ARM) || \ + defined(_M_ARM64) #if !defined(__thumb__) || defined(__thumb2__) #define USE_CLZ_INTRINSIC #endif #endif #ifdef USE_CLZ_INTRINSIC +#if defined(_MSC_VER) && !defined(__clang__) +#define JPEG_NBITS_NONZERO(x) (32 - _CountLeadingZeros(x)) +#else #define JPEG_NBITS_NONZERO(x) (32 - __builtin_clz(x)) +#endif #define JPEG_NBITS(x) (x ? JPEG_NBITS_NONZERO(x) : 0) #else #include "jpeg_nbits_table.h" @@ -6,6 +6,7 @@ * libjpeg-turbo Modifications: * Copyright (C) 2011, 2015, 2018, D. R. Commander. * Copyright (C) 2016, 2018, Matthieu Darbois. + * Copyright (C) 2020, Arm Limited. * For conditions of distribution and use, see the accompanying README.ijg * file. * @@ -51,15 +52,19 @@ * flags (this defines __thumb__). */ -/* NOTE: Both GCC and Clang define __GNUC__ */ -#if defined(__GNUC__) && (defined(__arm__) || defined(__aarch64__)) +#if defined(__arm__) || defined(__aarch64__) || defined(_M_ARM) || \ + defined(_M_ARM64) #if !defined(__thumb__) || defined(__thumb2__) #define USE_CLZ_INTRINSIC #endif #endif #ifdef USE_CLZ_INTRINSIC +#if defined(_MSC_VER) && !defined(__clang__) +#define JPEG_NBITS_NONZERO(x) (32 - _CountLeadingZeros(x)) +#else #define JPEG_NBITS_NONZERO(x) (32 - __builtin_clz(x)) +#endif #define JPEG_NBITS(x) (x ? JPEG_NBITS_NONZERO(x) : 0) #else #include "jpeg_nbits_table.h" diff --git a/simd/arm/aarch64/jchuff-neon.c b/simd/arm/aarch64/jchuff-neon.c index a0a57a66..f13fd1b5 100644 --- a/simd/arm/aarch64/jchuff-neon.c +++ b/simd/arm/aarch64/jchuff-neon.c @@ -1,7 +1,7 @@ /* * jchuff-neon.c - Huffman entropy encoding (64-bit Arm Neon) * - * Copyright (C) 2020, Arm Limited. All Rights Reserved. + * Copyright (C) 2020-2021, Arm Limited. All Rights Reserved. * Copyright (C) 2020, D. R. Commander. All Rights Reserved. * * This software is provided 'as-is', without any express or implied @@ -331,7 +331,7 @@ JOCTET *jsimd_huff_encode_one_block_neon(void *state, JOCTET *buffer, vst1q_u16(block_diff + 7 * DCTSIZE, row7_diff); while (bitmap != 0) { - r = BUILTIN_CLZL(bitmap); + r = BUILTIN_CLZLL(bitmap); i += r; bitmap <<= r; nbits = block_nbits[i]; @@ -370,7 +370,7 @@ JOCTET *jsimd_huff_encode_one_block_neon(void *state, JOCTET *buffer, /* Same as above but must mask diff bits and compute nbits on demand. */ while (bitmap != 0) { - r = BUILTIN_CLZL(bitmap); + r = BUILTIN_CLZLL(bitmap); i += r; bitmap <<= r; lz = BUILTIN_CLZ(block_abs[i]); diff --git a/simd/arm/aarch64/jsimd.c b/simd/arm/aarch64/jsimd.c index 4991bc0a..8570b82c 100644 --- a/simd/arm/aarch64/jsimd.c +++ b/simd/arm/aarch64/jsimd.c @@ -977,8 +977,6 @@ jsimd_idct_float(j_decompress_ptr cinfo, jpeg_component_info *compptr, GLOBAL(int) jsimd_can_huff_encode_one_block(void) { -/* Disable for Windows on Arm compiled with Clang-cl: crbug.com/1160249 */ -#if !(defined(_MSC_VER) && defined(__clang__)) init_simd(); if (DCTSIZE != 8) @@ -988,7 +986,6 @@ jsimd_can_huff_encode_one_block(void) if (simd_support & JSIMD_NEON && simd_huffman) return 1; -#endif return 0; } diff --git a/simd/arm/jcphuff-neon.c b/simd/arm/jcphuff-neon.c index 8b6d53be..86a263fa 100644 --- a/simd/arm/jcphuff-neon.c +++ b/simd/arm/jcphuff-neon.c @@ -1,7 +1,7 @@ /* * jcphuff-neon.c - prepare data for progressive Huffman encoding (Arm Neon) * - * Copyright (C) 2020, Arm Limited. All Rights Reserved. + * Copyright (C) 2020-2021, Arm Limited. All Rights Reserved. * * This software is provided 'as-is', without any express or implied * warranty. In no event will the authors be held liable for any damages @@ -572,7 +572,7 @@ int jsimd_encode_mcu_AC_refine_prepare_neon /* EOB position is defined to be 0 if all coefficients != 1. */ return 0; } else { - return 63 - BUILTIN_CLZL(bitmap); + return 63 - BUILTIN_CLZLL(bitmap); } #else /* Move bitmap to two 32-bit scalar registers. */ diff --git a/simd/arm/neon-compat.h b/simd/arm/neon-compat.h index 3ce3bcbe..543d8600 100644 --- a/simd/arm/neon-compat.h +++ b/simd/arm/neon-compat.h @@ -1,6 +1,6 @@ /* * Copyright (C) 2020, D. R. Commander. All Rights Reserved. - * Copyright (C) 2020, Arm Limited. All Rights Reserved. + * Copyright (C) 2020-2021, Arm Limited. All Rights Reserved. * * This software is provided 'as-is', without any express or implied * warranty. In no event will the authors be held liable for any damages @@ -28,10 +28,10 @@ /* Define compiler-independent count-leading-zeros macros */ #if defined(_MSC_VER) && !defined(__clang__) #define BUILTIN_CLZ(x) _CountLeadingZeros(x) -#define BUILTIN_CLZL(x) _CountLeadingZeros64(x) +#define BUILTIN_CLZLL(x) _CountLeadingZeros64(x) #elif defined(__clang__) || defined(__GNUC__) #define BUILTIN_CLZ(x) __builtin_clz(x) -#define BUILTIN_CLZL(x) __builtin_clzl(x) +#define BUILTIN_CLZLL(x) __builtin_clzll(x) #else #error "Unknown compiler" #endif diff --git a/simd/arm/neon-compat.h.in b/simd/arm/neon-compat.h.in index e2347b9b..23d6d28c 100644 --- a/simd/arm/neon-compat.h.in +++ b/simd/arm/neon-compat.h.in @@ -1,6 +1,6 @@ /* * Copyright (C) 2020, D. R. Commander. All Rights Reserved. - * Copyright (C) 2020, Arm Limited. All Rights Reserved. + * Copyright (C) 2020-2021, Arm Limited. All Rights Reserved. * * This software is provided 'as-is', without any express or implied * warranty. In no event will the authors be held liable for any damages @@ -26,10 +26,10 @@ /* Define compiler-independent count-leading-zeros macros */ #if defined(_MSC_VER) && !defined(__clang__) #define BUILTIN_CLZ(x) _CountLeadingZeros(x) -#define BUILTIN_CLZL(x) _CountLeadingZeros64(x) +#define BUILTIN_CLZLL(x) _CountLeadingZeros64(x) #elif defined(__clang__) || defined(__GNUC__) #define BUILTIN_CLZ(x) __builtin_clz(x) -#define BUILTIN_CLZL(x) __builtin_clzl(x) +#define BUILTIN_CLZLL(x) __builtin_clzll(x) #else #error "Unknown compiler" #endif |