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

[Duncan Sands]

Hi Ken,

> LegalizeOp() recursively calls itself on its result. Won't that cause
> the unaligned i24 store to be expanded into an i16 and i8 store? 

probably, but then you end up with i24 + i16 + i8 + i16 which doesn't
seem like an improvement over 4 x i16.  Also, if you are going to rely
on recursion, then you might as well do the whole thing recursively,
rather than using the loop I suggested.

Ciao,

Duncan.