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

Side by Side Diff: build/standalone.gypi

Issue 224443003: Use abstract configurations to switch between opt debug settings (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: updates Created 6 years, 8 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 | Annotate | Revision Log
OLDNEW
1 # Copyright 2012 the V8 project authors. All rights reserved. 1 # Copyright 2012 the V8 project authors. All rights reserved.
2 # Redistribution and use in source and binary forms, with or without 2 # Redistribution and use in source and binary forms, with or without
3 # modification, are permitted provided that the following conditions are 3 # modification, are permitted provided that the following conditions are
4 # met: 4 # met:
5 # 5 #
6 # * Redistributions of source code must retain the above copyright 6 # * Redistributions of source code must retain the above copyright
7 # notice, this list of conditions and the following disclaimer. 7 # notice, this list of conditions and the following disclaimer.
8 # * Redistributions in binary form must reproduce the above 8 # * Redistributions in binary form must reproduce the above
9 # copyright notice, this list of conditions and the following 9 # copyright notice, this list of conditions and the following
10 # disclaimer in the documentation and/or other materials provided 10 # disclaimer in the documentation and/or other materials provided
(...skipping 120 matching lines...) Expand 10 before | Expand all | Expand 10 after
131 }, 131 },
132 'target_defaults': { 132 'target_defaults': {
133 'variables': { 133 'variables': {
134 'v8_code%': '<(v8_code)', 134 'v8_code%': '<(v8_code)',
135 }, 135 },
136 'default_configuration': 'Debug', 136 'default_configuration': 'Debug',
137 'configurations': { 137 'configurations': {
138 'Debug': { 138 'Debug': {
139 'cflags': [ '-g', '-O0' ], 139 'cflags': [ '-g', '-O0' ],
140 }, 140 },
141 'Optdebug': {
142 'inherit_from': [ 'Debug' ],
143 },
141 'Release': { 144 'Release': {
142 # Xcode insists on this empty entry. 145 # Xcode insists on this empty entry.
143 }, 146 },
144 }, 147 },
145 'target_conditions': [ 148 'target_conditions': [
146 ['v8_code == 0', { 149 ['v8_code == 0', {
147 'defines!': [ 150 'defines!': [
148 'DEBUG', 151 'DEBUG',
149 ], 152 ],
150 'conditions': [ 153 'conditions': [
(...skipping 31 matching lines...) Expand 10 before | Expand all | Expand 10 after
182 '-fomit-frame-pointer', 185 '-fomit-frame-pointer',
183 ], 186 ],
184 'ldflags': [ 187 'ldflags': [
185 '-fsanitize=address', 188 '-fsanitize=address',
186 ], 189 ],
187 }, 190 },
188 }], 191 }],
189 ['OS=="linux" or OS=="freebsd" or OS=="openbsd" or OS=="solaris" \ 192 ['OS=="linux" or OS=="freebsd" or OS=="openbsd" or OS=="solaris" \
190 or OS=="netbsd"', { 193 or OS=="netbsd"', {
191 'target_defaults': { 194 'target_defaults': {
192 'cflags': [ '-Wall', '<(werror)', '-W', '-Wno-unused-parameter', 195 'cflags': [ '-Wall', '<(werror)', '-W', '-Wno-unused-parameter',
jochen (gone - plz use gerrit) 2014/04/03 22:21:38 Any idea why these settings are used "twice" in th
193 '-pthread', '-fno-exceptions', '-pedantic' ], 196 '-pthread', '-fno-exceptions', '-pedantic' ],
194 'cflags_cc': [ '-Wnon-virtual-dtor', '-fno-rtti' ], 197 'cflags_cc': [ '-Wnon-virtual-dtor', '-fno-rtti' ],
195 'ldflags': [ '-pthread', ], 198 'ldflags': [ '-pthread', ],
196 'conditions': [ 199 'conditions': [
197 [ 'OS=="linux"', { 200 [ 'OS=="linux"', {
198 'cflags': [ '-ansi' ], 201 'cflags': [ '-ansi' ],
199 }], 202 }],
200 [ 'visibility=="hidden" and v8_enable_backtrace==0', { 203 [ 'visibility=="hidden" and v8_enable_backtrace==0', {
201 'cflags': [ '-fvisibility=hidden' ], 204 'cflags': [ '-fvisibility=hidden' ],
202 }], 205 }],
(...skipping 149 matching lines...) Expand 10 before | Expand all | Expand 10 after
352 ], 355 ],
353 'target_conditions': [ 356 'target_conditions': [
354 ['_type!="static_library"', { 357 ['_type!="static_library"', {
355 'xcode_settings': {'OTHER_LDFLAGS': ['-Wl,-search_paths_first']}, 358 'xcode_settings': {'OTHER_LDFLAGS': ['-Wl,-search_paths_first']},
356 }], 359 }],
357 ], # target_conditions 360 ], # target_conditions
358 }, # target_defaults 361 }, # target_defaults
359 }], # OS=="mac" 362 }], # OS=="mac"
360 ], 363 ],
361 } 364 }
OLDNEW
« build.ninja ('K') | « build/features.gypi ('k') | build/toolchain.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698