Re: [lto][patch] Patch to fix multiple C++ streaming issues and a residual problem with preload_common_nodes

[Diego Novillo <dnovillo at google dot com>]

On 8/1/08 3:20 PM, Bill Maddox wrote:
> On Fri, Aug 1, 2008 at 6:21 AM, Diego Novillo <dnovillo@google.com> wrote:
> > On Tue, Jul 29, 2008 at 23:32, Bill Maddox <maddox@google.com> wrote:
>
> > > @@ -2756,7 +2812,9 @@ output_namespace_decl (struct output_blo
> > >    output_tree (ob, decl->decl_common.abstract_origin);
> > >
> > >    gcc_assert (decl->decl_common.mode == 0);
> > > -  gcc_assert (decl->decl_common.align == 1);
> > > +  /* FIXME lto:  I'm seeing both 0 and 1 here, but I don't
> > > +     think it is meaningful.  Verify this.  */
> > > +  /*gcc_assert (decl->decl_common.align == 1);*/
> > The alignment is in principle an arbitrary number of bits.  We
> > should not assert anything here.
>
> What is the significance of DECL_ALIGN in a NAMESPACE_DECL?

I don't know if it has any special significance, other than any other 
_DECL.  Nothing obvious comes up in cp/*.  Jason, Mark?


Diego.