Remove obsolete LiveRange::FirstPosWithHint helper.

R=svenpanne@chromium.org

Review URL: https://codereview.chromium.org/14650016

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14543 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
parent dbcd0e92
...@@ -229,13 +229,6 @@ bool LiveRange::CanBeSpilled(LifetimePosition pos) { ...@@ -229,13 +229,6 @@ bool LiveRange::CanBeSpilled(LifetimePosition pos) {
} }
UsePosition* LiveRange::FirstPosWithHint() const {
UsePosition* pos = first_pos_;
while (pos != NULL && !pos->HasHint()) pos = pos->next();
return pos;
}
LOperand* LiveRange::CreateAssignedOperand(Zone* zone) { LOperand* LiveRange::CreateAssignedOperand(Zone* zone) {
LOperand* op = NULL; LOperand* op = NULL;
if (HasRegisterAssigned()) { if (HasRegisterAssigned()) {
...@@ -1813,26 +1806,23 @@ bool LAllocator::TryAllocateFreeReg(LiveRange* current) { ...@@ -1813,26 +1806,23 @@ bool LAllocator::TryAllocateFreeReg(LiveRange* current) {
free_until_pos[cur_reg] = Min(free_until_pos[cur_reg], next_intersection); free_until_pos[cur_reg] = Min(free_until_pos[cur_reg], next_intersection);
} }
UsePosition* hinted_use = current->FirstPosWithHint(); LOperand* hint = current->FirstHint();
if (hinted_use != NULL) { if (hint != NULL && (hint->IsRegister() || hint->IsDoubleRegister())) {
LOperand* hint = hinted_use->hint(); int register_index = hint->index();
if (hint->IsRegister() || hint->IsDoubleRegister()) { TraceAlloc(
int register_index = hint->index(); "Found reg hint %s (free until [%d) for live range %d (end %d[).\n",
TraceAlloc( RegisterName(register_index),
"Found reg hint %s (free until [%d) for live range %d (end %d[).\n", free_until_pos[register_index].Value(),
RegisterName(register_index), current->id(),
free_until_pos[register_index].Value(), current->End().Value());
current->id(),
current->End().Value()); // The desired register is free until the end of the current live range.
if (free_until_pos[register_index].Value() >= current->End().Value()) {
// The desired register is free until the end of the current live range. TraceAlloc("Assigning preferred reg %s to live range %d\n",
if (free_until_pos[register_index].Value() >= current->End().Value()) { RegisterName(register_index),
TraceAlloc("Assigning preferred reg %s to live range %d\n", current->id());
RegisterName(register_index), SetLiveRangeAssignedRegister(current, register_index, mode_, zone_);
current->id()); return true;
SetLiveRangeAssignedRegister(current, register_index, mode_, zone_);
return true;
}
} }
} }
......
...@@ -328,10 +328,10 @@ class LiveRange: public ZoneObject { ...@@ -328,10 +328,10 @@ class LiveRange: public ZoneObject {
return assigned_register_ != kInvalidAssignment; return assigned_register_ != kInvalidAssignment;
} }
bool IsSpilled() const { return spilled_; } bool IsSpilled() const { return spilled_; }
UsePosition* FirstPosWithHint() const;
LOperand* FirstHint() const { LOperand* FirstHint() const {
UsePosition* pos = FirstPosWithHint(); UsePosition* pos = first_pos_;
while (pos != NULL && !pos->HasHint()) pos = pos->next();
if (pos != NULL) return pos->hint(); if (pos != NULL) return pos->hint();
return NULL; return NULL;
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment