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

Unified Diff: src/heap/scavenger.h

Issue 2005173003: Immediatelly promote marked objects (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Remove FORCE_PROMOTION Created 4 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
Index: src/heap/scavenger.h
diff --git a/src/heap/scavenger.h b/src/heap/scavenger.h
index 3db2e5d470f797a759bec2ccca7a40868f4a119c..bde03f827bd4e8e52c6f7d143a7e8bb0dabe666e 100644
--- a/src/heap/scavenger.h
+++ b/src/heap/scavenger.h
@@ -12,8 +12,7 @@ namespace v8 {
namespace internal {
typedef void (*ScavengingCallback)(Map* map, HeapObject** slot,
- HeapObject* object,
- PromotionMode promotion_mode);
+ HeapObject* object);
class Scavenger {
public:
@@ -26,15 +25,15 @@ class Scavenger {
// necessary, the object might be promoted to an old space. The caller must
// ensure the precondition that the object is (a) a heap object and (b) in
// the heap's from space.
- static inline void ScavengeObject(
- HeapObject** p, HeapObject* object,
- PromotionMode promotion_mode = DEFAULT_PROMOTION);
- static inline SlotCallbackResult CheckAndScavengeObject(
- Heap* heap, Address slot_address, PromotionMode promotion_mode);
+ template <PromotionMode promotion_mode>
+ static inline void ScavengeObject(HeapObject** p, HeapObject* object);
+ template <PromotionMode promotion_mode>
+ static inline SlotCallbackResult CheckAndScavengeObject(Heap* heap,
+ Address slot_address);
// Slow part of {ScavengeObject} above.
- static void ScavengeObjectSlow(HeapObject** p, HeapObject* object,
- PromotionMode promotion_mode);
+ template <PromotionMode promotion_mode>
+ static void ScavengeObjectSlow(HeapObject** p, HeapObject* object);
// Chooses an appropriate static visitor table depending on the current state
// of the heap (i.e. incremental marking, logging and profiling).
@@ -51,6 +50,7 @@ class Scavenger {
// Helper class for turning the scavenger into an object visitor that is also
// filtering out non-HeapObjects and objects which do not reside in new space.
+template <PromotionMode promotion_mode>
class ScavengeVisitor : public ObjectVisitor {
public:
explicit ScavengeVisitor(Heap* heap) : heap_(heap) {}
@@ -67,8 +67,9 @@ class ScavengeVisitor : public ObjectVisitor {
// Helper class for turning the scavenger into an object visitor that is also
// filtering out non-HeapObjects and objects which do not reside in new space.
+template <PromotionMode promotion_mode>
class StaticScavengeVisitor
- : public StaticNewSpaceVisitor<StaticScavengeVisitor> {
+ : public StaticNewSpaceVisitor<StaticScavengeVisitor<promotion_mode>> {
public:
static inline void VisitPointer(Heap* heap, HeapObject* object, Object** p);
};

Powered by Google App Engine
This is Rietveld 408576698