This is the mail archive of the binutils@sourceware.org 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]
Other format: [Raw text]

Re: [PATCH]: Fix ld bug triggered in arm-*-pe (?)


Alan Modra wrote:
On Mon, Aug 14, 2006 at 11:52:04PM +0100, Pedro Alves wrote:
   * ldlang.c (lang_size_sections_1, lang_assignment_statement_enum):
   Adjust the current address of DEFAULT_MEMORY_REGION even
   when dot hasn't changed.

This is OK.



Thanks,


Could someone commit this, please?

Attached is the same patch, but with the formatting fixed,
for your convenience.

Cheers,
Pedro Alves

---

ld/ChangeLog

2006-08-24 Pedro Alves <pedro_alves@portugalmail.pt>

   * ldlang.c (lang_size_sections_1, lang_assignment_statement_enum):
   Adjust the current address of DEFAULT_MEMORY_REGION even
   when dot hasn't changed.
Index: ldlang.c
===================================================================
RCS file: /cvs/src/src/ld/ldlang.c,v
retrieving revision 1.235
diff -u -p -r1.235 ldlang.c
--- ldlang.c	22 Aug 2006 07:41:05 -0000	1.235
+++ ldlang.c	24 Aug 2006 12:56:38 -0000
@@ -4621,7 +4621,7 @@ lang_size_sections_1
 			   output_section_statement->bfd_section,
 			   &newdot);
 
-	    if (newdot != dot && !output_section_statement->ignored)
+	    if (!output_section_statement->ignored)
 	      {
 		if (output_section_statement == abs_output_section)
 		  {
@@ -4630,7 +4630,7 @@ lang_size_sections_1
 		    lang_memory_region_lookup (DEFAULT_MEMORY_REGION,
 					       FALSE)->current = newdot;
 		  }
-		else
+		else if (newdot != dot)
 		  {
 		    /* Insert a pad after this statement.  We can't
 		       put the pad before when relaxing, in case the

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