commit 339e0c804ec60347d401a09bb62182dabb7ee54c
author oth <oth@chromium.org> Wed Oct 21 15:28:55 2015
committer Commit bot <commit-bot@chromium.org> Wed Oct 21 15:29:23 2015
tree 81856b7c9b7403e33c6b8b42ddb75d96691e78f0
parent 14ba9c3dea3ea6a597a63511fbed5190e74480bd

[Interpreter] Reduce temporary register usage in generated bytecode.

This change adds new flavors of Visit() methods for obtaining
expression results:

- VisitForAccumulatorValue() which places result in the accumulator.
- VisitForRegisterValue() which places the result in a register.
- VisitForEffect() which evaluates the expression and discards the result.

The targets of these calls place the expression result with
result_scope()->SetResultInRegister() or
result_scope()->SetResultInAccumulator().

By being smarter about result locations, there's less temporary
register usage. However, we now have a hazard with assignments
in binary expressions that didn't exist before. This change detects and
DCHECK's when a hazard is detected. A follow on CL will address this.

There are consequential changes to test-bytecode-generator.cc and
this change also adds new bytecode macros A(x, n) and THIS(n) for
register file entries for arguments and this.

BUG=v8:4280
LOG=NO

Review URL: https://codereview.chromium.org/1392933002

Cr-Commit-Position: refs/heads/master@{#31445}