[llvm-commits] [llvm] r60964 - /llvm/trunk/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp

[Duncan Sands]

Hi Dale,

> > While there I noticed that the existing code doesn't
> > handle trunc store of f64 to f32: it turns this into
> > an i64 store, which represents a 4 byte stack smash.
> > I added a FIXME about this.  Hopefully someone more
> > motivated than I am will take care of it.
> 
> This only occurs if f64 is not a legal type, right?

I don't think so: the DAG combiner can create these:

  // If this is an FP_ROUND or TRUNC followed by a store, fold this into a
  // truncating store.  We can do this even if this is already a truncstore.

Ciao,

Duncan.