x87: fix the use of CheckFloatEq and CheckDoubleEq in test.

test/cctest/compiler/value-helper.h

 // Copyright 2014 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_CCTEST_COMPILER_VALUE_HELPER_H_
 #define V8_CCTEST_COMPILER_VALUE_HELPER_H_
 
 #include <stdint.h>
 
 #include "src/compiler/common-operator.h"
@@ skipping 303 matching lines @@
 
 // TODO(bmeurer): Drop this crap once we switch to GTest/Gmock.
 static inline void CheckFloatEq(volatile float x, volatile float y) {
   if (std::isnan(x)) {
     CHECK(std::isnan(y));
   } else {
     CHECK_EQ(x, y);
   }
 }
 
+#define CHECK_FLOAT_EQ(lhs, rhs) \

[titzer, 2016/03/07 10:38:11]

You should use a do { .. } while(false) to be a good macro citizen.

+  volatile float tmp = lhs;      \
+  CheckFloatEq(tmp, rhs);
+
 static inline void CheckDoubleEq(volatile double x, volatile double y) {
   if (std::isnan(x)) {
     CHECK(std::isnan(y));
   } else {
     CHECK_EQ(x, y);
   }
 }
 
+#define CHECK_DOUBLE_EQ(lhs, rhs) \
+  volatile double tmp = lhs;      \
+  CheckDoubleEq(tmp, rhs);
+
 }  // namespace compiler
 }  // namespace internal
 }  // namespace v8
 
 #endif  // V8_CCTEST_COMPILER_VALUE_HELPER_H_