[PATCH] Re: Link error with today's CVS binutils

Ian Lance Taylor ian@zembu.com
Mon Jan 17 21:38:00 GMT 2000


   Date: Mon, 17 Jan 2000 18:25:17 -0800
   From: "H . J . Lu" <hjl@valinux.com>

   Have you followed this thread? Your change:

   1999-01-15  Nick Clifton  <nickc@cygnus.com>

	   * ldlang.c (lang_size_sections): Only update the current 
	     address of a region if the section just placed into it is an
	     allocated section.

   seems to cause the regression. This patch seems to fix it. Any
   comments?

Thanks for tracking this down.

I think that checking the size of the section is not the right way to
go here.  Instead, check for SEC_NEVER_LOAD.  Look at how
SEC_NEVER_LOAD is handled in ldwrite.c.

Ian


More information about the Binutils mailing list