Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(15855)

Issue 124550044: Add a new flag to use ctmalloc in BlockHeapManager. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
9 years, 8 months ago by Sébastien Marchand
Modified:
9 years, 8 months ago
Reviewers:
chrisha
CC:
sawbuck-changes_googlegroups.com
Base URL:
http://sawbuck.googlecode.com/svn/trunk
Visibility:
Public.

Description

Add a new flag to use ctmalloc in BlockHeapManager. This feature isn't supported yet, but I plan to plug it soon and then we'll have a flag to control it. BUG= R=chrisha@chromium.org Committed: https://code.google.com/p/sawbuck/source/detail?r=2261

Patch Set 1 #

Total comments: 4

Patch Set 2 : Address comments. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+36 lines, -13 lines) Patch
M syzygy/agent/asan/asan_runtime.cc View 1 1 chunk +1 line, -1 line 0 comments Download
M syzygy/agent/asan/heaps/ctmalloc_heap.h View 1 1 chunk +1 line, -1 line 0 comments Download
M syzygy/agent/asan/heaps/ctmalloc_heap.cc View 1 4 chunks +5 lines, -5 lines 0 comments Download
M syzygy/common/asan_parameters.h View 1 4 chunks +11 lines, -2 lines 0 comments Download
M syzygy/common/asan_parameters.cc View 1 4 chunks +10 lines, -1 line 0 comments Download
M syzygy/common/asan_parameters_unittest.cc View 1 4 chunks +6 lines, -1 line 0 comments Download
M syzygy/instrument/transforms/asan_transform.cc View 1 1 chunk +1 line, -1 line 0 comments Download
M third_party/ctmalloc/ctmalloc.gyp View 1 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 6
Sébastien Marchand
PTAL.
9 years, 8 months ago (2014-08-18 21:00:37 UTC) #1
chrisha
https://codereview.appspot.com/124550044/diff/1/syzygy/common/asan_parameters.h File syzygy/common/asan_parameters.h (right): https://codereview.appspot.com/124550044/diff/1/syzygy/common/asan_parameters.h#newcode99 syzygy/common/asan_parameters.h:99: // BlockHeapManager: Indicates if CTMalloc should be used to ...
9 years, 8 months ago (2014-08-18 21:05:04 UTC) #2
Sébastien Marchand
Thanks, PTAnL. https://codereview.appspot.com/124550044/diff/1/syzygy/common/asan_parameters.h File syzygy/common/asan_parameters.h (right): https://codereview.appspot.com/124550044/diff/1/syzygy/common/asan_parameters.h#newcode99 syzygy/common/asan_parameters.h:99: // BlockHeapManager: Indicates if CTMalloc should be ...
9 years, 8 months ago (2014-08-18 21:22:26 UTC) #3
chrisha
lgtm!
9 years, 8 months ago (2014-08-18 21:29:47 UTC) #4
Sébastien Marchand
Thanks, committing.
9 years, 8 months ago (2014-08-18 21:34:04 UTC) #5
Sébastien Marchand
9 years, 8 months ago (2014-08-18 23:08:17 UTC) #6
Message was sent while issue was closed.
Committed patchset #2 manually as 2261 (presubmit successful).
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b