[Interpreter] Ensure we always have an outer register allocation scope.

[Interpreter] Ensure we always have an outer register allocation scope.

Split RegisterAllocationScope out of ExpressionResult and allocate one
for each statement. This ensures that we always have an outer register
allocation scope for statement code (used in CountOperation and
RegisterExecutionResult). Also refactored the register allocator code to
move it to it's own file and rename from TemporaryRegisterScope to
BytecodeRegisterAllocator.

BUG=v8:4280
LOG=N

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

[rmcilroy, 2016-01-14 10:14:42]

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

[commit-bot: I haz the power, 2016-01-14 10:14:46]

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

[commit-bot: I haz the power, 2016-01-14 10:39:15]

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

[commit-bot: I haz the power, 2016-01-14 10:39:15]

Dry run: This issue passed the CQ dry run.

[rmcilroy, 2016-01-14 11:07:30]

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

[commit-bot: I haz the power, 2016-01-14 11:07:35]

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

[rmcilroy, 2016-01-14 11:07:47]

rmcilroy@chromium.org changed reviewers:
+ oth@chromium.org

[rmcilroy, 2016-01-14 11:07:47]

Orion, please take a look, thanks.

[commit-bot: I haz the power, 2016-01-14 12:03:06]

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

[commit-bot: I haz the power, 2016-01-14 12:03:06]

Dry run: This issue passed the CQ dry run.

[oth, 2016-01-14 13:16:10]

Awesome, lgtm.

[rmcilroy, 2016-01-14 13:42:07]

The CQ bit was checked by rmcilroy@chromium.org

[commit-bot: I haz the power, 2016-01-14 13:42:10]

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

[commit-bot: I haz the power, 2016-01-14 13:43:19]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-01-14 13:43:49]

Description was changed from

==========
[Interpreter] Ensure we always have an outer register allocation scope.

Split RegisterAllocationScope out of ExpressionResult and allocate one
for each statement. This ensures that we always have an outer register
allocation scope for statement code (used in CountOperation and
RegisterExecutionResult). Also refactored the register allocator code to
move it to it's own file and rename from TemporaryRegisterScope to
BytecodeRegisterAllocator.

BUG=v8:4280
LOG=N
==========

to

==========
[Interpreter] Ensure we always have an outer register allocation scope.

Split RegisterAllocationScope out of ExpressionResult and allocate one
for each statement. This ensures that we always have an outer register
allocation scope for statement code (used in CountOperation and
RegisterExecutionResult). Also refactored the register allocator code to
move it to it's own file and rename from TemporaryRegisterScope to
BytecodeRegisterAllocator.

BUG=v8:4280
LOG=N

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

[commit-bot: I haz the power, 2016-01-14 13:43:50]

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