This is the mail archive of the binutils@sourceware.cygnus.com mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

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


   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

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]