Fix gcc 4.9.2 signed-compare error.

Fix gcc 4.9.2 signed-compare error.

BUG=

Committed: https://crrev.com/f5c227b27dc85cdb011b96358f71c33033d70c6e
Cr-Commit-Position: refs/heads/master@{#31497}

[akos.palfi.imgtec, 2015-10-22 18:15:36]

akos.palfi@imgtec.com changed reviewers:
+ machenbach@chromium.org, mstarzinger@chromium.org, paul.lind@imgtec.com

[akos.palfi.imgtec, 2015-10-22 18:15:36]

Hi guys,

Google team, Paul: It fixes a gcc compilation error, PTAL.

[paul.l..., 2015-10-22 18:49:03]

https://codereview.chromium.org/1418693005/diff/1/src/compiler/js-inlining.cc
File src/compiler/js-inlining.cc (right):

https://codereview.chromium.org/1418693005/diff/1/src/compiler/js-inlining.cc...
src/compiler/js-inlining.cc:417:
DCHECK_EQ(static_cast<int32_t>(parameter_count),
nit: ValueOutputCount() is 'int', seems we should cast parameter_count to 'int'
as well. (None of the compilers I tried whined about this, but I only tried a
couple 64-bit builds.)

[akos.palfi.imgtec, 2015-10-22 18:58:33]

The CQ bit was checked by akos.palfi@imgtec.com to run a CQ dry run

[commit-bot: I haz the power, 2015-10-22 18:58:39]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1418693005/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1418693005/20001

[akos.palfi.imgtec, 2015-10-22 18:59:14]

Thanks! Nit is fixed.

https://codereview.chromium.org/1418693005/diff/1/src/compiler/js-inlining.cc
File src/compiler/js-inlining.cc (right):

https://codereview.chromium.org/1418693005/diff/1/src/compiler/js-inlining.cc...
src/compiler/js-inlining.cc:417:
DCHECK_EQ(static_cast<int32_t>(parameter_count),
On 2015/10/22 18:49:03, paul.l... wrote:
> nit: ValueOutputCount() is 'int', seems we should cast parameter_count to
'int'
> as well. (None of the compilers I tried whined about this, but I only tried a
> couple 64-bit builds.)

Done.

[paul.l..., 2015-10-22 19:10:21]

my l-g-t-m wouldn't help. Can one of you guys PTAL?

[commit-bot: I haz the power, 2015-10-22 19:19:21]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-10-22 19:19:21]

Dry run: This issue passed the CQ dry run.

[Michael Achenbach, 2015-10-23 07:23:52]

machenbach@chromium.org changed reviewers:
+ bmeurer@chromium.org

[Michael Achenbach, 2015-10-23 07:23:52]

lgtm (stamp)

[Michael Starzinger, 2015-10-23 07:48:17]

LGTM. Thanks!

[Michael Achenbach, 2015-10-23 08:19:21]

The CQ bit was checked by machenbach@chromium.org

[commit-bot: I haz the power, 2015-10-23 08:19:31]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1418693005/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1418693005/20001

[commit-bot: I haz the power, 2015-10-23 08:21:00]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-10-23 08:21:35]

Patchset 2 (id:??) landed as
https://crrev.com/f5c227b27dc85cdb011b96358f71c33033d70c6e
Cr-Commit-Position: refs/heads/master@{#31497}