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

Unified Diff: src/heap/spaces.h

Issue 1427973006: [heap] make inline allocation step size dynamic (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 5 years, 1 month 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 | src/heap/spaces.cc » ('j') | src/heap/spaces.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/heap/spaces.h
diff --git a/src/heap/spaces.h b/src/heap/spaces.h
index 3f2d22bee06cc5d8d726a1e3b270b9ba208e5a59..19f8573e132f20c4b46a3888b6afefc8871fe803 100644
--- a/src/heap/spaces.h
+++ b/src/heap/spaces.h
@@ -2521,6 +2521,7 @@ class InlineAllocationObserver {
private:
intptr_t step_size() const { return step_size_; }
+ intptr_t bytes_to_next_step() const { return bytes_to_next_step_; }
// Pure virtual method provided by the subclasses that gets called when more
// than step_size byte have been allocated.
@@ -2561,7 +2562,6 @@ class NewSpace : public Space {
to_space_(heap, kToSpace),
from_space_(heap, kFromSpace),
reservation_(),
- inline_allocation_limit_step_(0),
top_on_previous_step_(0) {}
// Sets up the new space using the given chunk.
@@ -2747,7 +2747,6 @@ class NewSpace : public Space {
void RemoveInlineAllocationObserver(InlineAllocationObserver* observer);
void DisableInlineAllocationSteps() {
- inline_allocation_limit_step_ = 0;
top_on_previous_step_ = 0;
UpdateInlineAllocationLimit(0);
}
@@ -2849,7 +2848,6 @@ class NewSpace : public Space {
// once in a while. This is done by setting allocation_info_.limit to be lower
// than the actual limit and and increasing it in steps to guarantee that the
// observers are notified periodically.
- intptr_t inline_allocation_limit_step_;
List<InlineAllocationObserver*> inline_allocation_observers_;
Address top_on_previous_step_;
« no previous file with comments | « no previous file | src/heap/spaces.cc » ('j') | src/heap/spaces.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698