change inline function optimization from /Ob0 to /Ob1. xref http://code.google.com/p/drmemory/issue…

change inline function optimization from /Ob0 to /Ob1 for build_for_tool=drmemory. xref http://code.google.com/p/drmemory/issues/detail?id=1054 
R=bruening@chromium.org
BUG=DrM-i#1054


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=162712

[Derek Bruening, 2012-10-18 04:11:25]

LGTM if the comment is updated.  suggestion: I'd also preface the commit message
with "for build_for_tool=drmemory" for context.

https://codereview.chromium.org/11188044/diff/1/build/common.gypi
File build/common.gypi (right):

https://codereview.chromium.org/11188044/diff/1/build/common.gypi#newcode1377
build/common.gypi:1377: 'win_release_InlineFunctionExpansion': '1',
expand the comment to explain why it's 1, as the current comment makes it sound
like it should be 0.  xref the issue w/ the compiler bug.

[commit-bot: I haz the power, 2012-10-18 13:45:14]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a lowly provisional 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.

[Reid Kleckner, 2012-10-18 13:51:06]

LGTM

Let's see if this works.

[commit-bot: I haz the power, 2012-10-18 13:51:24]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/zhaoqin@google.com/11188044/1002

[commit-bot: I haz the power, 2012-10-18 15:45:17]

Change committed as 162712