Fix mistake in FastNewFunctionContextStub

Fix mistake in FastNewFunctionContextStub

We assumed we could use a do..while, but the requested amount of slots could've
been 0 and we already started at MIN_CONTEXT_SLOTS. In that case the loop
should've been skipped.

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

[klaasb, 2016-08-09 14:55:32]

klaasb@google.com changed reviewers:
+ bmeurer@chromium.org, rmcilroy@chromium.org

[klaasb, 2016-08-09 14:55:32]

Fix for a wrong assumption in the original CL.

[klaasb, 2016-08-09 14:57:01]

Description was changed from

==========
Fix mistake in FastNewFunctionContextStub

We assumed we could use a do..while, but the requested amount of slots could've
been 0 and we already started at MIN_CONTEXT_SLOTS. In that case the loop
should've been skipped.

BUG=v8:4820
LOG=n
==========

to

==========
Fix mistake in FastNewFunctionContextStub

We assumed we could use a do..while, but the requested amount of slots could've
been 0 and we already started at MIN_CONTEXT_SLOTS. In that case the loop
should've been skipped.
==========

[Michael Starzinger, 2016-08-09 14:59:25]

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

[Michael Starzinger, 2016-08-09 14:59:25]

LGTM.

[klaasb, 2016-08-09 15:00:45]

The CQ bit was checked by klaasb@google.com

[commit-bot: I haz the power, 2016-08-09 15:00:48]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[rmcilroy, 2016-08-09 15:05:41]

lgtm

[commit-bot: I haz the power, 2016-08-09 15:20:58]

Description was changed from

==========
Fix mistake in FastNewFunctionContextStub

We assumed we could use a do..while, but the requested amount of slots could've
been 0 and we already started at MIN_CONTEXT_SLOTS. In that case the loop
should've been skipped.
==========

to

==========
Fix mistake in FastNewFunctionContextStub

We assumed we could use a do..while, but the requested amount of slots could've
been 0 and we already started at MIN_CONTEXT_SLOTS. In that case the loop
should've been skipped.
==========

[commit-bot: I haz the power, 2016-08-09 15:20:58]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-09 15:21:19]

Description was changed from

==========
Fix mistake in FastNewFunctionContextStub

We assumed we could use a do..while, but the requested amount of slots could've
been 0 and we already started at MIN_CONTEXT_SLOTS. In that case the loop
should've been skipped.
==========

to

==========
Fix mistake in FastNewFunctionContextStub

We assumed we could use a do..while, but the requested amount of slots could've
been 0 and we already started at MIN_CONTEXT_SLOTS. In that case the loop
should've been skipped.

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

[commit-bot: I haz the power, 2016-08-09 15:21:20]

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