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

Issue 339663009: MSan: introduce a GYP flag to control origin tracking. (Closed)

Created:
6 years, 6 months ago by earthdok
Modified:
6 years, 6 months ago
CC:
chromium-reviews, erikwright+watch_chromium.org, eugenis, inferno
Project:
chromium
Visibility:
Public.

Description

MSan: introduce a GYP flag to control origin tracking. By setting msan_track_origins, the user can now choose between no origin tracking (0), simple tracking (1) and chained origins (2). The default is still 1. Note that this doesn't affect instrumented libraries build, and is not currently compatible with instrumented libraries for values of msan_track_origins other than 1. BUG=178409 TBR=glider@chromium.org, thakis@chromium.org NOTRY=true TEST=compile Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=279127

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+11 lines, -7 lines) Patch
M base/base.gyp View 1 chunk +3 lines, -5 lines 0 comments Download
M build/common.gypi View 4 chunks +8 lines, -2 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
earthdok
ptal Nico as owner for /base/base.gyp (trivial change to correctly filter out the flag when ...
6 years, 6 months ago (2014-06-23 17:09:05 UTC) #1
earthdok
The CQ bit was checked by earthdok@chromium.org
6 years, 6 months ago (2014-06-23 17:11:52 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/earthdok@chromium.org/339663009/1
6 years, 6 months ago (2014-06-23 17:12:46 UTC) #3
commit-bot: I haz the power
6 years, 6 months ago (2014-06-23 17:14:33 UTC) #4
Message was sent while issue was closed.
Change committed as 279127

Powered by Google App Engine
This is Rietveld 408576698