Set x86 FPU precision to 64-bit for MinGW. Original patch by Jonathan Liu <net147...

BUG=v8:1062
TEST=test-strtod.cc

Review URL: https://chromiumcodereview.appspot.com/9599006
Patch from Jonathan Liu <net147@gmail.com>.

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10913 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

diff --git a/src/platform-win32.cc b/src/platform-win32.cc
index 115a6c946efcb088d245a3a76214dcd833e761f2..600320e9de85f4df0d8ece5865edf770018fb4b4 100644 (file)
--- a/src/platform-win32.cc
+++ b/src/platform-win32.cc
@@ -539,6 +539,10 @@ char* Time::LocalTimezone() {
 
 
 void OS::SetUp() {
+#ifdef __MINGW32__
+  // Set x87 FPU precision to 64-bit to be consistent with MSVC.
+  _controlfp(_PC_53, _MCW_PC);
+#endif
   // Seed the random number generator.
   // Convert the current time to a 64-bit integer first, before converting it
   // to an unsigned. Going directly can cause an overflow and the seed to be

diff --git a/src/win32-headers.h b/src/win32-headers.h
index 87f078dc0ca4f45e99db0a11a4aa143c81a5e029..f0b2d0a0c5dc6d89eb345ba4be059e8c95751f1e 100644 (file)
--- a/src/win32-headers.h
+++ b/src/win32-headers.h
@@ -63,6 +63,7 @@
 // header files to expose getaddrinfo.
 #undef _WIN32_WINNT
 #define _WIN32_WINNT 0x501
+#include <float.h>  // For _controlfp().
 #endif  // __MINGW32__
 #if !defined(__MINGW32__) || defined(__MINGW64_VERSION_MAJOR)
 #include <dbghelp.h>  // For SymLoadModule64 and al.
@@ -78,7 +79,7 @@
 #ifndef __MINGW32__
 #include <wspiapi.h>
 #endif  // __MINGW32__
-#include <process.h>  // for _beginthreadex()
+#include <process.h>  // For _beginthreadex().
 #include <stdlib.h>
 #endif  // V8_WIN32_HEADERS_FULL
 

diff --git a/test/cctest/test-strtod.cc b/test/cctest/test-strtod.cc
index da6b07bb8b96659649939195d4918d96f879db09..a94d6d51cd55fcbe69e3d57c19a737ac6526bec2 100644 (file)
--- a/test/cctest/test-strtod.cc
+++ b/test/cctest/test-strtod.cc
@@ -1,4 +1,29 @@
-// Copyright 2006-2008 the V8 project authors. All rights reserved.
+// Copyright 2012 the V8 project authors. All rights reserved.
+// Redistribution and use in source and binary forms, with or without
+// modification, are permitted provided that the following conditions are
+// met:
+//
+//     * Redistributions of source code must retain the above copyright
+//       notice, this list of conditions and the following disclaimer.
+//     * Redistributions in binary form must reproduce the above
+//       copyright notice, this list of conditions and the following
+//       disclaimer in the documentation and/or other materials provided
+//       with the distribution.
+//     * Neither the name of Google Inc. nor the names of its
+//       contributors may be used to endorse or promote products derived
+//       from this software without specific prior written permission.
+//
+// THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+// "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+// LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+// A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
+// OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+// SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
+// LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+// DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+// THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+// (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+// OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 
 #include <stdlib.h>
 
@@ -23,6 +48,8 @@ static double StrtodChar(const char* str, int exponent) {
 
 
 TEST(Strtod) {
+  v8::V8::Initialize();
+
   Vector<const char> vector;
 
   vector = StringToVector("0");