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

Side by Side Diff: build/common.gypi

Issue 10636055: Enable whole program optimization for WIN64 to make build-bot green. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Created 8 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 | « 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 362 matching lines...) Expand 10 before | Expand all | Expand 10 after
373 }], # OS=="mac" 373 }], # OS=="mac"
374 ['OS=="win"', { 374 ['OS=="win"', {
375 'msvs_settings': { 375 'msvs_settings': {
376 'VCCLCompilerTool': { 376 'VCCLCompilerTool': {
377 'Optimization': '2', 377 'Optimization': '2',
378 'InlineFunctionExpansion': '2', 378 'InlineFunctionExpansion': '2',
379 'EnableIntrinsicFunctions': 'true', 379 'EnableIntrinsicFunctions': 'true',
380 'FavorSizeOrSpeed': '0', 380 'FavorSizeOrSpeed': '0',
381 'OmitFramePointers': 'true', 381 'OmitFramePointers': 'true',
382 'StringPooling': 'true', 382 'StringPooling': 'true',
383
384 'conditions': [ 383 'conditions': [
385 ['OS=="win" and component=="shared_library"', { 384 ['OS=="win" and component=="shared_library"', {
386 'RuntimeLibrary': '2', #/MD 385 'RuntimeLibrary': '2', #/MD
387 }, { 386 }, {
388 'RuntimeLibrary': '0', #/MT 387 'RuntimeLibrary': '0', #/MT
389 }], 388 }],
389 ['v8_target_arch=="x64"', {
390 # TODO(V8:2207): remove this option once the bug is fixed.
391 'WholeProgramOptimization': 'true',
392 }]
Jakob Kummerow 2012/06/26 15:07:56 nit: missing comma
390 ], 393 ],
391 }, 394 },
392 'VCLinkerTool': { 395 'VCLinkerTool': {
393 'LinkIncremental': '1', 396 'LinkIncremental': '1',
394 'OptimizeReferences': '2', 397 'OptimizeReferences': '2',
395 'EnableCOMDATFolding': '2', 398 'EnableCOMDATFolding': '2',
396 }, 399 },
397 }, 400 },
398 }], # OS=="win" 401 }], # OS=="win"
399 ], # conditions 402 ], # conditions
400 }, # Release 403 }, # Release
401 }, # configurations 404 }, # configurations
402 }, # target_defaults 405 }, # target_defaults
403 } 406 }
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