From b464764dae4ffa40b880d96cdc368afd890a8a27 Mon Sep 17 00:00:00 2001 From: mtklein Date: Wed, 27 Jul 2016 05:35:28 -0700 Subject: Extend __mulodi4() hack to 32-bit x86. This gets the Android-Clang-x86 perf bot from failing on __mulodi4() to failing on the next problem... See https://luci-milo.appspot.com/swarming/task/3042e5f68636f810/steps/nanobench/0/stdout BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2190493002 Review-Url: https://codereview.chromium.org/2190493002 --- gyp/dng_sdk.gyp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'gyp') diff --git a/gyp/dng_sdk.gyp b/gyp/dng_sdk.gyp index be0c0f97db..05d6c25222 100644 --- a/gyp/dng_sdk.gyp +++ b/gyp/dng_sdk.gyp @@ -143,9 +143,9 @@ ['skia_os != "linux"', { 'sources': ['<@(headers)'], }], - ['skia_arch_type == "arm" and skia_clang_build', { + ['(skia_arch_type == "arm" or skia_arch_type == "x86") and skia_clang_build', { # DNG SDK uses __builtin_smulll_overflow() to detect 64x64 bit multiply overflow. - # On ARMv7, Clang implements this with __mulodi4() in libclang_rt. + # On ARMv7 and 32-bit x86, Clang implements this with __mulodi4() in libclang_rt. # I can't quite figure out how to link that here, so instead here's a shim for # __builtin_smulll_overflow() that multiplies normally assuming no overflow. # Tracked in b/29412086. -- cgit v1.2.3