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

Unified Diff: runtime/vm/object.cc

Issue 10832150: Get rid of ast node ids. (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart/
Patch Set: Created 8 years, 4 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: runtime/vm/object.cc
===================================================================
--- runtime/vm/object.cc (revision 10265)
+++ runtime/vm/object.cc (working copy)
@@ -6539,23 +6539,23 @@
}
-intptr_t PcDescriptors::NodeId(intptr_t index) const {
- return Smi::Value(*SmiAddr(index, kNodeIdEntry));
+intptr_t PcDescriptors::DeoptId(intptr_t index) const {
+ return Smi::Value(*SmiAddr(index, kDeoptIdEntry));
}
-void PcDescriptors::SetNodeId(intptr_t index, intptr_t value) const {
- *SmiAddr(index, kNodeIdEntry) = Smi::New(value);
+void PcDescriptors::SetDeoptId(intptr_t index, intptr_t value) const {
+ *SmiAddr(index, kDeoptIdEntry) = Smi::New(value);
}
-intptr_t PcDescriptors::TokenIndex(intptr_t index) const {
- return Smi::Value(*SmiAddr(index, kTokenIndexEntry));
+intptr_t PcDescriptors::TokenPos(intptr_t index) const {
+ return Smi::Value(*SmiAddr(index, kTokenPosEntry));
}
-void PcDescriptors::SetTokenIndex(intptr_t index, intptr_t value) const {
- *SmiAddr(index, kTokenIndexEntry) = Smi::New(value);
+void PcDescriptors::SetTokenPos(intptr_t index, intptr_t value) const {
+ *SmiAddr(index, kTokenPosEntry) = Smi::New(value);
}
@@ -6623,7 +6623,7 @@
const intptr_t multi_purpose_index = DescriptorKind(i) == kDeoptIndex ?
DeoptIndex(i) : TryIndex(i);
len += OS::SNPrint(NULL, 0, kFormat,
- PC(i), KindAsStr(i), NodeId(i), TokenIndex(i), multi_purpose_index);
+ PC(i), KindAsStr(i), DeoptId(i), TokenPos(i), multi_purpose_index);
}
// Allocate the buffer.
char* buffer = Isolate::Current()->current_zone()->Alloc<char>(len + 1);
@@ -6633,7 +6633,7 @@
const intptr_t multi_purpose_index = DescriptorKind(i) == kDeoptIndex ?
DeoptIndex(i) : TryIndex(i);
index += OS::SNPrint((buffer + index), (len - index), kFormat,
- PC(i), KindAsStr(i), NodeId(i), TokenIndex(i), multi_purpose_index);
+ PC(i), KindAsStr(i), DeoptId(i), TokenPos(i), multi_purpose_index);
}
return buffer;
}
@@ -6658,18 +6658,18 @@
for (intptr_t i = 0; i < Length(); i++) {
uword pc = PC(i);
PcDescriptors::Kind kind = DescriptorKind(i);
- // 'node_id' is set for kDeopt and kIcCall and must be unique for one kind.
- intptr_t node_id = AstNode::kNoId;
+ // 'deopt_id' is set for kDeopt and kIcCall and must be unique for one kind.
+ intptr_t deopt_id = Isolate::kNoDeoptId;
if (check_ids) {
if ((DescriptorKind(i) == PcDescriptors::kDeopt) ||
(DescriptorKind(i) == PcDescriptors::kIcCall)) {
- node_id = NodeId(i);
+ deopt_id = DeoptId(i);
}
}
for (intptr_t k = i + 1; k < Length(); k++) {
if (kind == DescriptorKind(k)) {
- if (node_id != AstNode::kNoId) {
- ASSERT(NodeId(k) != node_id);
+ if (deopt_id != Isolate::kNoDeoptId) {
+ ASSERT(DeoptId(k) != deopt_id);
}
ASSERT(pc != PC(k));
}
@@ -7196,7 +7196,7 @@
const PcDescriptors& descriptors = PcDescriptors::Handle(pc_descriptors());
for (intptr_t i = 0; i < descriptors.Length(); i++) {
if (descriptors.PC(i) == pc) {
- token_pos = descriptors.TokenIndex(i);
+ token_pos = descriptors.TokenPos(i);
break;
}
}
@@ -7204,10 +7204,10 @@
}
-uword Code::GetDeoptPcAtNodeId(intptr_t node_id) const {
+uword Code::GetDeoptPcAtDeoptId(intptr_t deopt_id) const {
const PcDescriptors& descriptors = PcDescriptors::Handle(pc_descriptors());
for (intptr_t i = 0; i < descriptors.Length(); i++) {
- if ((descriptors.NodeId(i) == node_id) &&
+ if ((descriptors.DeoptId(i) == deopt_id) &&
(descriptors.DescriptorKind(i) == PcDescriptors::kDeopt)) {
return descriptors.PC(i);
}
@@ -7259,11 +7259,11 @@
intptr_t max_id = -1;
for (intptr_t i = 0; i < descriptors.Length(); i++) {
if (descriptors.DescriptorKind(i) == PcDescriptors::kIcCall) {
- intptr_t node_id = descriptors.NodeId(i);
- if (node_id > max_id) {
- max_id = node_id;
+ intptr_t deopt_id = descriptors.DeoptId(i);
+ if (deopt_id > max_id) {
+ max_id = deopt_id;
}
- node_ids->Add(node_id);
+ node_ids->Add(deopt_id);
ic_data_obj = CodePatcher::GetInstanceCallIcDataAt(descriptors.PC(i));
ic_data_objs.Add(ic_data_obj);
}
« runtime/vm/isolate.h ('K') | « runtime/vm/object.h ('k') | runtime/vm/object_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698