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

Side by Side Diff: build/features.gypi

Issue 841083002: Remove "extra checks" (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: 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') | src/api.h » ('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 2013 the V8 project authors. All rights reserved. 1 # Copyright 2013 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 84 matching lines...) Expand 10 before | Expand all | Expand 10 after
95 'defines': ['V8_USE_EXTERNAL_STARTUP_DATA',], 95 'defines': ['V8_USE_EXTERNAL_STARTUP_DATA',],
96 }], 96 }],
97 ['dcheck_always_on!=0', { 97 ['dcheck_always_on!=0', {
98 'defines': ['DEBUG',], 98 'defines': ['DEBUG',],
99 }], 99 }],
100 ], # conditions 100 ], # conditions
101 'configurations': { 101 'configurations': {
102 'DebugBaseCommon': { 102 'DebugBaseCommon': {
103 'abstract': 1, 103 'abstract': 1,
104 'variables': { 104 'variables': {
105 'v8_enable_extra_checks%': 1,
106 'v8_enable_handle_zapping%': 1, 105 'v8_enable_handle_zapping%': 1,
107 }, 106 },
108 'conditions': [ 107 'conditions': [
109 ['v8_enable_extra_checks==1', {
110 'defines': ['ENABLE_EXTRA_CHECKS',],
111 }],
112 ['v8_enable_handle_zapping==1', { 108 ['v8_enable_handle_zapping==1', {
113 'defines': ['ENABLE_HANDLE_ZAPPING',], 109 'defines': ['ENABLE_HANDLE_ZAPPING',],
114 }], 110 }],
115 ], 111 ],
116 }, # Debug 112 }, # Debug
117 'Release': { 113 'Release': {
118 'variables': { 114 'variables': {
119 'v8_enable_extra_checks%': 0,
120 'v8_enable_handle_zapping%': 1, 115 'v8_enable_handle_zapping%': 1,
121 }, 116 },
122 'conditions': [ 117 'conditions': [
123 ['v8_enable_extra_checks==1', {
124 'defines': ['ENABLE_EXTRA_CHECKS',],
125 }],
126 ['v8_enable_handle_zapping==1', { 118 ['v8_enable_handle_zapping==1', {
127 'defines': ['ENABLE_HANDLE_ZAPPING',], 119 'defines': ['ENABLE_HANDLE_ZAPPING',],
128 }], 120 }],
129 ], # conditions 121 ], # conditions
130 }, # Release 122 }, # Release
131 }, # configurations 123 }, # configurations
132 }, # target_defaults 124 }, # target_defaults
133 } 125 }
OLDNEW
« no previous file with comments | « Makefile ('k') | src/api.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698