Fixes bad C++ code generation by XCode 7

runtime/vm/object.cc

 // Copyright (c) 2012, 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/object.h"
 
 #include "include/dart_api.h"
 #include "platform/assert.h"
 #include "vm/assembler.h"
 #include "vm/cpu.h"
@@ skipping 17280 matching lines @@
     const int64_t right_value = other.AsInt64Value();
     switch (operation) {
       case Token::kADD: {
         if (((left_value < 0) != (right_value < 0)) ||
             ((left_value + right_value) < 0) == (left_value < 0)) {
           return Integer::New(left_value + right_value, space);
         }
         break;
       }
       case Token::kSUB: {
-        if (((left_value < 0) == (right_value < 0)) ||
-            ((left_value - right_value) < 0) == (left_value < 0)) {
+        // TODO(srdjan): Investigate why XCode 7 produces wrong code
+        // if the comparison is inlined as in above (Token::kADD).
+        const bool both_same_sign = (left_value < 0) == (right_value < 0);
+        const bool result_same_sign_as_left =
+            ((left_value - right_value) < 0) == (left_value < 0);
+        if (both_same_sign || result_same_sign_as_left) {
           return Integer::New(left_value - right_value, space);
         }
         break;
       }
       case Token::kMUL: {
         if ((Utils::HighestBit(left_value) +
              Utils::HighestBit(right_value)) < 62) {
           return Integer::New(left_value * right_value, space);
         }
         break;
@@ skipping 4600 matching lines @@
   return tag_label.ToCString();
 }
 
 
 void UserTag::PrintJSONImpl(JSONStream* stream, bool ref) const {
   Instance::PrintJSONImpl(stream, ref);
 }
 
 
 }  // namespace dart