[llvm-commits] [llvm] r61934 - in /llvm/trunk: lib/Analysis/ValueTracking.cpp test/Transforms/InstCombine/2009-01-08-AlignAlloca.ll

[Dan Gohman]

On Jan 8, 2009, at 11:28 AM, Chris Lattner wrote:

> Author: lattner
> Date: Thu Jan  8 13:28:38 2009
> New Revision: 61934
>
> URL: http://llvm.org/viewvc/llvm-project?rev=61934&view=rev
> Log:
> ValueTracker can't assume that an alloca with no specified alignment
> will get its preferred alignment.  It has to be careful and  
> cautiously assume
> it will just get the ABI alignment.  This prevents instcombine from  
> rounding
> up the alignment of a load/store without adjusting the alignment of  
> the alloca.

When would a default-alignment Alloca not be given its preferred  
alignment?

Or, if you prefer that instcombine make give all Allocas an explicit
alignment, could you add a comment to this code?

Thanks,

Dan