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

Side by Side Diff: src/ast.cc

Issue 641373002: Introduce FeedbackVectorSlot type - better than int. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Ports. Created 6 years, 2 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « src/ast.h ('k') | src/compiler/ast-graph-builder.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2012 the V8 project authors. All rights reserved. 1 // Copyright 2012 the V8 project authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "src/ast.h" 5 #include "src/ast.h"
6 6
7 #include <cmath> // For isfinite. 7 #include <cmath> // For isfinite.
8 #include "src/builtins.h" 8 #include "src/builtins.h"
9 #include "src/code-stubs.h" 9 #include "src/code-stubs.h"
10 #include "src/contexts.h" 10 #include "src/contexts.h"
(...skipping 46 matching lines...) Expand 10 before | Expand all | Expand 10 after
57 return var != NULL && var->location() == Variable::UNALLOCATED && 57 return var != NULL && var->location() == Variable::UNALLOCATED &&
58 var_proxy->raw_name()->IsOneByteEqualTo("undefined"); 58 var_proxy->raw_name()->IsOneByteEqualTo("undefined");
59 } 59 }
60 60
61 61
62 VariableProxy::VariableProxy(Zone* zone, Variable* var, int position, 62 VariableProxy::VariableProxy(Zone* zone, Variable* var, int position,
63 IdGen* id_gen) 63 IdGen* id_gen)
64 : Expression(zone, position, id_gen), 64 : Expression(zone, position, id_gen),
65 raw_name_(var->raw_name()), 65 raw_name_(var->raw_name()),
66 interface_(var->interface()), 66 interface_(var->interface()),
67 variable_feedback_slot_(kInvalidFeedbackSlot), 67 variable_feedback_slot_(FeedbackVectorSlot::Invalid()),
68 is_this_(var->is_this()), 68 is_this_(var->is_this()),
69 is_assigned_(false), 69 is_assigned_(false),
70 is_resolved_(false) { 70 is_resolved_(false) {
71 BindTo(var); 71 BindTo(var);
72 } 72 }
73 73
74 74
75 VariableProxy::VariableProxy(Zone* zone, const AstRawString* name, bool is_this, 75 VariableProxy::VariableProxy(Zone* zone, const AstRawString* name, bool is_this,
76 Interface* interface, int position, IdGen* id_gen) 76 Interface* interface, int position, IdGen* id_gen)
77 : Expression(zone, position, id_gen), 77 : Expression(zone, position, id_gen),
78 raw_name_(name), 78 raw_name_(name),
79 interface_(interface), 79 interface_(interface),
80 variable_feedback_slot_(kInvalidFeedbackSlot), 80 variable_feedback_slot_(FeedbackVectorSlot::Invalid()),
81 is_this_(is_this), 81 is_this_(is_this),
82 is_assigned_(false), 82 is_assigned_(false),
83 is_resolved_(false) {} 83 is_resolved_(false) {}
84 84
85 85
86 void VariableProxy::BindTo(Variable* var) { 86 void VariableProxy::BindTo(Variable* var) {
87 DCHECK(!FLAG_harmony_modules || interface_->IsUnified(var->interface())); 87 DCHECK(!FLAG_harmony_modules || interface_->IsUnified(var->interface()));
88 DCHECK((is_this() && var->is_this()) || raw_name() == var->raw_name()); 88 DCHECK((is_this() && var->is_this()) || raw_name() == var->raw_name());
89 // Ideally CONST-ness should match. However, this is very hard to achieve 89 // Ideally CONST-ness should match. However, this is very hard to achieve
90 // because we don't know the exact semantics of conflicting (const and 90 // because we don't know the exact semantics of conflicting (const and
(...skipping 508 matching lines...) Expand 10 before | Expand all | Expand 10 after
599 if (!it->isolate()->heap()->InNewSpace(*candidate)) { 599 if (!it->isolate()->heap()->InNewSpace(*candidate)) {
600 target_ = candidate; 600 target_ = candidate;
601 return true; 601 return true;
602 } 602 }
603 } 603 }
604 return false; 604 return false;
605 } 605 }
606 606
607 607
608 void CallNew::RecordTypeFeedback(TypeFeedbackOracle* oracle) { 608 void CallNew::RecordTypeFeedback(TypeFeedbackOracle* oracle) {
609 int allocation_site_feedback_slot = FLAG_pretenuring_call_new 609 FeedbackVectorSlot allocation_site_feedback_slot =
610 ? AllocationSiteFeedbackSlot() 610 FLAG_pretenuring_call_new ? AllocationSiteFeedbackSlot()
611 : CallNewFeedbackSlot(); 611 : CallNewFeedbackSlot();
612 allocation_site_ = 612 allocation_site_ =
613 oracle->GetCallNewAllocationSite(allocation_site_feedback_slot); 613 oracle->GetCallNewAllocationSite(allocation_site_feedback_slot);
614 is_monomorphic_ = oracle->CallNewIsMonomorphic(CallNewFeedbackSlot()); 614 is_monomorphic_ = oracle->CallNewIsMonomorphic(CallNewFeedbackSlot());
615 if (is_monomorphic_) { 615 if (is_monomorphic_) {
616 target_ = oracle->GetCallNewTarget(CallNewFeedbackSlot()); 616 target_ = oracle->GetCallNewTarget(CallNewFeedbackSlot());
617 } 617 }
618 } 618 }
619 619
620 620
621 void ObjectLiteral::Property::RecordTypeFeedback(TypeFeedbackOracle* oracle) { 621 void ObjectLiteral::Property::RecordTypeFeedback(TypeFeedbackOracle* oracle) {
(...skipping 523 matching lines...) Expand 10 before | Expand all | Expand 10 after
1145 // static 1145 // static
1146 bool Literal::Match(void* literal1, void* literal2) { 1146 bool Literal::Match(void* literal1, void* literal2) {
1147 const AstValue* x = static_cast<Literal*>(literal1)->raw_value(); 1147 const AstValue* x = static_cast<Literal*>(literal1)->raw_value();
1148 const AstValue* y = static_cast<Literal*>(literal2)->raw_value(); 1148 const AstValue* y = static_cast<Literal*>(literal2)->raw_value();
1149 return (x->IsString() && y->IsString() && *x->AsString() == *y->AsString()) || 1149 return (x->IsString() && y->IsString() && *x->AsString() == *y->AsString()) ||
1150 (x->IsNumber() && y->IsNumber() && x->AsNumber() == y->AsNumber()); 1150 (x->IsNumber() && y->IsNumber() && x->AsNumber() == y->AsNumber());
1151 } 1151 }
1152 1152
1153 1153
1154 } } // namespace v8::internal 1154 } } // namespace v8::internal
OLDNEW
« no previous file with comments | « src/ast.h ('k') | src/compiler/ast-graph-builder.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698