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

Issue 196343003: Revert "Don't need a temp register for StoreKeyed double" (Closed)

Created:
6 years, 9 months ago by haitao.feng
Modified:
6 years, 9 months ago
Reviewers:
Weiliang, Sven Panne, danno
CC:
v8-dev
Visibility:
Public.

Description

Revert "Don't need a temp register for StoreKeyed double" This reverts commit r19829 for breaking the mozilla tests. R=svenpanne@chromium.org Committed: https://code.google.com/p/v8/source/detail?r=19830

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M src/x64/lithium-x64.cc View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 4 (0 generated)
haitao.feng
6 years, 9 months ago (2014-03-12 04:06:52 UTC) #1
haitao.feng
Weiliang changed x64 port but mozilla failed on ia32 port at http://build.chromium.org/p/client.v8/builders/V8%20Linux%20-%20deadcode/builds/2916. The mozilla ia32 ...
6 years, 9 months ago (2014-03-12 05:14:00 UTC) #2
Sven Panne
LGTM, I'll land it...
6 years, 9 months ago (2014-03-12 06:46:09 UTC) #3
Sven Panne
6 years, 9 months ago (2014-03-12 06:47:01 UTC) #4
Message was sent while issue was closed.
Committed patchset #1 manually as r19830 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698