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

Unified Diff: src/ast-numbering.cc

Issue 1376443002: Refactored interface of FeedbackVectorSpec and friends. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Fix Created 5 years, 3 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 | « src/ast.cc ('k') | src/factory.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/ast-numbering.cc
diff --git a/src/ast-numbering.cc b/src/ast-numbering.cc
index a89b5fb60a298d674be4a9865570da38f3744641..cc615f6e4795643bfc00c139a46030b9c75e491a 100644
--- a/src/ast-numbering.cc
+++ b/src/ast-numbering.cc
@@ -65,21 +65,8 @@ class AstNumberingVisitor final : public AstVisitor {
template <typename Node>
void ReserveFeedbackSlots(Node* node) {
- FeedbackVectorRequirements reqs =
- node->ComputeFeedbackRequirements(isolate(), &ic_slot_cache_);
- if (reqs.slots() > 0) {
- node->SetFirstFeedbackSlot(FeedbackVectorSlot(properties_.slots()));
- properties_.increase_slots(reqs.slots());
- }
- if (reqs.ic_slots() > 0) {
- int ic_slots = properties_.ic_slots();
- node->SetFirstFeedbackICSlot(FeedbackVectorICSlot(ic_slots),
- &ic_slot_cache_);
- properties_.increase_ic_slots(reqs.ic_slots());
- for (int i = 0; i < reqs.ic_slots(); i++) {
- properties_.SetKind(ic_slots + i, node->FeedbackICSlotKind(i));
- }
- }
+ node->AssignFeedbackVectorSlots(isolate(), properties_.get_spec(),
+ &ic_slot_cache_);
}
BailoutReason dont_optimize_reason() const { return dont_optimize_reason_; }
« no previous file with comments | « src/ast.cc ('k') | src/factory.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698