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

Unified Diff: src/heap/heap.cc

Issue 1757263003: Use a different GCCallbackFlag for GCs triggered by CollectAllAvailableGarbage (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 4 years, 10 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 | « include/v8.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/heap/heap.cc
diff --git a/src/heap/heap.cc b/src/heap/heap.cc
index ad83dfb8c7e02843df2b4851d5e14e5131e1bd2d..27a5fefd32c584e9722c759570bf0d76729ab16e 100644
--- a/src/heap/heap.cc
+++ b/src/heap/heap.cc
@@ -895,7 +895,8 @@ void Heap::CollectAllAvailableGarbage(const char* gc_reason) {
const int kMinNumberOfAttempts = 2;
for (int attempt = 0; attempt < kMaxNumberOfAttempts; attempt++) {
if (!CollectGarbage(MARK_COMPACTOR, gc_reason, NULL,
- v8::kGCCallbackFlagForced) &&
+ v8::kGCCallbackFlagForced |
+ v8::kGCCallbackFlagCollectAllAvailableGarbage) &&
haraken 2016/03/04 12:19:32 Sorry, I noticed that this needs to be 'forced | a
attempt + 1 >= kMinNumberOfAttempts) {
break;
}
« no previous file with comments | « include/v8.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698