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

Side by Side Diff: build/standalone.gypi

Issue 847753002: Merge v8_optimized_debug 1 and 2 (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: updates Created 5 years, 11 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 | « Makefile ('k') | build/toolchain.gypi » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 64 matching lines...) Expand 10 before | Expand all | Expand 10 after
75 75
76 # .gyp files or targets should set v8_code to 1 if they build V8 specific 76 # .gyp files or targets should set v8_code to 1 if they build V8 specific
77 # code, as opposed to external code. This variable is used to control such 77 # code, as opposed to external code. This variable is used to control such
78 # things as the set of warnings to enable, and whether warnings are treated 78 # things as the set of warnings to enable, and whether warnings are treated
79 # as errors. 79 # as errors.
80 'v8_code%': 0, 80 'v8_code%': 0,
81 81
82 # Speeds up Debug builds: 82 # Speeds up Debug builds:
83 # 0 - Compiler optimizations off (debuggable) (default). This may 83 # 0 - Compiler optimizations off (debuggable) (default). This may
84 # be 5x slower than Release (or worse). 84 # be 5x slower than Release (or worse).
85 # 1 - Turn on compiler optimizations. This may be hard or impossible to 85 # 1 - Turn on optimizations and disable slow DCHECKs, but leave
86 # debug. This may still be 2x slower than Release (or worse). 86 # V8_ENABLE_CHECKS and most other assertions enabled. This may cause
87 # 2 - Turn on optimizations, and also #undef DEBUG / #define NDEBUG 87 # some v8 tests to fail in the Debug configuration. This roughly
88 # (but leave V8_ENABLE_CHECKS and most other assertions enabled. 88 # matches the performance of a Release build and can be used by
89 # This may cause some v8 tests to fail in the Debug configuration. 89 # embedders that need to build their own code as debug but don't want
90 # This roughly matches the performance of a Release build and can 90 # or need a debug version of V8. This should produce near-release
91 # be used by embedders that need to build their own code as debug 91 # speeds.
92 # but don't want or need a debug version of V8. This should produce
93 # near-release speeds.
94 'v8_optimized_debug%': 0, 92 'v8_optimized_debug%': 0,
95 93
96 # Use external files for startup data blobs: 94 # Use external files for startup data blobs:
97 # the JS builtins sources and the start snapshot. 95 # the JS builtins sources and the start snapshot.
98 # Embedders that don't use standalone.gypi will need to add 96 # Embedders that don't use standalone.gypi will need to add
99 # their own default value. 97 # their own default value.
100 'v8_use_external_startup_data%': 0, 98 'v8_use_external_startup_data%': 0,
101 99
102 # Relative path to icu.gyp from this file. 100 # Relative path to icu.gyp from this file.
103 'icu_gyp_path': '../third_party/icu/icu.gyp', 101 'icu_gyp_path': '../third_party/icu/icu.gyp',
(...skipping 53 matching lines...) Expand 10 before | Expand all | Expand 10 after
157 'conditions': [ 155 'conditions': [
158 ['(v8_target_arch=="ia32" or v8_target_arch=="x87") and \ 156 ['(v8_target_arch=="ia32" or v8_target_arch=="x87") and \
159 OS=="linux"', { 157 OS=="linux"', {
160 'defines': [ 158 'defines': [
161 '_GLIBCXX_DEBUG' 159 '_GLIBCXX_DEBUG'
162 ], 160 ],
163 }], 161 }],
164 ], 162 ],
165 }, 163 },
166 'Optdebug': { 164 'Optdebug': {
167 'inherit_from': [ 'DebugBaseCommon', 'DebugBase2' ], 165 'inherit_from': [ 'DebugBaseCommon', 'DebugBase1' ],
168 }, 166 },
169 'Debug': { 167 'Debug': {
170 # Xcode insists on this empty entry. 168 # Xcode insists on this empty entry.
171 }, 169 },
172 'Release': { 170 'Release': {
173 # Xcode insists on this empty entry. 171 # Xcode insists on this empty entry.
174 }, 172 },
175 }, 173 },
176 'target_conditions': [ 174 'target_conditions': [
177 ['v8_code == 0', { 175 ['v8_code == 0', {
(...skipping 296 matching lines...) Expand 10 before | Expand all | Expand 10 after
474 ], 472 ],
475 }], 473 }],
476 ['clang==1 and OS=="win"', { 474 ['clang==1 and OS=="win"', {
477 'make_global_settings': [ 475 'make_global_settings': [
478 # On Windows, gyp's ninja generator only looks at CC. 476 # On Windows, gyp's ninja generator only looks at CC.
479 ['CC', '<(make_clang_dir)/bin/clang-cl'], 477 ['CC', '<(make_clang_dir)/bin/clang-cl'],
480 ], 478 ],
481 }], 479 }],
482 ], 480 ],
483 } 481 }
OLDNEW
« no previous file with comments | « Makefile ('k') | build/toolchain.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698