Fix another gcc 4.9.2 signed-compare error.

Fix another gcc 4.9.2 signed-compare error.

BUG=

Committed: https://crrev.com/9d66c8813aea9d573020dbf6bf28533d1183c490
Cr-Commit-Position: refs/heads/master@{#31199}

[paul.l..., 2015-10-09 00:12:10]

paul.lind@imgtec.com changed reviewers:
+ bradnelson@google.com, machenbach@google.com, titzer@chromium.org

[paul.l..., 2015-10-09 00:12:10]

PTAL.

[bradn, 2015-10-09 00:19:55]

lgtm

https://codereview.chromium.org/1397003002/diff/1/test/unittests/compiler/com...
File test/unittests/compiler/common-operator-unittest.cc (right):

https://codereview.chromium.org/1397003002/diff/1/test/unittests/compiler/com...
test/unittests/compiler/common-operator-unittest.cc:197: EXPECT_EQ(1u,
static_cast<uint32_t>(op->ControlInputCount()));
For this one you can just drop the cast.

[paul.l..., 2015-10-09 00:42:02]

The CQ bit was checked by paul.lind@imgtec.com to run a CQ dry run

[commit-bot: I haz the power, 2015-10-09 00:42:23]

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

[paul.l..., 2015-10-09 00:42:38]

Thanks Brad, I removed both casts, it seems fine, but please take another look.

https://codereview.chromium.org/1397003002/diff/1/test/unittests/compiler/com...
File test/unittests/compiler/common-operator-unittest.cc (right):

https://codereview.chromium.org/1397003002/diff/1/test/unittests/compiler/com...
test/unittests/compiler/common-operator-unittest.cc:197: EXPECT_EQ(1u,
static_cast<uint32_t>(op->ControlInputCount()));
On 2015/10/09 00:19:54, bradn wrote:
> For this one you can just drop the cast.

Done.

https://codereview.chromium.org/1397003002/diff/1/test/unittests/compiler/com...
test/unittests/compiler/common-operator-unittest.cc:200:
static_cast<uint32_t>(OperatorProperties::GetTotalInputCount(op)));
I removed the cast here was well. input_count and GetTotalInputCount() are both
ints.

[commit-bot: I haz the power, 2015-10-09 01:09:14]

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

[commit-bot: I haz the power, 2015-10-09 01:09:15]

Dry run: This issue passed the CQ dry run.

[bradn, 2015-10-09 16:00:45]

lgtm

[bradn, 2015-10-09 16:00:50]

The CQ bit was checked by bradnelson@google.com

[commit-bot: I haz the power, 2015-10-09 16:00:55]

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

[commit-bot: I haz the power, 2015-10-09 16:02:49]

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

[commit-bot: I haz the power, 2015-10-09 16:02:50]

Try jobs failed on following builders:
  v8_presubmit on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/6531)

[paul.l..., 2015-10-09 16:14:34]

On 2015/10/09 16:02:50, commit-bot: I haz the power wrote:
>   v8_presubmit on tryserver.v8 (JOB_FAILED,

Michael, Ben, could one of you give the OK on this one?

[Michael Achenbach, 2015-10-09 18:49:44]

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

[Michael Achenbach, 2015-10-09 18:49:44]

lgtm

[bradn, 2015-10-09 20:02:10]

The CQ bit was checked by bradnelson@google.com

[commit-bot: I haz the power, 2015-10-09 20:02:25]

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

[commit-bot: I haz the power, 2015-10-09 20:04:02]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-10-09 20:04:19]

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