Materialize booleans in the turbofan deoptimizer.

src/compiler/code-generator.cc

 // Copyright 2013 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/compiler/code-generator.h"
 
 #include "src/compiler/code-generator-impl.h"
 #include "src/compiler/linkage.h"
 #include "src/compiler/pipeline.h"
 
@@ skipping 510 matching lines @@
 
   return deoptimization_id;
 }
 
 
 void CodeGenerator::AddTranslationForOperand(Translation* translation,
                                              Instruction* instr,
                                              InstructionOperand* op,
                                              MachineType type) {
   if (op->IsStackSlot()) {
-    // TODO(jarin) kMachBool and kRepBit should materialize true and false
-    // rather than creating an int value.
-    if (type == kMachBool || type == kRepBit || type == kMachInt32 ||
-        type == kMachInt8 || type == kMachInt16) {
+    if (type == kMachBool || type == kRepBit) {
+      translation->StoreBoolStackSlot(StackSlotOperand::cast(op)->index());
+    } else if (type == kMachInt32 || type == kMachInt8 || type == kMachInt16) {
       translation->StoreInt32StackSlot(StackSlotOperand::cast(op)->index());
     } else if (type == kMachUint32 || type == kMachUint16 ||
                type == kMachUint8) {
       translation->StoreUint32StackSlot(StackSlotOperand::cast(op)->index());
     } else if ((type & kRepMask) == kRepTagged) {
       translation->StoreStackSlot(StackSlotOperand::cast(op)->index());
     } else {
       CHECK(false);
     }
   } else if (op->IsDoubleStackSlot()) {
     DCHECK((type & (kRepFloat32 | kRepFloat64)) != 0);
     translation->StoreDoubleStackSlot(
         DoubleStackSlotOperand::cast(op)->index());
   } else if (op->IsRegister()) {
     InstructionOperandConverter converter(this, instr);
-    // TODO(jarin) kMachBool and kRepBit should materialize true and false
-    // rather than creating an int value.
-    if (type == kMachBool || type == kRepBit || type == kMachInt32 ||
-        type == kMachInt8 || type == kMachInt16) {
+    if (type == kMachBool || type == kRepBit) {
+      translation->StoreBoolRegister(converter.ToRegister(op));
+    } else if (type == kMachInt32 || type == kMachInt8 || type == kMachInt16) {
       translation->StoreInt32Register(converter.ToRegister(op));
     } else if (type == kMachUint32 || type == kMachUint16 ||
                type == kMachUint8) {
       translation->StoreUint32Register(converter.ToRegister(op));
     } else if ((type & kRepMask) == kRepTagged) {
       translation->StoreRegister(converter.ToRegister(op));
     } else {
       CHECK(false);
     }
   } else if (op->IsDoubleRegister()) {
@@ skipping 96 matching lines @@
     : masm_(gen->masm()), next_(gen->ools_) {
   gen->ools_ = this;
 }
 
 
 OutOfLineCode::~OutOfLineCode() {}
 
 }  // namespace compiler
 }  // namespace internal
 }  // namespace v8