[llvm-commits] [llvm] r91614 - in /llvm/trunk: include/llvm/CodeGen/ValueTypes.h lib/CodeGen/SelectionDAG/LegalizeDAG.cpp

[Ken Dyck]

On Wednesday, January 06, 2010 10:55 AM, Duncan Sands wrote:
> 
> > Perhaps I'm misunderstanding, but with this approach, 
> wouldn't a store 
> > of i64 with an alignment of 2 expand to two i24 stores and one i16 
> > store since the largest legal type of alignment 2 is i24? 
> (Presumably 
> > the one unaligned i24 store would need to be further expanded to an 
> > i16 and an
> > i8 store).
> 
> yes, but that's correct!  

I agree. In your original example, you wrote "a store of i64 with
alignment 2 would be expanded into two i32 stores". Obviously a typo.

-Ken