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

Issue 133363005: MIPS: Fix test-assembler-mips/MIPS10 test. (Closed)

Created:
6 years, 10 months ago by kilvadyb
Modified:
6 years, 10 months ago
CC:
v8-dev
Base URL:
https://github.com/v8/v8.git@gbl
Visibility:
Public.

Description

MIPS: Fix test-assembler-mips/MIPS10 test. cvt_l_d() and cvt_d_l() instructions are replaced to cvt_w_d() and ctv_d_w() because kernel FPU emulator generates illegal instruction for double <-> long conversions. BUG=

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+14 lines, -22 lines) Patch
M test/cctest/test-assembler-mips.cc View 3 chunks +14 lines, -22 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Paul Lind
LGTM. Jakob, can you PTAL? (I can't approve this MIPS-specific cctest change.)
6 years, 10 months ago (2014-02-03 18:58:40 UTC) #1
Jakob Kummerow
lgtm
6 years, 10 months ago (2014-02-04 07:51:52 UTC) #2
palfia
6 years, 10 months ago (2014-02-04 14:06:11 UTC) #3
Committed as r19073.

Powered by Google App Engine
This is Rietveld 408576698