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

Side by Side Diff: build/common.gypi

Issue 18316006: Introduce a handle zapping setting, and enable it by default for release and debug (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Created 7 years, 5 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
« 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 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 472 matching lines...) Expand 10 before | Expand all | Expand 10 after
483 'cflags': [ '-I/usr/local/include' ], 483 'cflags': [ '-I/usr/local/include' ],
484 }], 484 }],
485 ['OS=="netbsd"', { 485 ['OS=="netbsd"', {
486 'cflags': [ '-I/usr/pkg/include' ], 486 'cflags': [ '-I/usr/pkg/include' ],
487 }], 487 }],
488 ], # conditions 488 ], # conditions
489 'configurations': { 489 'configurations': {
490 'Debug': { 490 'Debug': {
491 'variables': { 491 'variables': {
492 'v8_enable_extra_checks%': 1, 492 'v8_enable_extra_checks%': 1,
493 'v8_enable_handle_zapping%': 1,
493 }, 494 },
494 'defines': [ 495 'defines': [
495 'DEBUG', 496 'DEBUG',
496 'ENABLE_DISASSEMBLER', 497 'ENABLE_DISASSEMBLER',
497 'V8_ENABLE_CHECKS', 498 'V8_ENABLE_CHECKS',
498 'OBJECT_PRINT', 499 'OBJECT_PRINT',
499 'VERIFY_HEAP', 500 'VERIFY_HEAP',
500 ], 501 ],
501 'msvs_settings': { 502 'msvs_settings': {
502 'VCCLCompilerTool': { 503 'VCCLCompilerTool': {
503 'Optimization': '0', 504 'Optimization': '0',
504 505
505 'conditions': [ 506 'conditions': [
506 ['OS=="win" and component=="shared_library"', { 507 ['OS=="win" and component=="shared_library"', {
507 'RuntimeLibrary': '3', # /MDd 508 'RuntimeLibrary': '3', # /MDd
508 }, { 509 }, {
509 'RuntimeLibrary': '1', # /MTd 510 'RuntimeLibrary': '1', # /MTd
510 }], 511 }],
511 ], 512 ],
512 }, 513 },
513 'VCLinkerTool': { 514 'VCLinkerTool': {
514 'LinkIncremental': '2', 515 'LinkIncremental': '2',
515 }, 516 },
516 }, 517 },
517 'conditions': [ 518 'conditions': [
518 ['v8_enable_extra_checks==1', { 519 ['v8_enable_extra_checks==1', {
519 'defines': ['ENABLE_EXTRA_CHECKS',], 520 'defines': ['ENABLE_EXTRA_CHECKS',],
520 }], 521 }],
522 ['v8_enable_handle_zapping==1', {
523 'defines': ['ENABLE_HANDLE_ZAPPING',],
524 }],
521 ['OS=="linux" or OS=="freebsd" or OS=="openbsd" or OS=="netbsd"', { 525 ['OS=="linux" or OS=="freebsd" or OS=="openbsd" or OS=="netbsd"', {
522 'cflags': [ '-Wall', '<(werror)', '-W', '-Wno-unused-parameter', 526 'cflags': [ '-Wall', '<(werror)', '-W', '-Wno-unused-parameter',
523 '-Wnon-virtual-dtor', '-Woverloaded-virtual' ], 527 '-Wnon-virtual-dtor', '-Woverloaded-virtual' ],
524 }], 528 }],
525 ['OS=="linux" and v8_enable_backtrace==1', { 529 ['OS=="linux" and v8_enable_backtrace==1', {
526 # Support for backtrace_symbols. 530 # Support for backtrace_symbols.
527 'ldflags': [ '-rdynamic' ], 531 'ldflags': [ '-rdynamic' ],
528 }], 532 }],
529 ['OS=="android"', { 533 ['OS=="android"', {
530 'variables': { 534 'variables': {
(...skipping 12 matching lines...) Expand all
543 ['OS=="mac"', { 547 ['OS=="mac"', {
544 'xcode_settings': { 548 'xcode_settings': {
545 'GCC_OPTIMIZATION_LEVEL': '0', # -O0 549 'GCC_OPTIMIZATION_LEVEL': '0', # -O0
546 }, 550 },
547 }], 551 }],
548 ], 552 ],
549 }, # Debug 553 }, # Debug
550 'Release': { 554 'Release': {
551 'variables': { 555 'variables': {
552 'v8_enable_extra_checks%': 0, 556 'v8_enable_extra_checks%': 0,
557 'v8_enable_handle_zapping%': 1,
553 }, 558 },
554 'conditions': [ 559 'conditions': [
555 ['v8_enable_extra_checks==1', { 560 ['v8_enable_extra_checks==1', {
556 'defines': ['ENABLE_EXTRA_CHECKS',], 561 'defines': ['ENABLE_EXTRA_CHECKS',],
557 }], 562 }],
563 ['v8_enable_handle_zapping==1', {
564 'defines': ['ENABLE_HANDLE_ZAPPING',],
565 }],
558 ['OS=="linux" or OS=="freebsd" or OS=="openbsd" or OS=="netbsd"', { 566 ['OS=="linux" or OS=="freebsd" or OS=="openbsd" or OS=="netbsd"', {
559 'cflags!': [ 567 'cflags!': [
560 '-O2', 568 '-O2',
561 '-Os', 569 '-Os',
562 ], 570 ],
563 'cflags': [ 571 'cflags': [
564 '-fdata-sections', 572 '-fdata-sections',
565 '-ffunction-sections', 573 '-ffunction-sections',
566 '-O3', 574 '-O3',
567 ], 575 ],
(...skipping 61 matching lines...) Expand 10 before | Expand all | Expand 10 after
629 'OptimizeReferences': '2', 637 'OptimizeReferences': '2',
630 'EnableCOMDATFolding': '2', 638 'EnableCOMDATFolding': '2',
631 }, 639 },
632 }, 640 },
633 }], # OS=="win" 641 }], # OS=="win"
634 ], # conditions 642 ], # conditions
635 }, # Release 643 }, # Release
636 }, # configurations 644 }, # configurations
637 }, # target_defaults 645 }, # target_defaults
638 } 646 }
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