Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(237)

Side by Side Diff: third_party/opus/opus.gyp

Issue 349293006: Revert of Enable FIXED_POINT and -O3 for Opus when building for ARM. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « build/common.gypi ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 { 5 {
6 'target_defaults': {
7 'variables': {
8 # Performance gains are substantial on ARM (v7,v8) with -O3 over the
9 # default -Os configured in common.gypi.
10 'release_optimize': '3',
11 'debug_optimize': '3',
12 },
13 },
14 'variables': { 6 'variables': {
15 'conditions': [ 7 'conditions': [
16 ['target_arch=="arm" or target_arch=="armv7" or target_arch=="arm64"', { 8 ['((OS=="android" or chromeos==1) and target_arch=="arm") or (OS=="ios" an d target_arch=="armv7")', {
17 'use_opus_fixed_point%': 1, 9 'use_opus_fixed_point%': 1,
10 'use_opus_arm_optimization%': 1,
18 }, { 11 }, {
19 'use_opus_fixed_point%': 0, 12 'use_opus_fixed_point%': 0,
20 }],
21 ['target_arch=="arm" or target_arch=="armv7"', {
22 'use_opus_arm_optimization%': 1,
23 }, {
24 'use_opus_arm_optimization%': 0, 13 'use_opus_arm_optimization%': 0,
25 }], 14 }],
26 ['target_arch=="arm"', { 15 ['(OS=="android" or chromeos==1) and target_arch=="arm"', {
27 'use_opus_rtcd%': 1, 16 'use_opus_rtcd%': 1,
28 }, { 17 }, {
29 'use_opus_rtcd%': 0, 18 'use_opus_rtcd%': 0,
30 }], 19 }],
31 ], 20 ],
32 }, 21 },
33 'targets': [ 22 'targets': [
34 { 23 {
35 'target_name': 'opus', 24 'target_name': 'opus',
36 'type': 'static_library', 25 'type': 'static_library',
(...skipping 22 matching lines...) Expand all
59 }, { 48 }, {
60 'defines': [ 49 'defines': [
61 'USE_ALLOCA', 50 'USE_ALLOCA',
62 'inline=__inline', 51 'inline=__inline',
63 ], 52 ],
64 'msvs_disabled_warnings': [ 53 'msvs_disabled_warnings': [
65 4305, # Disable truncation warning in celt/pitch.c . 54 4305, # Disable truncation warning in celt/pitch.c .
66 4334, # Disable 32-bit shift warning in src/opus_encoder.c . 55 4334, # Disable 32-bit shift warning in src/opus_encoder.c .
67 ], 56 ],
68 }], 57 }],
69 ['os_posix==1 and OS!="android"', { 58 [ 'os_posix==1 and OS!="android"', {
70 # Suppress a warning given by opus_decoder.c that tells us 59 # Suppress a warning given by opus_decoder.c that tells us
71 # optimizations are turned off. 60 # optimizations are turned off.
72 'cflags': [ 61 'cflags': [
73 '-Wno-#pragma-messages', 62 '-Wno-#pragma-messages',
74 ], 63 ],
75 'xcode_settings': { 64 'xcode_settings': {
76 'WARNING_CFLAGS': [ 65 'WARNING_CFLAGS': [
77 '-Wno-#pragma-messages', 66 '-Wno-#pragma-messages',
78 ], 67 ],
79 }, 68 },
(...skipping 66 matching lines...) Expand 10 before | Expand all | Expand 10 after
146 'sources': [ 135 'sources': [
147 'src/src/opus_demo.c', 136 'src/src/opus_demo.c',
148 ], 137 ],
149 'include_dirs': [ 138 'include_dirs': [
150 'src/celt', 139 'src/celt',
151 'src/silk', 140 'src/silk',
152 ], 141 ],
153 }, # target opus_demo 142 }, # target opus_demo
154 ] 143 ]
155 } 144 }
OLDNEW
« no previous file with comments | « build/common.gypi ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698