Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(168)

Issue 700053003: Set the FPU precision control to double-precision. (Closed)

Created:
6 years, 1 month ago by chunyang.dai
Modified:
6 years, 1 month ago
Reviewers:
Weiliang
CC:
v8-dev
Base URL:
https://chromium.googlesource.com/external/v8.git@bleeding_edge
Project:
v8
Visibility:
Public.

Description

Set the FPU precision control to double-precision. The default double precision control of FPU is extended double-precision. While the number definition for JavaScript is double-precision. We use the FPU control word to set the doulbe precision and replace the original solution which store the data to memory and load it again. This patch also fixes the error that Sunspider 1.0.2 can not run with V8 if sse2 support is disabled. BUG=

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+11 lines, -6 lines) Patch
M src/x87/lithium-codegen-x87.cc View 2 chunks +3 lines, -6 lines 0 comments Download
M src/x87/macro-assembler-x87.h View 1 chunk +1 line, -0 lines 0 comments Download
M src/x87/macro-assembler-x87.cc View 1 chunk +7 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (1 generated)
chunyang.dai
wl. PTAL. thanks.
6 years, 1 month ago (2014-11-05 03:40:40 UTC) #2
Weiliang
lgtm
6 years, 1 month ago (2014-11-05 06:19:53 UTC) #3
Weiliang
6 years, 1 month ago (2014-11-05 06:22:16 UTC) #4
Committed as r25125

Powered by Google App Engine
This is Rietveld 408576698