[llvm-commits] [llvm] r61915 - in /llvm/trunk: lib/Transforms/Scalar/ScalarReplAggregates.cpp test/Transforms/ScalarRepl/copy-aggregate.ll

[Chris Lattner]

On Thu, 8 Jan 2009, Duncan Sands wrote:
> Hi Chris,
>> +/// RewriteLoadUserOfWholeAlloca - We found an load of the entire allocation to
>> +/// an integer.  Load the individual pieces to form the aggregate value.
>
> what if the type has alignment or other padding?  Probably you should just not bother
> if HasPadding returns true.

Before deciding that it is safe to do the xform, SROA.cpp:612 sets 
"isMemCpySrc" to true if there is a load like this.  "isMemCpySrc" is used 
to handle the padding cases.

-Chris

-- 
http://nondot.org/sabre/
http://llvm.org/