[Interpreter] Reduce temporary register usage in generated bytecode.

test/unittests/interpreter/bytecode-array-builder-unittest.cc

 // Copyright 2014 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "src/v8.h"
 
 #include "src/interpreter/bytecode-array-builder.h"
 #include "src/interpreter/bytecode-array-iterator.h"
 #include "test/unittests/test-utils.h"
 
@@ skipping 160 matching lines @@
 
 
 TEST_F(BytecodeArrayBuilderTest, FrameSizesLookGood) {
   for (int locals = 0; locals < 5; locals++) {
     for (int contexts = 0; contexts < 4; contexts++) {
       for (int temps = 0; temps < 3; temps++) {
         BytecodeArrayBuilder builder(isolate(), zone());
         builder.set_parameter_count(0);
         builder.set_locals_count(locals);
         builder.set_context_count(contexts);
-        builder.Return();
 
         TemporaryRegisterScope temporaries(&builder);
         for (int i = 0; i < temps; i++) {
-          temporaries.NewRegister();
+          builder.StoreAccumulatorInRegister(temporaries.NewRegister());
         }
+        builder.Return();
 
         Handle<BytecodeArray> the_array = builder.ToBytecodeArray();
         int total_registers = locals + contexts + temps;
         CHECK_EQ(the_array->frame_size(), total_registers * kPointerSize);
       }
     }
   }
 }
 
 
@@ skipping 43 matching lines @@
   builder.set_parameter_count(10);
   builder.set_locals_count(0);
   builder.set_context_count(0);
 
   Register param0(builder.Parameter(0));
   Register param9(builder.Parameter(9));
   CHECK_EQ(param9.index() - param0.index(), 9);
 }
 
 
+TEST_F(BytecodeArrayBuilderTest, RegisterType) {
+  BytecodeArrayBuilder builder(isolate(), zone());
+  builder.set_parameter_count(10);
+  builder.set_locals_count(3);
+  builder.set_context_count(0);
+
+  TemporaryRegisterScope temporary_register_scope(&builder);
+  Register temp0 = temporary_register_scope.NewRegister();
+  Register param0(builder.Parameter(0));
+  Register param9(builder.Parameter(9));
+  Register temp1 = temporary_register_scope.NewRegister();
+  Register reg0(0);
+  Register reg1(1);
+  Register reg2(2);
+  Register temp2 = temporary_register_scope.NewRegister();
+  CHECK_EQ(builder.RegisterIsParameterOrLocal(temp0), false);
+  CHECK_EQ(builder.RegisterIsParameterOrLocal(temp1), false);
+  CHECK_EQ(builder.RegisterIsParameterOrLocal(temp2), false);
+  CHECK_EQ(builder.RegisterIsParameterOrLocal(param0), true);
+  CHECK_EQ(builder.RegisterIsParameterOrLocal(param9), true);
+  CHECK_EQ(builder.RegisterIsParameterOrLocal(reg0), true);
+  CHECK_EQ(builder.RegisterIsParameterOrLocal(reg1), true);
+  CHECK_EQ(builder.RegisterIsParameterOrLocal(reg2), true);
+}
+
+
 TEST_F(BytecodeArrayBuilderTest, Constants) {
   BytecodeArrayBuilder builder(isolate(), zone());
   builder.set_parameter_count(0);
   builder.set_locals_count(0);
   builder.set_context_count(0);
 
   Factory* factory = isolate()->factory();
   Handle<HeapObject> heap_num_1 = factory->NewHeapNumber(3.14);
   Handle<HeapObject> heap_num_2 = factory->NewHeapNumber(5.2);
   Handle<Object> large_smi(Smi::FromInt(0x12345678), isolate());
@@ skipping 241 matching lines @@
 
   CHECK_EQ(iterator.current_bytecode(), Bytecode::kReturn);
   iterator.Advance();
   CHECK(iterator.done());
 }
 
 
 }  // namespace interpreter
 }  // namespace internal
 }  // namespace v8