[llvm-commits] [llvm] r56384 - in /llvm/trunk/lib/CodeGen: RegAllocLinearScan.cpp SimpleRegisterCoalescing.cpp

[Chris Lattner]

On Sep 23, 2008, at 3:17 PM, Dale Johannesen wrote:

>>>
>>> I would strongly prefer this.  Even if there are cases that don't
>>> "just work", I think this is the correct way to model it, and the
>>> right long-term answer.
>>
>> Ok fine. Sorry for being overly careful when it comes to the
>> coalescer.
>
> This did basically Just Work (I haven't run the gcc testsuite yet, but
> it doesn't break anything in
> the llvm testsuite and fixes all the examples in our bug reports.)

Great!

> Unfortunately, this approach
> only works in the linear scan RA, while we use a different RA with -
> O0.... I guess I need to fix
> that before turning it on.

Yep, that's expected.  Thanks a lot Dale!

-Chris