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

Unified Diff: src/objects.h

Issue 1095433002: Refactor compilation dependency handling. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 5 years, 8 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
« src/dependencies.cc ('K') | « src/lithium.cc ('k') | src/objects.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/objects.h
diff --git a/src/objects.h b/src/objects.h
index f9ed74a480a42e74ab7b789d5e34aaa6c98b6845..d71c42680ffff7d18ac634d50cb17abdc880effd 100644
--- a/src/objects.h
+++ b/src/objects.h
@@ -5694,8 +5694,6 @@ class Code: public HeapObject {
};
-class CompilationInfo;
-
// This class describes the layout of dependent codes array of a map. The
// array is partitioned into several groups of dependent codes. Each group
// contains codes with the same dependency on the map. The array has the
@@ -5765,7 +5763,7 @@ class DependentCode: public FixedArray {
bool Contains(DependencyGroup group, WeakCell* code_cell);
- static Handle<DependentCode> InsertCompilationInfo(
+ static Handle<DependentCode> InsertCompilationDependencies(
Handle<DependentCode> entries, DependencyGroup group,
Handle<Foreign> info);
@@ -5776,8 +5774,8 @@ class DependentCode: public FixedArray {
void UpdateToFinishedCode(DependencyGroup group, Foreign* info,
WeakCell* code_cell);
- void RemoveCompilationInfo(DependentCode::DependencyGroup group,
- Foreign* info);
+ void RemoveCompilationDependencies(DependentCode::DependencyGroup group,
+ Foreign* info);
void DeoptimizeDependentCodeGroup(Isolate* isolate,
DependentCode::DependencyGroup group);
@@ -6346,10 +6344,6 @@ class Map: public HeapObject {
inline bool CanOmitMapChecks();
- static void AddDependentCompilationInfo(Handle<Map> map,
- DependentCode::DependencyGroup group,
- CompilationInfo* info);
-
static void AddDependentCode(Handle<Map> map,
DependentCode::DependencyGroup group,
Handle<Code> code);
@@ -8552,12 +8546,6 @@ class AllocationSite: public Struct {
static void DigestTransitionFeedback(Handle<AllocationSite> site,
ElementsKind to_kind);
- static void RegisterForDeoptOnTenureChange(Handle<AllocationSite> site,
- CompilationInfo* info);
-
- static void RegisterForDeoptOnTransitionChange(Handle<AllocationSite> site,
- CompilationInfo* info);
-
DECLARE_PRINTER(AllocationSite)
DECLARE_VERIFIER(AllocationSite)
@@ -8588,10 +8576,6 @@ class AllocationSite: public Struct {
kSize> BodyDescriptor;
private:
- static void AddDependentCompilationInfo(Handle<AllocationSite> site,
- DependentCode::DependencyGroup group,
- CompilationInfo* info);
-
bool PretenuringDecisionMade() {
return pretenure_decision() != kUndecided;
}
@@ -9824,9 +9808,6 @@ class PropertyCell : public HeapObject {
static Handle<PropertyCell> InvalidateEntry(Handle<NameDictionary> dictionary,
int entry);
- static void AddDependentCompilationInfo(Handle<PropertyCell> cell,
- CompilationInfo* info);
-
DECLARE_CAST(PropertyCell)
// Dispatched behavior.
« src/dependencies.cc ('K') | « src/lithium.cc ('k') | src/objects.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698