> Evan's patch to avoid FPreg->intreg copy for cvt; store to mem > > +// avoid copying result of FTOUIZ to int reg when we're just > storing it > +let AddedComplexity = 1 in > +def : ARMPat<(store (i32 (bitconvert SPR:$src)), addrmode5:$addr), > + (FSTS SPR:$src, addrmode5:$addr)>; > + > // > ===------------------------------------------------------------------- > ---===// > // Load / store multiple Instructions. > //
Interesting approach. Instead of adding this to one target's .td file, can you please investigate whether we can enable the dag combine xform at DAGCombiner.cpp:3412 now? The primary thing blocking that xform was that we need to know (in this case) that an f32 store doesn't require greater alignment than an i32 store has. -Chris _______________________________________________ llvm-commits mailing list llvm-commits@cs.uiuc.edu http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits