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

Issue 1513043: Add the linux_use_debugallocation flag.... (Closed)

Created:
10 years, 8 months ago by Alexander Potapenko
Modified:
9 years, 6 months ago
CC:
chromium-reviews, brettw-cc_chromium.org
Visibility:
Public.

Description

Add the linux_use_debugallocation flag. Setting this flag to 1 shall force the use of debugallocation.cc instead of tcmalloc.cc The flag is off by default because it still needs testing. BUG=30715 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=44759

Patch Set 1 #

Patch Set 2 : '' #

Total comments: 4

Patch Set 3 : '' #

Patch Set 4 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+33 lines, -14 lines) Patch
M base/allocator/allocator.gyp View 1 2 3 3 chunks +30 lines, -14 lines 0 comments Download
M build/common.gypi View 1 2 3 1 chunk +3 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Alexander Potapenko
10 years, 8 months ago (2010-04-15 13:49:32 UTC) #1
willchan no longer on Chromium
Excellent, thanks for working on this! LGTM except some gyp nitpicks. http://codereview.chromium.org/1513043/diff/3001/4002 File base/allocator/allocator.gyp (right): ...
10 years, 8 months ago (2010-04-15 17:24:51 UTC) #2
Alexander Potapenko
10 years, 8 months ago (2010-04-16 07:19:56 UTC) #3
http://codereview.chromium.org/1513043/diff/3001/4002
File base/allocator/allocator.gyp (right):

http://codereview.chromium.org/1513043/diff/3001/4002#newcode339
base/allocator/allocator.gyp:339: [ 'linux_use_debugallocation==1', {
On 2010/04/15 17:24:51, willchan wrote:
> I think it's cleaner to use an 'else' rather than explicitly doing each
> condition.  What you should do is something like:
> [ 'linux_use_debugallocation==1', {
>   ...
> }, { # linux_use_debugallocation != 1
>   ...
> }],

Done.

http://codereview.chromium.org/1513043/diff/3001/4002#newcode341
base/allocator/allocator.gyp:341: '<(tcmalloc_dir)/src/tcmalloc.cc',
On 2010/04/15 17:24:51, willchan wrote:
> This stuff is magic to people who don't understand debugallocation.cc.  You
> should add a comment explaining that debugallocation.cc will #include
> tcmalloc.cc.

Done.

Powered by Google App Engine
This is Rietveld 408576698