[llvm-commits] [llvm] r132581 - in /llvm/trunk: lib/CodeGen/AllocationOrder.cpp lib/CodeGen/AllocationOrder.h lib/CodeGen/RegAllocBase.h lib/CodeGen/RegAllocBasic.cpp lib/CodeGen/RegAllocGreedy.cpp test/CodeGen/Thumb2/2009-10-15-ITBlockBranch.ll

[Jay Foad]

Hi Jakob,

On 3 June 2011 21:34, Jakob Stoklund Olesen <stoklund at 2pi.dk> wrote:
> Author: stoklund
> Date: Fri Jun  3 15:34:53 2011
> New Revision: 132581
>
> URL: http://llvm.org/viewvc/llvm-project?rev=132581&view=rev
> Log:
> Switch AllocationOrder to using RegisterClassInfo instead of a BitVector
> of reserved registers.
>
> Use RegisterClassInfo in RABasic as well. This slightly changes som
> allocation orders because RegisterClassInfo puts CSR aliases last.
>
> Modified:
>    llvm/trunk/lib/CodeGen/AllocationOrder.cpp
>    llvm/trunk/lib/CodeGen/AllocationOrder.h
>    llvm/trunk/lib/CodeGen/RegAllocBase.h
>    llvm/trunk/lib/CodeGen/RegAllocBasic.cpp
>    llvm/trunk/lib/CodeGen/RegAllocGreedy.cpp
>    llvm/trunk/test/CodeGen/Thumb2/2009-10-15-ITBlockBranch.ll

> Modified: llvm/trunk/lib/CodeGen/AllocationOrder.h
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/AllocationOrder.h?rev=132581&r1=132580&r2=132581&view=diff
> ==============================================================================
> --- llvm/trunk/lib/CodeGen/AllocationOrder.h (original)
> +++ llvm/trunk/lib/CodeGen/AllocationOrder.h Fri Jun  3 15:34:53 2011
> @@ -19,14 +19,14 @@
>
>  namespace llvm {
>
> -class BitVector;
> +class RegisterClassInfo;
>  class VirtRegMap;
>
>  class AllocationOrder {
>   const unsigned *Begin;
>   const unsigned *End;
>   const unsigned *Pos;
> -  const BitVector &Reserved;
> +  const RegisterClassInfo &RCI;
>   unsigned Hint;
>  public:
>
> @@ -37,7 +37,7 @@
>   ///        TargetRegisterInfo::getReservedRegs().
>   AllocationOrder(unsigned VirtReg,
>                   const VirtRegMap &VRM,
> -                  const BitVector &ReservedRegs);
> +                  const RegisterClassInfo &RegClassInfo);

The comment above this function needs updating for this change.

Thanks,
Jay.