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

Issue 7042031: MIPS: Minor fixes to simulator and builtins-mips. (Closed)

Created:
9 years, 7 months ago by Paul Lind
Modified:
9 years, 7 months ago
CC:
v8-dev
Visibility:
Public.

Description

MIPS: Minor fixes to simulator and builtins-mips. Updated to include fixes to several mips arch-specific files, corresponding to recent changes in r7944, r7935, r7926, r7914, r7910, r7895, and parts of r7423, which had previously been missed for mips. Rebased on r7964. The simulator changes were missed on r7893 for code-stubs-mips, where the DirectCEntry stuff was added. There are also a couple small changes to builtins-mips following r7879 for the other architectures. BUG= TEST= Committed: http://code.google.com/p/v8/source/detail?r=7977

Patch Set 1 #

Patch Set 2 : Rebased on r7964, updated for recent commits. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+447 lines, -235 lines) Patch
M src/mips/builtins-mips.cc View 3 chunks +10 lines, -16 lines 0 comments Download
M src/mips/code-stubs-mips.cc View 1 9 chunks +53 lines, -42 lines 0 comments Download
M src/mips/full-codegen-mips.cc View 1 2 chunks +0 lines, -15 lines 0 comments Download
M src/mips/ic-mips.cc View 1 5 chunks +33 lines, -11 lines 0 comments Download
M src/mips/macro-assembler-mips.h View 1 2 chunks +12 lines, -2 lines 0 comments Download
M src/mips/macro-assembler-mips.cc View 1 4 chunks +21 lines, -6 lines 0 comments Download
M src/mips/simulator-mips.h View 3 chunks +4 lines, -3 lines 0 comments Download
M src/mips/simulator-mips.cc View 2 chunks +23 lines, -7 lines 0 comments Download
M src/mips/stub-cache-mips.cc View 1 18 chunks +291 lines, -133 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Paul Lind
9 years, 7 months ago (2011-05-19 05:56:39 UTC) #1
Paul Lind
9 years, 7 months ago (2011-05-20 03:50:44 UTC) #2
Søren Thygesen Gjesse
9 years, 7 months ago (2011-05-23 07:12:42 UTC) #3
LGTM

I will commit it.

Powered by Google App Engine
This is Rietveld 408576698