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

Issue 1311413007: X87: Reland Vector ICs: platform support for vector-based stores. (Closed)

Created:
5 years, 3 months ago by chunyang.dai
Modified:
5 years, 3 months ago
Reviewers:
Weiliang
CC:
v8-dev
Base URL:
https://chromium.googlesource.com/v8/v8.git@master
Target Ref:
refs/pending/heads/master
Project:
v8
Visibility:
Public.

Description

X87: Reland Vector ICs: platform support for vector-based stores. port 40fbed0609ddb3e4ee4338049383004b62d13853 (r30581) original commit message: The last changes for vector store functionality, they are in 3 areas: 1) The new vector [keyed] store code stubs - implementation. 2) IC and handler compiler adjustments 3) Odds and ends. A change in ast.cc, a test update, a small Oracle fix. BUG= Committed: https://crrev.com/6b69d5365d5bf6ebca97bd07bf4d373c23fcd8bc Cr-Commit-Position: refs/heads/master@{#30612}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+387 lines, -38 lines) Patch
M src/ic/x87/access-compiler-x87.cc View 1 chunk +2 lines, -1 line 0 comments Download
M src/ic/x87/handler-compiler-x87.cc View 5 chunks +31 lines, -12 lines 0 comments Download
M src/ic/x87/ic-compiler-x87.cc View 1 chunk +4 lines, -1 line 0 comments Download
M src/ic/x87/ic-x87.cc View 3 chunks +21 lines, -5 lines 0 comments Download
M src/ic/x87/stub-cache-x87.cc View 3 chunks +29 lines, -17 lines 0 comments Download
M src/x87/code-stubs-x87.cc View 2 chunks +300 lines, -2 lines 0 comments Download

Messages

Total messages: 7 (2 generated)
chunyang.dai
PTAL
5 years, 3 months ago (2015-09-06 10:22:44 UTC) #2
Weiliang
lgtm
5 years, 3 months ago (2015-09-07 07:19:30 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1311413007/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1311413007/1
5 years, 3 months ago (2015-09-07 07:19:38 UTC) #5
commit-bot: I haz the power
Committed patchset #1 (id:1)
5 years, 3 months ago (2015-09-07 08:19:45 UTC) #6
commit-bot: I haz the power
5 years, 3 months ago (2015-09-07 08:19:56 UTC) #7
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/6b69d5365d5bf6ebca97bd07bf4d373c23fcd8bc
Cr-Commit-Position: refs/heads/master@{#30612}

Powered by Google App Engine
This is Rietveld 408576698