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

Unified Diff: src/heap/objects-visiting.cc

Issue 1016353002: Remove early bail-out in VisitWeakList to investigate chrasher. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 5 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/heap/objects-visiting.cc
diff --git a/src/heap/objects-visiting.cc b/src/heap/objects-visiting.cc
index 9d6d99ccae22bdaca59d02d04838498c6c4c482c..520e539c7cca5e779f71aa1096ad72e4b2fd20c7 100644
--- a/src/heap/objects-visiting.cc
+++ b/src/heap/objects-visiting.cc
@@ -202,7 +202,6 @@ Object* VisitWeakList(Heap* heap, Object* list, WeakObjectRetainer* retainer,
while (list != undefined) {
// Check whether to keep the candidate in the list.
T* candidate = reinterpret_cast<T*>(list);
- T* original_candidate = candidate;
Object* retained = retainer->RetainAs(list);
if (retained != NULL) {
@@ -227,18 +226,6 @@ Object* VisitWeakList(Heap* heap, Object* list, WeakObjectRetainer* retainer,
// tail is a live object, visit it.
WeakListVisitor<T>::VisitLiveObject(heap, tail, retainer);
- // The list of weak objects is usually order. It starts with objects
- // recently allocated in the young generation followed by objects
- // allocated in the old generation. When a migration failure happened,
- // the list is not ordered until the next GC that has no migration
- // failure.
- // For young generation collections we just have to visit until the last
- // young generation objects.
- if (stop_after_young && !heap->migration_failure() &&
- !heap->previous_migration_failure() &&
- !heap->InNewSpace(original_candidate)) {
- return head;
- }
} else {
WeakListVisitor<T>::VisitPhantomObject(heap, candidate);
}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698