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

Side by Side Diff: build/standalone.gypi

Issue 862313002: Fix ninja build with clang=1. (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 | « 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 15 matching lines...) Expand all
26 # OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. 26 # OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
27 27
28 # Definitions to be used when building stand-alone V8 binaries. 28 # Definitions to be used when building stand-alone V8 binaries.
29 29
30 { 30 {
31 # We need to include toolchain.gypi here for third-party sources that don't 31 # We need to include toolchain.gypi here for third-party sources that don't
32 # directly include it themselves. 32 # directly include it themselves.
33 'includes': ['toolchain.gypi'], 33 'includes': ['toolchain.gypi'],
34 'variables': { 34 'variables': {
35 'component%': 'static_library', 35 'component%': 'static_library',
36 'make_clang_dir%': '../third_party/llvm-build/Release+Asserts', 36 'clang_dir%': 'third_party/llvm-build/Release+Asserts',
37 'clang_xcode%': 0, 37 'clang_xcode%': 0,
38 'visibility%': 'hidden', 38 'visibility%': 'hidden',
39 'v8_enable_backtrace%': 0, 39 'v8_enable_backtrace%': 0,
40 'v8_enable_i18n_support%': 1, 40 'v8_enable_i18n_support%': 1,
41 'v8_deprecation_warnings': 1, 41 'v8_deprecation_warnings': 1,
42 'msvs_multi_core_compile%': '1', 42 'msvs_multi_core_compile%': '1',
43 'mac_deployment_target%': '10.5', 43 'mac_deployment_target%': '10.5',
44 'variables': { 44 'variables': {
45 'variables': { 45 'variables': {
46 'variables': { 46 'variables': {
(...skipping 433 matching lines...) Expand 10 before | Expand all | Expand 10 after
480 }], 480 }],
481 ], 481 ],
482 'target_conditions': [ 482 'target_conditions': [
483 ['_type!="static_library"', { 483 ['_type!="static_library"', {
484 'xcode_settings': {'OTHER_LDFLAGS': ['-Wl,-search_paths_first']}, 484 'xcode_settings': {'OTHER_LDFLAGS': ['-Wl,-search_paths_first']},
485 }], 485 }],
486 ], # target_conditions 486 ], # target_conditions
487 }, # target_defaults 487 }, # target_defaults
488 }], # OS=="mac" 488 }], # OS=="mac"
489 ['clang==1 and ((OS!="mac" and OS!="ios") or clang_xcode==0) ' 489 ['clang==1 and ((OS!="mac" and OS!="ios") or clang_xcode==0) '
490 'and OS!="win"', { 490 'and OS!="win" and "<(GENERATOR)"=="make"', {
491 'make_global_settings': [ 491 'make_global_settings': [
492 ['CC', '<(make_clang_dir)/bin/clang'], 492 ['CC', '../<(clang_dir)/bin/clang'],
493 ['CXX', '<(make_clang_dir)/bin/clang++'], 493 ['CXX', '../<(clang_dir)/bin/clang++'],
494 ['CC.host', '$(CC)'], 494 ['CC.host', '$(CC)'],
495 ['CXX.host', '$(CXX)'], 495 ['CXX.host', '$(CXX)'],
496 ], 496 ],
497 }],
498 ['clang==1 and ((OS!="mac" and OS!="ios") or clang_xcode==0) '
499 'and OS!="win" and "<(GENERATOR)"=="ninja"', {
500 'make_global_settings': [
501 ['CC', '<(clang_dir)/bin/clang'],
502 ['CXX', '<(clang_dir)/bin/clang++'],
503 ['CC.host', '$(CC)'],
504 ['CXX.host', '$(CXX)'],
505 ],
497 }], 506 }],
498 ['clang==1 and OS=="win"', { 507 ['clang==1 and OS=="win"', {
499 'make_global_settings': [ 508 'make_global_settings': [
500 # On Windows, gyp's ninja generator only looks at CC. 509 # On Windows, gyp's ninja generator only looks at CC.
501 ['CC', '<(make_clang_dir)/bin/clang-cl'], 510 ['CC', '../<(clang_dir)/bin/clang-cl'],
502 ], 511 ],
503 }], 512 }],
504 # TODO(yyanagisawa): supports GENERATOR==make 513 # TODO(yyanagisawa): supports GENERATOR==make
505 # make generator doesn't support CC_wrapper without CC 514 # make generator doesn't support CC_wrapper without CC
506 # in make_global_settings yet. 515 # in make_global_settings yet.
507 ['use_goma==1 and ("<(GENERATOR)"=="ninja" or clang==1)', { 516 ['use_goma==1 and ("<(GENERATOR)"=="ninja" or clang==1)', {
508 'make_global_settings': [ 517 'make_global_settings': [
509 ['CC_wrapper', '<(gomadir)/gomacc'], 518 ['CC_wrapper', '<(gomadir)/gomacc'],
510 ['CXX_wrapper', '<(gomadir)/gomacc'], 519 ['CXX_wrapper', '<(gomadir)/gomacc'],
511 ['CC.host_wrapper', '<(gomadir)/gomacc'], 520 ['CC.host_wrapper', '<(gomadir)/gomacc'],
512 ['CXX.host_wrapper', '<(gomadir)/gomacc'], 521 ['CXX.host_wrapper', '<(gomadir)/gomacc'],
513 ], 522 ],
514 }], 523 }],
515 ], 524 ],
516 } 525 }
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