[arm] Address TODO in CallConstructStub::Generate.

[arm] Address TODO in CallConstructStub::Generate.

R=jacob.bramley@arm.com

Committed: https://crrev.com/1b616a682f9ae4adb241612e6147023e55d3db2e
Cr-Commit-Position: refs/heads/master@{#29774}

[Michael Starzinger, 2015-07-21 13:22:18]

mstarzinger@chromium.org changed reviewers:
+ jacob.bramley@arm.com

[Michael Starzinger, 2015-07-21 13:22:26]

https://codereview.chromium.org/1244023002/diff/1/src/arm/code-stubs-arm.cc
File src/arm/code-stubs-arm.cc (right):

https://codereview.chromium.org/1244023002/diff/1/src/arm/code-stubs-arm.cc#n...
src/arm/code-stubs-arm.cc:2398: __ Push(r4);
Please don't make me implement the Push() variant with 5 registers. :)

[jbramley, 2015-07-21 13:28:02]

lgtm

https://codereview.chromium.org/1244023002/diff/1/src/arm/code-stubs-arm.cc
File src/arm/code-stubs-arm.cc (right):

https://codereview.chromium.org/1244023002/diff/1/src/arm/code-stubs-arm.cc#n...
src/arm/code-stubs-arm.cc:2398: __ Push(r4);
On 2015/07/21 13:22:26, Michael Starzinger wrote:
> Please don't make me implement the Push() variant with 5 registers. :)

It doesn't matter quite as much for ARM anyway, since it doesn't have the
per-push sp update overhead.

[Michael Starzinger, 2015-07-21 13:53:56]

The CQ bit was checked by mstarzinger@chromium.org

[commit-bot: I haz the power, 2015-07-21 13:54:08]

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

[commit-bot: I haz the power, 2015-07-21 14:13:22]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-07-21 14:13:45]

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