[llvm-commits] [llvm] r111499 - /llvm/trunk/include/llvm/Target/TargetRegisterInfo.h

Daniel Dunbar daniel at zuster.org
Thu Aug 19 10:03:28 PDT 2010


Hi Eric,

Did you time the performance impact of this change?

I have some tentative evidence that it may be involved in some compile
time regressions, but unfortunately the Clang build was broken at the
time so I don't have an exact commit identified yet.

 - Daniel

On Wed, Aug 18, 2010 at 6:25 PM, Eric Christopher <echristo at apple.com> wrote:
> Author: echristo
> Date: Wed Aug 18 20:25:50 2010
> New Revision: 111499
>
> URL: http://llvm.org/viewvc/llvm-project?rev=111499&view=rev
> Log:
> Reapply the virtual register patch from 109102. The places where we were
> depending on the number of virtual registers appear to have all been handled
> now.
>
> Modified:
>    llvm/trunk/include/llvm/Target/TargetRegisterInfo.h
>
> Modified: llvm/trunk/include/llvm/Target/TargetRegisterInfo.h
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Target/TargetRegisterInfo.h?rev=111499&r1=111498&r2=111499&view=diff
> ==============================================================================
> --- llvm/trunk/include/llvm/Target/TargetRegisterInfo.h (original)
> +++ llvm/trunk/include/llvm/Target/TargetRegisterInfo.h Wed Aug 18 20:25:50 2010
> @@ -301,7 +301,7 @@
>     /// considered to be a 'virtual' register, which is part of the SSA
>     /// namespace.  This must be the same for all targets, which means that each
>     /// target is limited to this fixed number of registers.
> -    FirstVirtualRegister = 1024
> +    FirstVirtualRegister = 16384
>   };
>
>   /// isPhysicalRegister - Return true if the specified register number is in
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>




More information about the llvm-commits mailing list