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

Side by Side Diff: build/common.gypi

Issue 11275324: Disable EXTRA_CHECKS in Release (Closed) Base URL: git://github.com/v8/v8.git@bleeding_edge
Patch Set: Created 8 years, 1 month 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 | « no previous file | 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 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 52 matching lines...) Expand 10 before | Expand all | Expand 10 after
63 # Similar to the ARM hard float ABI but on MIPS. 63 # Similar to the ARM hard float ABI but on MIPS.
64 'v8_use_mips_abi_hardfloat%': 'true', 64 'v8_use_mips_abi_hardfloat%': 'true',
65 65
66 # Default arch variant for MIPS. 66 # Default arch variant for MIPS.
67 'mips_arch_variant%': 'mips32r2', 67 'mips_arch_variant%': 'mips32r2',
68 68
69 'v8_enable_debugger_support%': 1, 69 'v8_enable_debugger_support%': 1,
70 70
71 'v8_enable_disassembler%': 0, 71 'v8_enable_disassembler%': 0,
72 72
73 # Enable extra checks in API functions and other strategic places.
74 'v8_enable_extra_checks%': 1,
75
76 'v8_enable_gdbjit%': 0, 73 'v8_enable_gdbjit%': 0,
77 74
78 'v8_object_print%': 0, 75 'v8_object_print%': 0,
79 76
80 # Enable profiling support. Only required on Windows. 77 # Enable profiling support. Only required on Windows.
81 'v8_enable_prof%': 0, 78 'v8_enable_prof%': 0,
82 79
83 'v8_enable_verify_heap%': 0, 80 'v8_enable_verify_heap%': 0,
84 81
85 # Some versions of GCC 4.5 seem to need -fno-strict-aliasing. 82 # Some versions of GCC 4.5 seem to need -fno-strict-aliasing.
(...skipping 21 matching lines...) Expand all
107 'v8_interpreted_regexp%': 0, 104 'v8_interpreted_regexp%': 0,
108 }, 105 },
109 'target_defaults': { 106 'target_defaults': {
110 'conditions': [ 107 'conditions': [
111 ['v8_enable_debugger_support==1', { 108 ['v8_enable_debugger_support==1', {
112 'defines': ['ENABLE_DEBUGGER_SUPPORT',], 109 'defines': ['ENABLE_DEBUGGER_SUPPORT',],
113 }], 110 }],
114 ['v8_enable_disassembler==1', { 111 ['v8_enable_disassembler==1', {
115 'defines': ['ENABLE_DISASSEMBLER',], 112 'defines': ['ENABLE_DISASSEMBLER',],
116 }], 113 }],
117 ['v8_enable_extra_checks==1', {
118 'defines': ['ENABLE_EXTRA_CHECKS',],
119 }],
120 ['v8_enable_gdbjit==1', { 114 ['v8_enable_gdbjit==1', {
121 'defines': ['ENABLE_GDB_JIT_INTERFACE',], 115 'defines': ['ENABLE_GDB_JIT_INTERFACE',],
122 }], 116 }],
123 ['v8_object_print==1', { 117 ['v8_object_print==1', {
124 'defines': ['OBJECT_PRINT',], 118 'defines': ['OBJECT_PRINT',],
125 }], 119 }],
126 ['v8_enable_verify_heap==1', { 120 ['v8_enable_verify_heap==1', {
127 'defines': ['VERIFY_HEAP',], 121 'defines': ['VERIFY_HEAP',],
128 }], 122 }],
129 ['v8_interpreted_regexp==1', { 123 ['v8_interpreted_regexp==1', {
(...skipping 199 matching lines...) Expand 10 before | Expand all | Expand 10 after
329 }], 323 }],
330 ['OS=="freebsd" or OS=="openbsd"', { 324 ['OS=="freebsd" or OS=="openbsd"', {
331 'cflags': [ '-I/usr/local/include' ], 325 'cflags': [ '-I/usr/local/include' ],
332 }], 326 }],
333 ['OS=="netbsd"', { 327 ['OS=="netbsd"', {
334 'cflags': [ '-I/usr/pkg/include' ], 328 'cflags': [ '-I/usr/pkg/include' ],
335 }], 329 }],
336 ], # conditions 330 ], # conditions
337 'configurations': { 331 'configurations': {
338 'Debug': { 332 'Debug': {
333 'variables': {
334 'v8_enable_extra_checks%': 1,
335 },
339 'defines': [ 336 'defines': [
340 'DEBUG', 337 'DEBUG',
341 'ENABLE_DISASSEMBLER', 338 'ENABLE_DISASSEMBLER',
342 'V8_ENABLE_CHECKS', 339 'V8_ENABLE_CHECKS',
343 'OBJECT_PRINT', 340 'OBJECT_PRINT',
344 'VERIFY_HEAP', 341 'VERIFY_HEAP',
345 ], 342 ],
346 'msvs_settings': { 343 'msvs_settings': {
347 'VCCLCompilerTool': { 344 'VCCLCompilerTool': {
348 'Optimization': '0', 345 'Optimization': '0',
349 346
350 'conditions': [ 347 'conditions': [
351 ['OS=="win" and component=="shared_library"', { 348 ['OS=="win" and component=="shared_library"', {
352 'RuntimeLibrary': '3', # /MDd 349 'RuntimeLibrary': '3', # /MDd
353 }, { 350 }, {
354 'RuntimeLibrary': '1', # /MTd 351 'RuntimeLibrary': '1', # /MTd
355 }], 352 }],
356 ], 353 ],
357 }, 354 },
358 'VCLinkerTool': { 355 'VCLinkerTool': {
359 'LinkIncremental': '2', 356 'LinkIncremental': '2',
360 }, 357 },
361 }, 358 },
362 'conditions': [ 359 'conditions': [
360 ['v8_enable_extra_checks==1', {
361 'defines': ['ENABLE_EXTRA_CHECKS',],
362 }],
363 ['OS=="linux" or OS=="freebsd" or OS=="openbsd" or OS=="netbsd"', { 363 ['OS=="linux" or OS=="freebsd" or OS=="openbsd" or OS=="netbsd"', {
364 'cflags': [ '-Wall', '<(werror)', '-W', '-Wno-unused-parameter', 364 'cflags': [ '-Wall', '<(werror)', '-W', '-Wno-unused-parameter',
365 '-Wnon-virtual-dtor', '-Woverloaded-virtual' ], 365 '-Wnon-virtual-dtor', '-Woverloaded-virtual' ],
366 }], 366 }],
367 ['OS=="android"', { 367 ['OS=="android"', {
368 'variables': { 368 'variables': {
369 'android_full_debug%': 1, 369 'android_full_debug%': 1,
370 }, 370 },
371 'conditions': [ 371 'conditions': [
372 ['android_full_debug==0', { 372 ['android_full_debug==0', {
373 # Disable full debug if we want a faster v8 in a debug build. 373 # Disable full debug if we want a faster v8 in a debug build.
374 # TODO(2304): pass DISABLE_DEBUG_ASSERT instead of hiding DEBUG. 374 # TODO(2304): pass DISABLE_DEBUG_ASSERT instead of hiding DEBUG.
375 'defines!': [ 375 'defines!': [
376 'DEBUG', 376 'DEBUG',
377 ], 377 ],
378 }], 378 }],
379 ], 379 ],
380 }], 380 }],
381 ], 381 ],
382 }, # Debug 382 }, # Debug
383 'Release': { 383 'Release': {
384 'variables': {
385 'v8_enable_extra_checks%': 0,
386 },
384 'conditions': [ 387 'conditions': [
388 ['v8_enable_extra_checks==1', {
389 'defines': ['ENABLE_EXTRA_CHECKS',],
390 }],
385 ['OS=="linux" or OS=="freebsd" or OS=="openbsd" or OS=="netbsd" \ 391 ['OS=="linux" or OS=="freebsd" or OS=="openbsd" or OS=="netbsd" \
386 or OS=="android"', { 392 or OS=="android"', {
387 'cflags!': [ 393 'cflags!': [
388 '-O2', 394 '-O2',
389 '-Os', 395 '-Os',
390 ], 396 ],
391 'cflags': [ 397 'cflags': [
392 '-fdata-sections', 398 '-fdata-sections',
393 '-ffunction-sections', 399 '-ffunction-sections',
394 '-O3', 400 '-O3',
(...skipping 43 matching lines...) Expand 10 before | Expand all | Expand 10 after
438 'OptimizeReferences': '2', 444 'OptimizeReferences': '2',
439 'EnableCOMDATFolding': '2', 445 'EnableCOMDATFolding': '2',
440 }, 446 },
441 }, 447 },
442 }], # OS=="win" 448 }], # OS=="win"
443 ], # conditions 449 ], # conditions
444 }, # Release 450 }, # Release
445 }, # configurations 451 }, # configurations
446 }, # target_defaults 452 }, # target_defaults
447 } 453 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698