CpuProfiler: fix for CollectDeoptEvents test on arm

CpuProfiler: fix for CollectDeoptEvents test on arm

The same idea as in https://codereview.chromium.org/984893003/

BUG=chromium:452067
LOG=n

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

[loislo, 2015-03-10 12:26:44]

loislo@chromium.org changed reviewers:
+ alph@chromium.org, jacob.bramley@arm.com, svenpanne@chromium.org,
yurys@chromium.org

[loislo, 2015-03-10 12:26:44]

PTAL

[jbramley, 2015-03-10 14:24:40]

https://codereview.chromium.org/997513003/diff/1/src/arm/lithium-codegen-arm.cc
File src/arm/lithium-codegen-arm.cc (right):

https://codereview.chromium.org/997513003/diff/1/src/arm/lithium-codegen-arm....
src/arm/lithium-codegen-arm.cc:356: bool need_branch = ((i + 1) != length);
Because every entry (including the last) now has a branch, need_branch should
effectively always be true.

[loislo, 2015-03-10 14:30:37]

https://codereview.chromium.org/997513003/diff/1/src/arm/lithium-codegen-arm.cc
File src/arm/lithium-codegen-arm.cc (right):

https://codereview.chromium.org/997513003/diff/1/src/arm/lithium-codegen-arm....
src/arm/lithium-codegen-arm.cc:356: bool need_branch = ((i + 1) != length);
On 2015/03/10 14:24:40, jbramley wrote:
> Because every entry (including the last) now has a branch, need_branch should
> effectively always be true.

Done.

[jbramley, 2015-03-10 14:34:23]

lgtm

[loislo, 2015-03-10 14:36:23]

The CQ bit was checked by loislo@chromium.org

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

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

[commit-bot: I haz the power, 2015-03-10 14:56:23]

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

[commit-bot: I haz the power, 2015-03-10 14:56:24]

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

[Sven Panne, 2015-03-10 15:00:53]

The CQ bit was checked by svenpanne@chromium.org

[Sven Panne, 2015-03-10 15:00:53]

lgtm

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

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

[commit-bot: I haz the power, 2015-03-10 15:15:17]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-03-10 15:15:30]

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