[ignition] Reuse code-stub-assembler's context load operations.

[ignition] Reuse code-stub-assembler's context load operations.

- Remove (one version of) InterpreterAssembler::LoadContextSlot in favor of the
  identical CodeStubAssembler::LoadContextElement.
- Use CodeStubAssembler::LoadNativeContext instead of doing the load manually.

R=rmcilroy@chromium.org
BUG=

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

[neis, 2016-11-04 10:56:02]

Description was changed from

==========
[ignition] Reuse code-stub-assembler's context load operations.

- Remove (one version of) InterpreterAssembler::LoadContextSlot in favor of the
  identical CodeStubAssembler::LoadContextElement.
- Use CodeStubAssembler::LoadNativeContext instead of doing the load manually.

R=rmcilroy@chromium.org
BUG=
==========

to

==========
[ignition] Reuse code-stub-assembler's context load operations.

- Remove (one version of) InterpreterAssembler::LoadContextSlot in favor of the
  identical CodeStubAssembler::LoadContextElement.
- Use CodeStubAssembler::LoadNativeContext instead of doing the load manually.

R=rmcilroy@chromium.org
BUG=
==========

[neis, 2016-11-04 10:56:02]

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

[neis, 2016-11-04 10:56:19]

+mstarzinger since ross is ooo

[rmcilroy, 2016-11-04 11:29:56]

LGTM with an optional suggestion.

https://codereview.chromium.org/2470253003/diff/1/src/interpreter/interpreter...
File src/interpreter/interpreter-assembler.h (right):

https://codereview.chromium.org/2470253003/diff/1/src/interpreter/interpreter...
src/interpreter/interpreter-assembler.h:102: compiler::Node* value);
Optional - Could we move all of these to the code-stub-assembler too?

[neis, 2016-11-04 13:00:33]

https://codereview.chromium.org/2470253003/diff/1/src/interpreter/interpreter...
File src/interpreter/interpreter-assembler.h (right):

https://codereview.chromium.org/2470253003/diff/1/src/interpreter/interpreter...
src/interpreter/interpreter-assembler.h:102: compiler::Node* value);
On 2016/11/04 11:29:56, rmcilroy (OOO - slow) wrote:
> Optional - Could we move all of these to the code-stub-assembler too?

Fine with me, I can do that in a followup.

[neis, 2016-11-04 13:00:37]

The CQ bit was checked by neis@chromium.org

[commit-bot: I haz the power, 2016-11-04 13:00:47]

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

[commit-bot: I haz the power, 2016-11-04 13:25:28]

Description was changed from

==========
[ignition] Reuse code-stub-assembler's context load operations.

- Remove (one version of) InterpreterAssembler::LoadContextSlot in favor of the
  identical CodeStubAssembler::LoadContextElement.
- Use CodeStubAssembler::LoadNativeContext instead of doing the load manually.

R=rmcilroy@chromium.org
BUG=
==========

to

==========
[ignition] Reuse code-stub-assembler's context load operations.

- Remove (one version of) InterpreterAssembler::LoadContextSlot in favor of the
  identical CodeStubAssembler::LoadContextElement.
- Use CodeStubAssembler::LoadNativeContext instead of doing the load manually.

R=rmcilroy@chromium.org
BUG=
==========

[commit-bot: I haz the power, 2016-11-04 13:25:29]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-17 22:22:50]

Description was changed from

==========
[ignition] Reuse code-stub-assembler's context load operations.

- Remove (one version of) InterpreterAssembler::LoadContextSlot in favor of the
  identical CodeStubAssembler::LoadContextElement.
- Use CodeStubAssembler::LoadNativeContext instead of doing the load manually.

R=rmcilroy@chromium.org
BUG=
==========

to

==========
[ignition] Reuse code-stub-assembler's context load operations.

- Remove (one version of) InterpreterAssembler::LoadContextSlot in favor of the
  identical CodeStubAssembler::LoadContextElement.
- Use CodeStubAssembler::LoadNativeContext instead of doing the load manually.

R=rmcilroy@chromium.org
BUG=

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

[commit-bot: I haz the power, 2016-11-17 22:22:51]

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