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

Issue 2627973007: X87: [TypeFeedbackVector] Root literal arrays in function literals slots. (Closed)

Created:
3 years, 11 months ago by zhengxing.li
Modified:
3 years, 11 months ago
Reviewers:
mvstanton, chunyang.dai
CC:
v8-reviews_googlegroups.com
Target Ref:
refs/pending/heads/master
Project:
v8
Visibility:
Public.

Description

X87: [TypeFeedbackVector] Root literal arrays in function literals slots. port b8294aaa978e972978b8d82cedf63befad5d7af1 (r42264) original commit message: Literal arrays and feedback vectors for a function can be garbage collected if we don't have a rooted closure for the function, which happens often. It's expensive to come back from this (recreating boilerplates and gathering feedback again), and the cost is disproportionate if the function was inlined into optimized code. To guard against losing these arrays when we need them, we'll now create literal arrays when creating the feedback vector for the outer closure, and root them strongly in that vector. BUG= Review-Url: https://codereview.chromium.org/2627973007 Cr-Commit-Position: refs/heads/master@{#42351} Committed: https://chromium.googlesource.com/v8/v8/+/f9677ecf5ff55165c8b11b34ae8d9e75ba504990

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -17 lines) Patch
M src/builtins/x87/builtins-x87.cc View 5 chunks +8 lines, -17 lines 0 comments Download

Messages

Total messages: 8 (4 generated)
zhengxing.li
PTAL, thanks!
3 years, 11 months ago (2017-01-13 08:44:06 UTC) #2
chunyang.dai
On 2017/01/13 08:44:06, zhengxing.li wrote: > PTAL, thanks! LGTM. thanks
3 years, 11 months ago (2017-01-16 02:32:40 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2627973007/1
3 years, 11 months ago (2017-01-16 02:34:11 UTC) #5
commit-bot: I haz the power
3 years, 11 months ago (2017-01-16 02:54:14 UTC) #8
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/v8/v8/+/f9677ecf5ff55165c8b11b34ae8d9e75ba5...

Powered by Google App Engine
This is Rietveld 408576698