Reviewers: Vyacheslav Egorov, Description: Remove unused function AddGapMove from the register allocator.
Please review this at http://codereview.chromium.org/6355009/ SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/ Affected files: M src/lithium-allocator.h M src/lithium-allocator.cc Index: src/lithium-allocator.cc =================================================================== --- src/lithium-allocator.cc (revision 6414) +++ src/lithium-allocator.cc (working copy) @@ -2013,20 +2013,6 @@ } -void LAllocator::AddGapMove(int pos, LiveRange* prev, LiveRange* next) { - UsePosition* prev_pos = prev->AddUsePosition( - LifetimePosition::FromInstructionIndex(pos)); - UsePosition* next_pos = next->AddUsePosition( - LifetimePosition::FromInstructionIndex(pos)); - LOperand* prev_operand = prev_pos->operand(); - LOperand* next_operand = next_pos->operand(); - LGap* gap = chunk_->GetGapAt(pos); - gap->GetOrCreateParallelMove(LGap::START)-> - AddMove(prev_operand, next_operand); - next_pos->set_hint(prev_operand); -} - - LiveRange* LAllocator::SplitAt(LiveRange* range, LifetimePosition pos) { ASSERT(!range->IsFixed()); TraceAlloc("Splitting live range %d at %d\n", range->id(), pos.Value()); Index: src/lithium-allocator.h =================================================================== --- src/lithium-allocator.h (revision 6414) +++ src/lithium-allocator.h (working copy) @@ -591,7 +591,6 @@ void Spill(LiveRange* range); bool IsBlockBoundary(LifetimePosition pos); - void AddGapMove(int pos, LiveRange* prev, LiveRange* next); // Helper methods for resolving control flow. void ResolveControlFlow(LiveRange* range, -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
