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

[Eric Christopher]

On Aug 19, 2010, at 10:37 AM, Bob Wilson wrote:

> 
> On Aug 19, 2010, at 10:33 AM, Eric Christopher wrote:
> 
>> 
>> On Aug 19, 2010, at 10:28 AM, Daniel Dunbar wrote:
>> 
>>> On Thu, Aug 19, 2010 at 10:20 AM, Eric Christopher <echristo at apple.com> wrote:
>>>> 
>>>> On Aug 19, 2010, at 10:03 AM, Daniel Dunbar wrote:
>>>> 
>>>>> 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.
>>>> 
>>>> I didn't, I'll back it out again.  It seems that my nightly tester didn't like it
>>>> either.
>>> 
>>> Ok, you don't need to back it out for my sake, I was just curious.
>>> Although at least this way I should be able to tell if it was the
>>> cause or not.
>> 
>> Yeah something funky was happening on one of my testers too so I'll back it out to
>> double check.
> 
> Unlike the last time, I didn't notice any compile time regressions on any of the ARM testers.

OK.  I think the plaid run on my testers was a configuration error on my side.  I'm double checking and if Daniel says that it looks good on his side (no change) then I'll re-recommit it :)

-eric