varasm.c (assemble_variable): Narrow test for uninitialized without BSS target support.

        * varasm.c (assemble_variable): Narrow test for uninitialized
        without BSS target support.

From-SVN: r55843

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 6d0731f83bb0923114ec0a7e42fedbb13fe91ea3..abcb381ade233e5798bd21d7dd7e05609f0bac3b 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,8 @@
+2002-07-29  David Edelsohn  <edelsohn@gnu.org>
+
+       * varasm.c (assemble_variable): Narrow test for uninitialized
+       without BSS target support.
+
 2002-07-29  Nathan Sidwell  <nathan@codesourcery.com>
 
        * profile.c: Add file comment describing the overall algorithm and

diff --git a/gcc/varasm.c b/gcc/varasm.c
index 435f6f759aa8fde30cd5fbad76e3f86b81f1f813..ce6f7318320b2b9b778905fa69a3353599ebeb91 100644 (file)
--- a/gcc/varasm.c
+++ b/gcc/varasm.c
@@ -1598,7 +1598,9 @@ assemble_variable (decl, top_level, at_end, dont_output_data)
      in .bss, then we have to use .data.  */
   /* ??? We should handle .bss via select_section mechanisms rather than
      via special target hooks.  That would eliminate this special case.  */
-  else if (!DECL_COMMON (decl))
+  /* Duplicate BSS test in asm_emit_uninitialized instead of having it
+     return success or failure for that case.  Shrug.  */
+  else if (TREE_PUBLIC (decl) && !DECL_COMMON (decl))
     ;
 #endif
   else if (DECL_INITIAL (decl) == 0