[crankshaft] Generate lazy bailout point for HProlog instruction only when it is necessary.

[crankshaft] Generate lazy bailout point for HProlog instruction only when it is necessary.

This fix removes unnecessary nops from function prolog and seems to recover performance regression on some of SunSpider benchmarks.

TBR=bmeurer@chromium.org
BUG=chromium:531369
LOG=N

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

[Igor Sheludko, 2016-03-15 18:41:24]

The CQ bit was checked by ishell@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-03-15 18:41:31]

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

[Igor Sheludko, 2016-03-15 18:58:58]

ishell@chromium.org changed reviewers:
+ mstarzinger@chromium.org

[Igor Sheludko, 2016-03-15 18:58:58]

PTAL

[commit-bot: I haz the power, 2016-03-15 19:01:21]

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

[commit-bot: I haz the power, 2016-03-15 19:01:22]

Dry run: This issue passed the CQ dry run.

[Igor Sheludko, 2016-03-15 19:21:08]

Description was changed from

==========
[crankshaft] Generate lazy bailout point for HProlog instruction only when it is
necessary.

This fix removes unnecessary nop instructions from the function prolog and seems
to recover regression on some SunSpider benchmarks.

BUG=chromium:531369
LOG=N
==========

to

==========
[crankshaft] Generate lazy bailout point for HProlog instruction only when it is
necessary.

This fix removes unnecessary nops from the function prolog and seems to recover
performance regression on some of SunSpider benchmarks.

BUG=chromium:531369
LOG=N
==========

[Igor Sheludko, 2016-03-15 19:21:31]

Description was changed from

==========
[crankshaft] Generate lazy bailout point for HProlog instruction only when it is
necessary.

This fix removes unnecessary nops from the function prolog and seems to recover
performance regression on some of SunSpider benchmarks.

BUG=chromium:531369
LOG=N
==========

to

==========
[crankshaft] Generate lazy bailout point for HProlog instruction only when it is
necessary.

This fix removes unnecessary nops from function prolog and seems to recover
performance regression on some of SunSpider benchmarks.

BUG=chromium:531369
LOG=N
==========

[Michael Starzinger, 2016-03-16 09:30:00]

LGTM.

[Igor Sheludko, 2016-03-16 11:12:28]

The CQ bit was checked by ishell@chromium.org

[commit-bot: I haz the power, 2016-03-16 11:12:36]

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

[commit-bot: I haz the power, 2016-03-16 11:16:23]

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

[commit-bot: I haz the power, 2016-03-16 11:16: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/12424)

[Igor Sheludko, 2016-03-16 11:17:00]

Description was changed from

==========
[crankshaft] Generate lazy bailout point for HProlog instruction only when it is
necessary.

This fix removes unnecessary nops from function prolog and seems to recover
performance regression on some of SunSpider benchmarks.

BUG=chromium:531369
LOG=N
==========

to

==========
[crankshaft] Generate lazy bailout point for HProlog instruction only when it is
necessary.

This fix removes unnecessary nops from function prolog and seems to recover
performance regression on some of SunSpider benchmarks.

TBR=bmeurer@chromium.org
BUG=chromium:531369
LOG=N
==========

[Igor Sheludko, 2016-03-16 11:17:05]

The CQ bit was checked by ishell@chromium.org

[commit-bot: I haz the power, 2016-03-16 11:17:13]

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

[commit-bot: I haz the power, 2016-03-16 11:19:14]

Description was changed from

==========
[crankshaft] Generate lazy bailout point for HProlog instruction only when it is
necessary.

This fix removes unnecessary nops from function prolog and seems to recover
performance regression on some of SunSpider benchmarks.

TBR=bmeurer@chromium.org
BUG=chromium:531369
LOG=N
==========

to

==========
[crankshaft] Generate lazy bailout point for HProlog instruction only when it is
necessary.

This fix removes unnecessary nops from function prolog and seems to recover
performance regression on some of SunSpider benchmarks.

TBR=bmeurer@chromium.org
BUG=chromium:531369
LOG=N
==========

[commit-bot: I haz the power, 2016-03-16 11:19:14]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-16 11:21:00]

Description was changed from

==========
[crankshaft] Generate lazy bailout point for HProlog instruction only when it is
necessary.

This fix removes unnecessary nops from function prolog and seems to recover
performance regression on some of SunSpider benchmarks.

TBR=bmeurer@chromium.org
BUG=chromium:531369
LOG=N
==========

to

==========
[crankshaft] Generate lazy bailout point for HProlog instruction only when it is
necessary.

This fix removes unnecessary nops from function prolog and seems to recover
performance regression on some of SunSpider benchmarks.

TBR=bmeurer@chromium.org
BUG=chromium:531369
LOG=N

Committed: https://crrev.com/b82e10d39d04d769794223ffea4b8c1bd29be1c5
Cr-Commit-Position: refs/heads/master@{#34810}
==========

[commit-bot: I haz the power, 2016-03-16 11:21:01]

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