Change Bool::Get to return a handle instead of a pointer to a raw object.

runtime/vm/intermediate_language_arm.cc

 // Copyright (c) 2013, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #include "vm/globals.h"  // Needed here to get TARGET_ARCH_ARM.
 #if defined(TARGET_ARCH_ARM)
 
 #include "vm/intermediate_language.h"
 
 #include "vm/dart_entry.h"
@@ skipping 385 matching lines @@
                                  equality_ic_data);
   Label check_ne;
   __ b(&check_ne);
 
   __ Bind(&check_identity);
   Label equality_done;
   if (compiler->is_optimizing()) {
     // No need to update IC data.
     __ PopList((1 << R0) | (1 << R1));
     __ cmp(R0, ShifterOperand(R1));
-    __ LoadObject(R0, (kind == Token::kEQ) ? Bool::False() : Bool::True(), NE);
-    __ LoadObject(R0, (kind == Token::kEQ) ? Bool::True() : Bool::False(), EQ);
+    __ LoadObject(R0, Bool::Get(kind != Token::kEQ), NE);
+    __ LoadObject(R0, Bool::Get(kind == Token::kEQ), EQ);
     if (kind == Token::kNE) {
       // Skip not-equal result conversion.
       __ b(&equality_done);
     }
   } else {
     // Call stub, load IC data in register. The stub will update ICData if
     // necessary.
     Register ic_data_reg = locs->temp(0).reg();
     ASSERT(ic_data_reg == R5);  // Stub depends on it.
     __ LoadObject(ic_data_reg, equality_ic_data);
@@ skipping 163 matching lines @@
     if (i == (len - 1)) {
       __ b(deopt, NE);
     } else {
       __ b(&identity_compare, EQ);
     }
   }
   __ Bind(&identity_compare);
   __ cmp(left, ShifterOperand(right));
   if (branch == NULL) {
     Register result = locs.out().reg();
-    __ LoadObject(result,
-                  (kind == Token::kEQ) ? Bool::True() : Bool::False(), EQ);
-    __ LoadObject(result,
-                  (kind == Token::kEQ) ? Bool::False() : Bool::True(), NE);
+    __ LoadObject(result, Bool::Get(kind == Token::kEQ), EQ);
+    __ LoadObject(result, Bool::Get(kind != Token::kEQ), NE);
   } else {
     Condition cond = TokenKindToSmiCondition(kind);
     branch->EmitBranchOnCondition(compiler, cond);
   }
 }
 
 
 // First test if receiver is NULL, in which case === is applied.
 // If type feedback was provided (lists of <class-id, target>), do a
 // type by type check (either === or static call to the operator.
@@ skipping 3923 matching lines @@
 // Special code for numbers (compare values instead of references.)
 void StrictCompareInstr::EmitNativeCode(FlowGraphCompiler* compiler) {
   ASSERT(kind() == Token::kEQ_STRICT || kind() == Token::kNE_STRICT);
   Location left = locs()->in(0);
   Location right = locs()->in(1);
   if (left.IsConstant() && right.IsConstant()) {
     // TODO(vegorov): should be eliminated earlier by constant propagation.
     const bool result = (kind() == Token::kEQ_STRICT) ?
         left.constant().raw() == right.constant().raw() :
         left.constant().raw() != right.constant().raw();
-    __ LoadObject(locs()->out().reg(), result ? Bool::True() : Bool::False());
+    __ LoadObject(locs()->out().reg(), Bool::Get(result));
     return;
   }
   if (left.IsConstant()) {
     compiler->EmitEqualityRegConstCompare(right.reg(),
                                           left.constant(),
                                           needs_number_check(),
                                           token_pos());
   } else if (right.IsConstant()) {
     compiler->EmitEqualityRegConstCompare(left.reg(),
                                           right.constant(),
@@ skipping 121 matching lines @@
   compiler->GenerateCall(token_pos(),
                          &label,
                          PcDescriptors::kOther,
                          locs());
   __ Drop(2);  // Discard type arguments and receiver.
 }
 
 }  // namespace dart
 
 #endif  // defined TARGET_ARCH_ARM