The "b" flag for COFF sections only unsets the LOAD attribute. It should also clear...

	* config/obj-coff.c (obj_coff_section): Set the bss flag on
	sections with the "b" attribute.

diff --git a/gas/ChangeLog b/gas/ChangeLog
index dcac44c638ea4f361ac687f132305b6cf55e2fe1..fc436ff3c3af7d35104fe72953129bbf22c42b95 100644 (file)
--- a/gas/ChangeLog
+++ b/gas/ChangeLog
@@ -1,3 +1,8 @@
+2020-03-25  J.W. Jagersma  <jwjagersma@gmail.com>
+
+       * config/obj-coff.c (obj_coff_section): Set the bss flag on
+       sections with the "b" attribute.
+
 2020-03-22  Alan Modra  <amodra@gmail.com>
 
        * testsuite/gas/s12z/truncated.d: Update expected output.

diff --git a/gas/config/obj-coff.c b/gas/config/obj-coff.c
index 178ccaad6a6f90e7b01dec1e476e59f299e9d98c..70d9fb32d5052aa5e11ef2d4c074e90a29bdfa21 100644 (file)
--- a/gas/config/obj-coff.c
+++ b/gas/config/obj-coff.c
@@ -1532,6 +1532,7 @@ obj_coff_section (int ignore ATTRIBUTE_UNUSED)
   unsigned int exp;
   flagword flags, oldflags;
   asection *sec;
+  bfd_boolean is_bss = FALSE;
 
   if (flag_mri)
     {
@@ -1581,6 +1582,7 @@ obj_coff_section (int ignore ATTRIBUTE_UNUSED)
                  /* Uninitialised data section.  */
                  flags |= SEC_ALLOC;
                  flags &=~ SEC_LOAD;
+                 is_bss = TRUE;
                  break;
 
                case 'n':
@@ -1652,6 +1654,9 @@ obj_coff_section (int ignore ATTRIBUTE_UNUSED)
 
   sec = subseg_new (name, (subsegT) exp);
 
+  if (is_bss)
+    seg_info (sec)->bss = 1;
+
   if (alignment >= 0)
     sec->alignment_power = alignment;