[interpreter] Put object in register for ToObject/ForInPrepare

src/interpreter/interpreter.cc

 // Copyright 2015 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/interpreter/interpreter.h"
 
 #include <fstream>
 #include <memory>
 
 #include "src/ast/prettyprinter.h"
@@ skipping 1044 matching lines @@
 void Interpreter::DoToNumber(InterpreterAssembler* assembler) {
   Node* result = BuildUnaryOp(CodeFactory::ToNumber(isolate_), assembler);
   __ StoreRegister(result, __ BytecodeOperandReg(0));
   __ Dispatch();
 }
 
 // ToObject
 //
 // Cast the object referenced by the accumulator to a JSObject.
 void Interpreter::DoToObject(InterpreterAssembler* assembler) {
-  DoUnaryOp(CodeFactory::ToObject(isolate_), assembler);
+  Node* result = BuildUnaryOp(CodeFactory::ToObject(isolate_), assembler);
+  __ StoreRegister(result, __ BytecodeOperandReg(0));
+  __ Dispatch();
 }
 
 // Inc
 //
 // Increments value in the accumulator by one.
 void Interpreter::DoInc(InterpreterAssembler* assembler) {
   DoUnaryOp<IncStub>(assembler);
 }
 
 // Dec
@@ skipping 829 matching lines @@
   __ StoreRegister(cache_length, output_register);
 }
 
 // ForInPrepare <cache_info_triple>
 //
 // Returns state for for..in loop execution based on the object in the
 // accumulator. The result is output in registers |cache_info_triple| to
 // |cache_info_triple + 2|, with the registers holding cache_type, cache_array,
 // and cache_length respectively.
 void Interpreter::DoForInPrepare(InterpreterAssembler* assembler) {
-  Node* object = __ GetAccumulator();
+  Node* object_reg = __ BytecodeOperandReg(0);
+  Node* object = __ LoadRegister(object_reg);
   Node* context = __ GetContext();
   Node* const zero_smi = __ SmiConstant(Smi::FromInt(0));
 
   Label test_if_null(assembler), test_if_undefined(assembler),
       nothing_to_iterate(assembler, Label::kDeferred),
       convert_to_receiver(assembler, Label::kDeferred),
       already_receiver(assembler), check_enum_cache(assembler);
 
   Variable receiver(assembler, MachineRepresentation::kTagged);
 
@@ skipping 40 matching lines @@
     // iterated over and use the cache for the iteration.
     Node* cache_type = __ LoadMap(receiver.value());
     Node* cache_length = __ EnumLength(cache_type);
     __ GotoIf(assembler->WordEqual(cache_length, zero_smi),
               &nothing_to_iterate);
     Node* descriptors = __ LoadMapDescriptors(cache_type);
     Node* cache_offset =
         __ LoadObjectField(descriptors, DescriptorArray::kEnumCacheOffset);
     Node* cache_array = __ LoadObjectField(
         cache_offset, DescriptorArray::kEnumCacheBridgeCacheOffset);
-    Node* output_register = __ BytecodeOperandReg(0);
+    Node* output_register = __ BytecodeOperandReg(1);
     BuildForInPrepareResult(output_register, cache_type, cache_array,
                             cache_length, assembler);
     __ Dispatch();
   }
 
   __ Bind(&use_runtime);
   {
     Node* result_triple =
         __ CallRuntime(Runtime::kForInPrepare, context, object);
     Node* cache_type = __ Projection(0, result_triple);
     Node* cache_array = __ Projection(1, result_triple);
     Node* cache_length = __ Projection(2, result_triple);
-    Node* output_register = __ BytecodeOperandReg(0);
+    Node* output_register = __ BytecodeOperandReg(1);
     BuildForInPrepareResult(output_register, cache_type, cache_array,
                             cache_length, assembler);
     __ Dispatch();
   }
 
   __ Bind(&nothing_to_iterate);
   {
     // Receiver is null or undefined or descriptors are zero length.
-    Node* output_register = __ BytecodeOperandReg(0);
+    Node* output_register = __ BytecodeOperandReg(1);
     BuildForInPrepareResult(output_register, zero_smi, zero_smi, zero_smi,
                             assembler);
     __ Dispatch();
   }
 }
 
 // ForInNext <receiver> <index> <cache_info_pair>
 //
 // Returns the next enumerable property in the the accumulator.
 void Interpreter::DoForInNext(InterpreterAssembler* assembler) {
@@ skipping 165 matching lines @@
   __ StoreObjectField(generator, JSGeneratorObject::kContinuationOffset,
       __ SmiTag(new_state));
   __ SetAccumulator(old_state);
 
   __ Dispatch();
 }
 
 }  // namespace interpreter
 }  // namespace internal
 }  // namespace v8