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

Unified Diff: runtime/vm/pages.h

Issue 2916173002: Avoid allocating a whole new page for two one-element arrays when finalizing the VM isolate. (Closed)
Patch Set: Created 3 years, 7 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 | « runtime/vm/clustered_snapshot.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/vm/pages.h
diff --git a/runtime/vm/pages.h b/runtime/vm/pages.h
index 7699625cc978d36801011d479bbccd7d3c89504a..729e94f6a47f36744d6c13629804573d5ce2112a 100644
--- a/runtime/vm/pages.h
+++ b/runtime/vm/pages.h
@@ -320,6 +320,9 @@ class PageSpace {
void SetupImagePage(void* pointer, uword size, bool is_executable);
+ // Return any bump allocation block to the freelist.
+ void AbandonBumpAllocation();
+
private:
// Ids for time and data records in Heap::GCStats.
enum {
@@ -353,8 +356,6 @@ class PageSpace {
bool is_locked);
// Makes bump block walkable; do not call concurrently with mutator.
void MakeIterable() const;
- // Return any bump allocation block to the freelist.
- void AbandonBumpAllocation();
HeapPage* AllocatePage(HeapPage::PageType type);
void FreePage(HeapPage* page, HeapPage* previous_page);
HeapPage* AllocateLargePage(intptr_t size, HeapPage::PageType type);
« no previous file with comments | « runtime/vm/clustered_snapshot.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698