[Interpreter] Remove all Ldr style bytecodes and replace with Star lookahead.

[Interpreter] Remove all Ldr style bytecodes and replace with Star lookahead.

We seem to get some small wins from avoiding the Ldr bytecodes, probably due
to reduced icache pressure since there are less bytecode handlers. Replace
the Ldr bytecodes with Star lookahead inlined into the Lda versions.

Also fixes IsAccumulatorLoadWithoutEffects to include LdaContextSlot and
LdaCurrentContextSlot

BUG=v8:4280
Committed: https://crrev.com/f633218b62477e1b9bc585d412297908887b75b4
Cr-Commit-Position: refs/heads/master@{#40883}

[rmcilroy, 2016-11-09 16:19:07]

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

[commit-bot: I haz the power, 2016-11-09 16:19:10]

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

[rmcilroy, 2016-11-09 16:21:03]

rmcilroy@chromium.org changed reviewers:
+ jarin@chromium.org, mythria@chromium.org

[rmcilroy, 2016-11-09 16:21:04]

Trying the same for all our other Ldr bytecodes since we actually see a small
win from removing these bytecodes when running Ignition (probably due to reduced
icache pressure due to less bytecode handlers).

Mythri / Jaro, PTAL.

[commit-bot: I haz the power, 2016-11-09 16:45:23]

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

[commit-bot: I haz the power, 2016-11-09 16:45:23]

Dry run: This issue passed the CQ dry run.

[rmcilroy, 2016-11-09 17:24:03]

Seems performance neutral, so probably worth removing for code complexity.

[Jarin, 2016-11-10 07:51:01]

lgtm, nice!

[mythria, 2016-11-10 10:33:13]

lgtm.

[rmcilroy, 2016-11-10 10:39:42]

The CQ bit was checked by rmcilroy@chromium.org

[commit-bot: I haz the power, 2016-11-10 10:39:45]

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-10 10:41:53]

Committed patchset #1 (id:1)

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

Description was changed from

==========
[Interpreter] Remove all Ldr style bytecodes and replace with Star lookahead.

We seem to get some small wins from avoiding the Ldr bytecodes, probably due
to reduced icache pressure since there are less bytecode handlers. Replace
the Ldr bytecodes with Star lookahead inlined into the Lda versions.

Also fixes IsAccumulatorLoadWithoutEffects to include LdaContextSlot and
LdaCurrentContextSlot

BUG=v8:4280
==========

to

==========
[Interpreter] Remove all Ldr style bytecodes and replace with Star lookahead.

We seem to get some small wins from avoiding the Ldr bytecodes, probably due
to reduced icache pressure since there are less bytecode handlers. Replace
the Ldr bytecodes with Star lookahead inlined into the Lda versions.

Also fixes IsAccumulatorLoadWithoutEffects to include LdaContextSlot and
LdaCurrentContextSlot

BUG=v8:4280

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

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

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