Phase 1 of refactoring the MachineRegisterInfo iterators to make them suitable
authorOwen Anderson <resistor@mac.com>
Thu, 13 Mar 2014 06:02:25 +0000 (06:02 +0000)
committerOwen Anderson <resistor@mac.com>
Thu, 13 Mar 2014 06:02:25 +0000 (06:02 +0000)
commit76604af7748badc535017534f5f4c627516fdd7a
treedc890f85693c35cc98af8b75950334fb965ce358
parent178df3f1bb3e75a41af58be923852f8d295d5dd4
Phase 1 of refactoring the MachineRegisterInfo iterators to make them suitable
for use with C++11 range-based for-loops.

The gist of phase 1 is to remove the skipInstruction() and skipBundle()
methods from these iterators, instead splitting each iterator into a version
that walks operands, a version that walks instructions, and a version that
walks bundles.  This has the result of making some "clever" loops in lib/CodeGen
more verbose, but also makes their iterator invalidation characteristics much
more obvious to the casual reader. (Making them concise again in the future is a
good motivating case for a pre-incrementing range adapter!)

Phase 2 of this undertaking with consist of removing the getOperand() method,
and changing operator*() of the operand-walker to return a MachineOperand&.  At
that point, it should be possible to add range views for them that work as one
might expect.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@203757 91177308-0d34-0410-b5e6-96231b3b80d8
include/llvm/CodeGen/MachineRegisterInfo.h
lib/CodeGen/CalcSpillWeights.cpp
lib/CodeGen/InlineSpiller.cpp
lib/CodeGen/LiveIntervalAnalysis.cpp
lib/CodeGen/RegAllocBase.cpp
lib/CodeGen/RegisterCoalescer.cpp
lib/CodeGen/RegisterPressure.cpp
lib/CodeGen/SelectionDAG/SelectionDAGISel.cpp