* config/tc-mips.c (s_change_section): When parsing the MIPS-specific
authorRichard Sandiford <rdsandiford@googlemail.com>
Thu, 18 Dec 2003 10:18:17 +0000 (10:18 +0000)
committerRichard Sandiford <rdsandiford@googlemail.com>
Thu, 18 Dec 2003 10:18:17 +0000 (10:18 +0000)
.section syntax, map SHT_MIPS_DWARF to SHT_PROGBITS.

gas/ChangeLog
gas/config/tc-mips.c

index 2ee1c9105a210f2cf78c96d8d29f576adcc2c60f..aec39ccda3036b895eb924586b47cf803cef68de 100644 (file)
@@ -1,3 +1,8 @@
+2003-12-18  Richard Sandiford  <rsandifo@redhat.com>
+
+       * config/tc-mips.c (s_change_section): When parsing the MIPS-specific
+       .section syntax, map SHT_MIPS_DWARF to SHT_PROGBITS.
+
 2003-12-17  Mark Mitchell  <mark@codesourcery.com>
 
        * config/tc-arm.c (arm_archs): Change "armv6" to "armv6j".
index d60e75f4954c17a5e83554e21305a53e5d170e60..e6997ca78f6aac6cf441917cfed60226dd070361 100644 (file)
@@ -11981,6 +11981,22 @@ s_change_section (int ignore ATTRIBUTE_UNUSED)
 
   section_name = xstrdup (section_name);
 
+  /* When using the generic form of .section (as implemented by obj-elf.c),
+     there's no way to set the section type to SHT_MIPS_DWARF.  Users have
+     traditionally had to fall back on the more common @progbits instead.
+
+     There's nothing really harmful in this, since bfd will correct
+     SHT_PROGBITS to SHT_MIPS_DWARF before writing out the file.  But it
+     means that, for backwards compatibiltiy, the special_section entries
+     for dwarf sections must use SHT_PROGBITS rather than SHT_MIPS_DWARF.
+
+     Even so, we shouldn't force users of the MIPS .section syntax to
+     incorrectly label the sections as SHT_PROGBITS.  The best compromise
+     seems to be to map SHT_MIPS_DWARF to SHT_PROGBITS before calling the
+     generic type-checking code.  */
+  if (section_type == SHT_MIPS_DWARF)
+    section_type = SHT_PROGBITS;
+
   obj_elf_change_section (section_name, section_type, section_flag,
                          section_entry_size, 0, 0, 0);