[llvm-commits] [llvm] r72507 - in /llvm/trunk: include/llvm/Target/TargetLowering.h lib/CodeGen/SelectionDAG/DAGCombiner.cpp lib/Target/X86/X86ISelLowering.cpp lib/Target/X86/X86ISelLowering.h test/CodeGen/X86/narrow_op-1.ll test/CodeGen/X86/narrow_op-2.ll

[Duncan Sands]

Hi Evan,

>> Hi Evan, nice transform!  But I think you shouldn't do it if the
>> original store was volatile.  Also, mightn't this turn an aligned
> 
> Why not? This would just reduce the store width.

yes, that's unvalid if you are writing into i/o memory for example.
The very fact of writing to an address can have an effect.  This
is the case even if the value you are writing is the value that
you just read from it.

Ciao,

Duncan.