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

Issue 769193003: [turbofan] Cache the JSStoreProperty operator(s). (Closed)

Created:
6 years ago by Benedikt Meurer
Modified:
6 years ago
Reviewers:
dcarney
CC:
v8-dev
Base URL:
https://chromium.googlesource.com/v8/v8.git@master
Project:
v8
Visibility:
Public.

Description

[turbofan] Cache the JSStoreProperty operator(s). TEST=unittests R=dcarney@chromium.org Committed: https://chromium.googlesource.com/v8/v8/+/a94a612220bcd68e22c3bcae107805487082bbb3

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+86 lines, -5 lines) Patch
M src/compiler/js-operator.cc View 2 chunks +18 lines, -5 lines 0 comments Download
M test/unittests/compiler/js-operator-unittest.cc View 2 chunks +68 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
Benedikt Meurer
6 years ago (2014-12-02 11:39:04 UTC) #1
Benedikt Meurer
PTAL
6 years ago (2014-12-02 11:39:16 UTC) #2
dcarney
lgtm
6 years ago (2014-12-02 11:40:03 UTC) #3
Benedikt Meurer
6 years ago (2014-12-02 11:40:37 UTC) #4
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
a94a612220bcd68e22c3bcae107805487082bbb3 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698