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

Issue 1421903012: [heap] Separate out optimized code map processing. (Closed)

Created:
5 years, 1 month ago by Michael Starzinger
Modified:
5 years, 1 month ago
Reviewers:
ulan
CC:
v8-reviews_googlegroups.com
Base URL:
https://chromium.googlesource.com/v8/v8.git@master
Target Ref:
refs/pending/heads/master
Project:
v8
Visibility:
Public.

Description

[heap] Separate out optimized code map processing. This separates the post-processing step for optimized code maps out of the CodeFlusher. It uses the complete SharedFunctionInfo::Iterator to visit all candidates instead of gathering candidates during marking. Gathering candidates during marking no longer makes sense, now that the majority of SharedFunctionInfo objects will hold such an optimized code map. Also it reduces complexity of the implementation. Also conflating this mechanism with "code flushing" was confusing. This reverts commit b6644e849151dedffc1844d60b7a76da4d9e5115. R=ulan@chromium.org Committed: https://crrev.com/bb7a5eb2d89bae25f2b5ecb9515669f0ac73c111 Cr-Commit-Position: refs/heads/master@{#31836}

Patch Set 1 #

Patch Set 2 : Make verifier happy. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+114 lines, -198 lines) Patch
M src/heap/gc-tracer.h View 1 chunk +1 line, -0 lines 0 comments Download
M src/heap/gc-tracer.cc View 2 chunks +2 lines, -0 lines 0 comments Download
M src/heap/mark-compact.h View 5 chunks +8 lines, -18 lines 0 comments Download
M src/heap/mark-compact.cc View 5 chunks +73 lines, -129 lines 0 comments Download
M src/heap/mark-compact-inl.h View 2 chunks +0 lines, -28 lines 0 comments Download
M src/heap/objects-visiting-inl.h View 2 chunks +13 lines, -12 lines 0 comments Download
M src/objects.h View 1 chunk +2 lines, -3 lines 0 comments Download
M src/objects.cc View 1 1 chunk +15 lines, -8 lines 0 comments Download

Dependent Patchsets:

Messages

Total messages: 7 (1 generated)
Michael Starzinger
As we already suspected offline, the heap verifier got mad about stale entries in the ...
5 years, 1 month ago (2015-11-05 16:50:28 UTC) #1
ulan
lgtm
5 years, 1 month ago (2015-11-05 17:06:27 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1421903012/20001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1421903012/20001
5 years, 1 month ago (2015-11-05 17:24:34 UTC) #4
commit-bot: I haz the power
Committed patchset #2 (id:20001)
5 years, 1 month ago (2015-11-05 17:30:08 UTC) #5
commit-bot: I haz the power
Patchset 2 (id:??) landed as https://crrev.com/bb7a5eb2d89bae25f2b5ecb9515669f0ac73c111 Cr-Commit-Position: refs/heads/master@{#31836}
5 years, 1 month ago (2015-11-05 17:30:29 UTC) #6
Michael Starzinger
5 years, 1 month ago (2015-11-05 18:49:31 UTC) #7
Message was sent while issue was closed.
A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/1412063012/ by mstarzinger@chromium.org.

The reason for reverting is: Causes GC-Stress failures..

Powered by Google App Engine
This is Rietveld 408576698