chrome/app/app_asan.saves - Issue 7828043: If the 'asan' GYP variable is set to 1 on Mac, do not strip the __asan_init symbol from the binar...

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Side by Side Diff: chrome/app/app_asan.saves

Issue 7828043: If the 'asan' GYP variable is set to 1 on Mac, do not strip the __asan_init symbol from the binar... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/

Patch Set: '' Created 9 years, 3 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

OLD	NEW
(Empty)
	1 # Copyright (c) 2011 The Chromium Authors. All rights reserved.

	2 # Use of this source code is governed by a BSD-style license that can be

	3 # found in the LICENSE file.

	4

	5 # This file lists symbols that should not be stripped from the Mac browser and

	6 # helper app executables built with AddressSanitizer

	7 # (http://dev.chromium.org/developers/testing/addresssanitizer).

	8

	9 # The list of browser-specific symbols should be the same as that
	Mark Mentovai 2011/09/02 14:38:14 “Browser” is the wrong term here, it’s too specifi “Browser” is the wrong term here, it’s too specific. As the comment above indicates, this file is used for the browser and helpers. The same applies to the use of “browser” on line 14. _com_google_glider 2011/09/02 14:48:53 Chromium applications/Chromium? Show quoted text On 2011/09/02 14:38:14, Mark Mentovai wrote: > “Browser” is the wrong term here, it’s too specific. As the comment above > indicates, this file is used for the browser and helpers. > > The same applies to the use of “browser” on line 14. Chromium applications/Chromium?
	10 # in chrome/app/app.saves

	11 # The list of symbols that should be preserved in all the binaries built with

	12 # AddressSanitizer should correspond to build/asan.saves

	13

	14 # Browser-specific symbols.

	15 _main

	16 _NXArgc

	17 _NXArgv

	18

	19 # AddressSanitizer-specific symbols.

	20 ___asan_init

OLD	NEW

« build/asan.saves ('K') | « build/common.gypi ('k') | chrome/chrome.gyp » ('j') | no next file with comments »