Reshuffle registers in JSConstructStub to avoid trashing costructor and new.target on fast path (so…

Reshuffle registers in JSConstructStub to avoid trashing costructor and new.target on fast path (so we don't need to push/pop them).

This CL also fixed register usages in MacroAssembler::Allocate() broken by 2fc2cb99 (r32144).

BUG=chromium:560239
LOG=Y

Committed: https://crrev.com/0ef5ad5ab97657f7e3393be7512fef883b205c87
Cr-Commit-Position: refs/heads/master@{#32219}

[Igor Sheludko, 2015-11-24 11:32:23]

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

[commit-bot: I haz the power, 2015-11-24 11:32:33]

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

[Igor Sheludko, 2015-11-24 11:49:06]

Description was changed from

==========
Reshuffle registers in JSConstructStub to avoid trashing costructor and
new.target on fast path (so we don't need to push/pop them).

This CL also fixed register usages in MacroAssembler::Allocate() broken by
2fc2cb99 (r32144).

BUG=chromium:560239
LOG=Y
==========

to

==========
Reshuffle registers in JSConstructStub to avoid trashing costructor and
new.target on fast path (so we don't need to push/pop them).

This CL also fixed register usages in MacroAssembler::Allocate() broken by
2fc2cb99 (r32144).

BUG=chromium:560239
LOG=Y
==========

[Igor Sheludko, 2015-11-24 11:49:52]

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

[Igor Sheludko, 2015-11-24 11:49:52]

PTAL

[Igor Sheludko, 2015-11-24 11:50:52]

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

[commit-bot: I haz the power, 2015-11-24 11:51:01]

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

[commit-bot: I haz the power, 2015-11-24 12:14:43]

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

[commit-bot: I haz the power, 2015-11-24 12:14:44]

Dry run: This issue passed the CQ dry run.

[Igor Sheludko, 2015-11-24 13:49:12]

Patchset #1 (id:1) has been deleted

[Toon Verwaest, 2015-11-24 14:55:50]

lgtm, much easier to read...

[Toon Verwaest, 2015-11-24 14:57:42]

The CQ bit was checked by verwaest@chromium.org

[commit-bot: I haz the power, 2015-11-24 14:57:59]

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

[commit-bot: I haz the power, 2015-11-24 15:36:38]

Committed patchset #1 (id:20001)

[commit-bot: I haz the power, 2015-11-24 15:37:13]

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