[llvm-commits] [llvm] r59648 - /llvm/trunk/lib/Target/CellSPU/SPUISelLowering.cpp

[Duncan Sands]

Hi Scott,

> +  setTruncStoreAction(MVT::i8,    MVT::i1, Promote);
> +  setTruncStoreAction(MVT::i16 ,  MVT::i1, Custom);
> +  setTruncStoreAction(MVT::i32 ,  MVT::i1, Custom);
> +  setTruncStoreAction(MVT::i64 ,  MVT::i1, Custom);
> +  setTruncStoreAction(MVT::i128,  MVT::i1, Custom);

I don't think you need these.  In fact I'm fairly sure
that they will never even be queried.  For extending
loads the situation is slightly different: you probably
want to set the extending load action to promote (see
the comments in LegalizeDAG.cpp, lines 2098 to 2104).
Even there you probably only need to set it for the
extending load of i1 to i8 case.

Ciao,

Duncan.