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

Unified Diff: src/type-feedback-vector.h

Issue 1489413006: Revert of Provide call counts for constructor calls, surface them as a vector IC. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 5 years 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 | « src/objects-printer.cc ('k') | src/type-feedback-vector.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/type-feedback-vector.h
diff --git a/src/type-feedback-vector.h b/src/type-feedback-vector.h
index 7fb4cf326cbc741982fca2fc65e602322aeb13b5..05dfff7dba302b9b54279bf931f3f2ab60ba889b 100644
--- a/src/type-feedback-vector.h
+++ b/src/type-feedback-vector.h
@@ -23,7 +23,6 @@
INVALID,
CALL_IC,
- CONSTRUCT_IC,
LOAD_IC,
KEYED_LOAD_IC,
STORE_IC,
@@ -46,10 +45,6 @@
FeedbackVectorSlot AddCallICSlot() {
return AddSlot(FeedbackVectorSlotKind::CALL_IC);
- }
-
- FeedbackVectorSlot AddConstructICSlot() {
- return AddSlot(FeedbackVectorSlotKind::CONSTRUCT_IC);
}
FeedbackVectorSlot AddLoadICSlot() {
@@ -160,7 +155,7 @@
static const char* Kind2String(FeedbackVectorSlotKind kind);
private:
- static const int kFeedbackVectorSlotKindBits = 4;
+ static const int kFeedbackVectorSlotKindBits = 3;
STATIC_ASSERT(static_cast<int>(FeedbackVectorSlotKind::KINDS_NUMBER) <
(1 << kFeedbackVectorSlotKindBits));
@@ -404,7 +399,7 @@
};
-class CallICNexus : public FeedbackNexus {
+class CallICNexus final : public FeedbackNexus {
public:
// Monomorphic call ics store call counts. Platform code needs to increment
// the count appropriately (ie, by 2).
@@ -412,13 +407,11 @@
CallICNexus(Handle<TypeFeedbackVector> vector, FeedbackVectorSlot slot)
: FeedbackNexus(vector, slot) {
- DCHECK(vector->GetKind(slot) == FeedbackVectorSlotKind::CALL_IC ||
- vector->GetKind(slot) == FeedbackVectorSlotKind::CONSTRUCT_IC);
+ DCHECK_EQ(FeedbackVectorSlotKind::CALL_IC, vector->GetKind(slot));
}
CallICNexus(TypeFeedbackVector* vector, FeedbackVectorSlot slot)
: FeedbackNexus(vector, slot) {
- DCHECK(vector->GetKind(slot) == FeedbackVectorSlotKind::CALL_IC ||
- vector->GetKind(slot) == FeedbackVectorSlotKind::CONSTRUCT_IC);
+ DCHECK_EQ(FeedbackVectorSlotKind::CALL_IC, vector->GetKind(slot));
}
void Clear(Code* host);
@@ -441,19 +434,7 @@
return length == 0;
}
- Handle<Object> GetCallFeedback();
int ExtractCallCount();
-};
-
-
-class ConstructICNexus final : public CallICNexus {
- public:
- ConstructICNexus(Handle<TypeFeedbackVector> vector, FeedbackVectorSlot slot)
- : CallICNexus(vector, slot) {}
- ConstructICNexus(TypeFeedbackVector* vector, FeedbackVectorSlot slot)
- : CallICNexus(vector, slot) {}
-
- void Clear(Code* host);
};
« no previous file with comments | « src/objects-printer.cc ('k') | src/type-feedback-vector.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698