2010-09-22 Kai Tietz <kai.tietz@onevision.com>
authorKai Tietz <kai.tietz@onevision.com>
Wed, 22 Sep 2010 14:20:24 +0000 (14:20 +0000)
committerKai Tietz <kai.tietz@onevision.com>
Wed, 22 Sep 2010 14:20:24 +0000 (14:20 +0000)
        * ldlang.c (lang_add_section): Allow for debugging
        section to be marked as noload but to keep content.
        (IGNORE_SECTION): Likewise.
        (lang_check_section_addresses): Likewise.
        * ldwrite.c (build_link_order): Likewise.

ld/ChangeLog
ld/ldlang.c
ld/ldwrite.c

index c7169e02a71bf08d613499847f865fc29bc71398..0f4fa4365708f5b0cced6bf570616d538c059a85 100644 (file)
@@ -1,5 +1,11 @@
 2010-09-22  Kai Tietz  <kai.tietz@onevision.com>
 
+       * ldlang.c (lang_add_section): Allow for debugging
+       section to be marked as noload but to keep content.
+       (IGNORE_SECTION): Likewise.
+       (lang_check_section_addresses): Likewise.
+       * ldwrite.c (build_link_order): Likewise.
+
        * emultempl/pe.em (gld_${EMULATION_NAME}_place_orphan): Add
        idata to orphan set.
        * emultempl/pep.em: Likewise.
index 75fa25c28d01d0b836d117178597d294e2b0a98a..1c3861831504b51afd8f15a7d0500cf32f0a7894 100644 (file)
@@ -2245,7 +2245,15 @@ lang_add_section (lang_statement_list_type *ptr,
     case noload_section:
       flags &= ~SEC_LOAD;
       flags |= SEC_NEVER_LOAD;
-      if ((flags & SEC_COFF_SHARED_LIBRARY) == 0)
+      if (((bfd_get_flavour (section->owner)
+           == bfd_target_ecoff_flavour)
+          || (bfd_get_flavour (section->owner)
+              == bfd_target_coff_flavour)))
+       {
+         if ((flags & (SEC_COFF_SHARED_LIBRARY | SEC_DEBUGGING)) == 0)
+           flags &= ~SEC_HAS_CONTENTS;
+       }
+      else
        flags &= ~SEC_HAS_CONTENTS;
       break;
     }
@@ -4555,8 +4563,11 @@ sort_sections_by_lma (const void *arg1, const void *arg2)
   return 0;
 }
 
+/* On ELF, a debugging section must never set SEC_NEVER_LOAD, as no output
+   would be written for it. So the combination of debugging and never-load
+   is something which can only happen for pe-coff and must not be ignored.  */
 #define IGNORE_SECTION(s) \
-  ((s->flags & SEC_NEVER_LOAD) != 0                            \
+  ((s->flags & (SEC_NEVER_LOAD | SEC_DEBUGGING)) == SEC_NEVER_LOAD     \
    || (s->flags & SEC_ALLOC) == 0                              \
    || ((s->flags & SEC_THREAD_LOCAL) != 0                      \
        && (s->flags & SEC_LOAD) == 0))
@@ -4590,7 +4601,7 @@ lang_check_section_addresses (void)
   for (s = link_info.output_bfd->sections; s != NULL; s = s->next)
     {
       /* Only consider loadable sections with real contents.  */
-      if ((s->flags & SEC_NEVER_LOAD)
+      if ((s->flags & (SEC_NEVER_LOAD | SEC_DEBUGGING)) == SEC_NEVER_LOAD
          || !(s->flags & SEC_LOAD)
          || !(s->flags & SEC_ALLOC)
          || s->size == 0)
index 6b8423b568317a22b3d4aebc0fe72deb3b6d129d..d0616fcbd99b82798f92ed81366bc16a85f39c91 100644 (file)
@@ -245,7 +245,8 @@ build_link_order (lang_statement_union_type *statement)
                link_order = bfd_new_link_order (link_info.output_bfd,
                                                 output_section);
 
-               if (i->flags & SEC_NEVER_LOAD)
+               if ((i->flags & SEC_NEVER_LOAD) != 0
+                   && (i->flags & SEC_DEBUGGING) == 0)
                  {
                    /* We've got a never load section inside one which
                       is going to be output, we'll change it into a