Hi,

On Thu, 14 Apr 2011, Richard Guenther wrote:

> > +  if (align > DECL_ALIGN (decl))
> > +    DECL_ALIGN (decl) = align;
> 
> Shouldn't this unconditionally set DECL_ALIGN in case
> LOCAL_DECL_ALINGMENT returns something smaller?

Decreasing alignment of DECLs points to a problem elsewhere, so perhaps an 
assert that this doesn't happen is better.  Decreasing is a problem 
because it's not conservative: there might have been code generated 
already assuming the once larger alignment that then possibly breaks if it 
turns out the alignment is actually smaller.


Ciao,
Michael.

Reply via email to