Re: PATCH: PR 797: Alignment in empty section changes the output layout

["H. J. Lu" <hjl@lucon.org>]

On Thu, May 05, 2005 at 12:07:17PM +0930, Alan Modra wrote:
> On Wed, May 04, 2005 at 11:09:41AM -0700, H. J. Lu wrote:
> > +  if (command_line.relax || !link_info.relocatable)
> > +    {
> >        /* Final extra sizing to report errors.  */
> >        lang_do_assignments (statement_list.head, abs_output_section, NULL, 0);
> > +      if (!link_info.relocatable)
> > +	{
> > +	  lang_mark_used_section ();
> > +	  /* Do it here so that the unused output_sections won't affect
> > +	     memory layout.  */ 
> > +	  strip_unused_output_sections ();
> > +	}
> 
> Can you please look at merging lang_mark_used_section and
> strip_unused_output_sections into strip_excluded_output_sections?
> 
> I think it should be possible if you call lang_do_assignments before
> bfd_gc_sections to ensure that linker script symbols defined inside
> output sections are in the hash table.

I am not sure it will work since the order is quite important. I
got many

/export/build/gnu/binutils-debug/build-i686-linux/ld/ld-new:
tmpdir/ld1: Not enough room for program headers (allocated 2, need 4)
/export/build/gnu/binutils-debug/build-i686-linux/ld/ld-new: final link
failed: Bad value


H.J.
H.J.