[interpreter] Add a register operand to ToNumber

src/interpreter/bytecodes.h

 // 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.
 
 #ifndef V8_INTERPRETER_BYTECODES_H_
 #define V8_INTERPRETER_BYTECODES_H_
 
 #include <cstdint>
 #include <iosfwd>
 #include <string>
@@ skipping 194 matching lines @@
   V(TestEqualStrict, AccumulatorUse::kReadWrite, OperandType::kReg)            \
   V(TestLessThan, AccumulatorUse::kReadWrite, OperandType::kReg)               \
   V(TestGreaterThan, AccumulatorUse::kReadWrite, OperandType::kReg)            \
   V(TestLessThanOrEqual, AccumulatorUse::kReadWrite, OperandType::kReg)        \
   V(TestGreaterThanOrEqual, AccumulatorUse::kReadWrite, OperandType::kReg)     \
   V(TestInstanceOf, AccumulatorUse::kReadWrite, OperandType::kReg)             \
   V(TestIn, AccumulatorUse::kReadWrite, OperandType::kReg)                     \
                                                                                \
   /* Cast operators */                                                         \
   V(ToName, AccumulatorUse::kReadWrite)                                        \
-  V(ToNumber, AccumulatorUse::kReadWrite)                                      \
+  V(ToNumber, AccumulatorUse::kRead, OperandType::kRegOut)                     \

[rmcilroy, 2016/07/21 09:31:48]

You need to modify BytecodeGraphBuilder as well, otherwise optimized code will
break. 

Do no tests break with this change? If so, could you add something to
test-run-bytecode-graph-builder which would break until you update it.

[klaasb, 2016/07/21 10:55:19]

Done.
Some did actually, sorry.

   V(ToObject, AccumulatorUse::kReadWrite)                                      \
                                                                                \
   /* Literals */                                                               \
   V(CreateRegExpLiteral, AccumulatorUse::kWrite, OperandType::kIdx,            \
     OperandType::kIdx, OperandType::kFlag8)                                    \
   V(CreateArrayLiteral, AccumulatorUse::kWrite, OperandType::kIdx,             \
     OperandType::kIdx, OperandType::kFlag8)                                    \
   V(CreateObjectLiteral, AccumulatorUse::kWrite, OperandType::kIdx,            \
     OperandType::kIdx, OperandType::kFlag8)                                    \
                                                                                \
@@ skipping 337 matching lines @@
 std::ostream& operator<<(std::ostream& os, const AccumulatorUse& use);
 std::ostream& operator<<(std::ostream& os, const OperandScale& operand_scale);
 std::ostream& operator<<(std::ostream& os, const OperandSize& operand_size);
 std::ostream& operator<<(std::ostream& os, const OperandType& operand_type);
 
 }  // namespace interpreter
 }  // namespace internal
 }  // namespace v8
 
 #endif  // V8_INTERPRETER_BYTECODES_H_