[llvm-commits] [llvm] r78604 - in /llvm/trunk: lib/Target/ARM/ARMInstrFormats.td lib/Target/ARM/ARMInstrNEON.td lib/Target/ARM/ARMInstrVFP.td test/CodeGen/ARM/fsitos.ll test/CodeGen/ARM/ftosizs.ll test/CodeGen/ARM/ftouizs.ll test/CodeGen/ARM/fuitos.ll

[Eric Christopher]

Hi David,

This patch:

On Aug 10, 2009, at 3:17 PM, David Goodwin wrote:

> URL: http://llvm.org/viewvc/llvm-project?rev=78604&view=rev
> Log:
> Use NEON for single-precision int<->FP conversions.
>
> Added:
>    llvm/trunk/test/CodeGen/ARM/fsitos.ll
>    llvm/trunk/test/CodeGen/ARM/ftosizs.ll
>    llvm/trunk/test/CodeGen/ARM/ftouizs.ll
>    llvm/trunk/test/CodeGen/ARM/fuitos.ll
> Modified:
>    llvm/trunk/lib/Target/ARM/ARMInstrFormats.td
>    llvm/trunk/lib/Target/ARM/ARMInstrNEON.td
>    llvm/trunk/lib/Target/ARM/ARMInstrVFP.td

is causing this tablegen warning:

Warning: neverHasSideEffects set on instruction 'VCVTs2fd_sfp' which  
already has a pattern
Warning: neverHasSideEffects set on instruction 'VCVTu2fd_sfp' which  
already has a pattern
Warning: neverHasSideEffects set on instruction 'VCVTs2fd_sfp' which  
already has a pattern
Warning: neverHasSideEffects set on instruction 'VCVTu2fd_sfp' which  
already has a pattern

think you could fix it up? Thanks.

-eric