[llvm-commits] [llvm] r99436 - in /llvm/trunk/lib/Target/ARM: ARMInstrFormats.td ARMInstrInfo.td ARMInstrVFP.td ARMSubtarget.cpp ARMSubtarget.h

[Jim Grosbach]

On Mar 24, 2010, at 10:38 PM, Anton Korobeynikov wrote:

> Hi, Jim
> 
>> Make the use of the vmla and vmls VFP instructions controllable via cmd line.
>> Preliminary testing shows significant performance wins by not using these
>> instructions.
> not using where? I believe this should be made conditional on A8 only.
> 

Have you run benchmarks on other variants showing the instructions to be a win? This looks to be a pretty fundamental flaw in the VFP unit.

-Jim