[llvm-commits] [llvm] r98889 - in /llvm/trunk/lib/Target/ARM: ARMISelLowering.cpp ARMISelLowering.h ARMInstrFormats.td ARMInstrInfo.td ARMInstrNEON.td ARMInstrVFP.td

[Evan Cheng]

On Mar 19, 2010, at 3:17 PM, Anton Korobeynikov wrote:

> Hi, Bob
> 
>> This looks like a nice cleanup but it is causing performance regressions.  Since I don't get the impression from your commit message that this is critical to your work, I'm going to revert it for now.  If I can find a small testcase, I will certainly send it to you.  Otherwise, I may see if I can figure out what's going wrong.  Basically I'm seeing lots of things like:
>> 
>>        vmov    r2, s2
>>        vmov    s2, r2
> Hrm, maybe we need to teach cross-rc coalescer to remove these moves away?

Can't. Each of these moves by itself is not a nop.

Evan

> 
> -- 
> With best regards, Anton Korobeynikov
> Faculty of Mathematics and Mechanics, Saint Petersburg State University
> 
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits