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

Side by Side Diff: base/base.gyp

Issue 292123008: Never instrument the crazy linker. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Follow review Created 6 years, 7 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 | build/common.gypi » ('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 (c) 2012 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 { 5 {
6 'variables': { 6 'variables': {
7 'chromium_code': 1, 7 'chromium_code': 1,
8 }, 8 },
9 'includes': [ 9 'includes': [
10 '../build/win_precompile.gypi', 10 '../build/win_precompile.gypi',
(...skipping 1397 matching lines...) Expand 10 before | Expand all | Expand 10 after
1408 'conditions': [ 1408 'conditions': [
1409 ['android_webview_build == 0 and target_arch != "x64" and \ 1409 ['android_webview_build == 0 and target_arch != "x64" and \
1410 target_arch != "arm64"', { 1410 target_arch != "arm64"', {
1411 # Avoid breaking the webview/64-bit build because they 1411 # Avoid breaking the webview/64-bit build because they
1412 # don't have <(android_ndk_root)/crazy_linker.gyp. 1412 # don't have <(android_ndk_root)/crazy_linker.gyp.
1413 # Note that webview never uses the linker anyway. 1413 # Note that webview never uses the linker anyway.
1414 # Note there is no 64-bit support in the linker. 1414 # Note there is no 64-bit support in the linker.
1415 'sources': [ 1415 'sources': [
1416 'android/linker/linker_jni.cc', 1416 'android/linker/linker_jni.cc',
1417 ], 1417 ],
1418 # The crazy linker is never instrumented.
1419 'cflags!': [
1420 '-finstrument-functions',
1421 ],
1418 'dependencies': [ 1422 'dependencies': [
1419 '<(android_ndk_root)/crazy_linker.gyp:crazy_linker', 1423 '<(android_ndk_root)/crazy_linker.gyp:crazy_linker',
1420 ], 1424 ],
1421 }], 1425 }],
1422 ], 1426 ],
1423 }, 1427 },
1424 1428
1425 ], 1429 ],
1426 }], 1430 }],
1427 ['OS == "android"', { 1431 ['OS == "android"', {
(...skipping 56 matching lines...) Expand 10 before | Expand all | Expand 10 after
1484 'base_unittests.isolate', 1488 'base_unittests.isolate',
1485 ], 1489 ],
1486 'sources': [ 1490 'sources': [
1487 'base_unittests.isolate', 1491 'base_unittests.isolate',
1488 ], 1492 ],
1489 }, 1493 },
1490 ], 1494 ],
1491 }], 1495 }],
1492 ], 1496 ],
1493 } 1497 }
OLDNEW
« no previous file with comments | « no previous file | build/common.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698