Chromium Code Reviews
|
|
Chromium Code Reviews
Issue
1304313002:
Add default MSan options. (Closed)
DescriptionAdd default MSan options.
Disable memcmp() interceptor pending cleanup.
Strip generated source path from reports (the same as other sanitizers).
BUG=523384, 523428
NOTRY=true
Committed: https://crrev.com/8281c52db39e19b1e6678f31e70991512c9a4acc
Cr-Commit-Position: refs/heads/master@{#344855}
Patch Set 1 #
Messages
Total messages: 20 (8 generated)
eugenis@chromium.org changed reviewers: + glider@chromium.org, kcc@chromium.org, samsonov@google.com
samsonov@chromium.org changed reviewers: + samsonov@chromium.org
lgtm
The CQ bit was checked by eugenis@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1304313002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1304313002/1
The CQ bit was unchecked by commit-bot@chromium.org
No L-G-T-M from a valid reviewer yet. Only full committers are accepted. Even if an L-G-T-M may have been provided, it was from a non-committer, _not_ a full super star committer. See http://www.chromium.org/getting-involved/become-a-committer Note that this has nothing to do with OWNERS files.
LGTM
The CQ bit was checked by eugenis@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1304313002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1304313002/1
The CQ bit was unchecked by commit-bot@chromium.org
No L-G-T-M from a valid reviewer yet. Only full committers are accepted. Even if an L-G-T-M may have been provided, it was from a non-committer, _not_ a full super star committer. See http://www.chromium.org/getting-involved/become-a-committer Note that this has nothing to do with OWNERS files.
On 2015/08/21 18:37:07, commit-bot: I haz the power wrote: > No L-G-T-M from a valid reviewer yet. Only full committers are accepted. > Even if an L-G-T-M may have been provided, it was from a non-committer, > _not_ a full super star committer. > See http://www.chromium.org/getting-involved/become-a-committer > Note that this has nothing to do with OWNERS files. Does this mean that being full committer+owner is not enough, and I need an LGTM from another full committer?
hans@chromium.org changed reviewers: + hans@chromium.org
lgtm+++
The CQ bit was checked by eugenis@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1304313002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1304313002/1
Message was sent while issue was closed.
Committed patchset #1 (id:1)
Message was sent while issue was closed.
Patchset 1 (id:??) landed as https://crrev.com/8281c52db39e19b1e6678f31e70991512c9a4acc Cr-Commit-Position: refs/heads/master@{#344855} |
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
