Patchwork [H8300] Remove ASM_OUTPUT_BSS

login
register
mail settings
Submitter Anatoly Sokolov
Date March 27, 2011, 8:32 p.m.
Message ID <519161994.20110328003210@post.ru>
Download mbox | patch
Permalink /patch/88525/
State New
Headers show

Comments

Anatoly Sokolov - March 27, 2011, 8:32 p.m.
Hi.

  In config/h8300/h8300.h file both ASM_OUTPUT_BSS and ASM_OUTPUT_ALIGNED_BSS 
macros is defined, but the ASM_OUTPUT_BSS macro is not used when 
ASM_OUTPUT_ALIGNED_BSS is defined. This patch remove ASM_OUTPUT_BSS macro 
from H8300 target. 

  Regression tested on h8300-unknown-elf with no new failure.

  OK to install? 

        * config/h8300/h8300.h (ASM_OUTPUT_BSS): Remove macro.



Anatoly.
Jeff Law - March 28, 2011, 2:31 p.m.
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 03/27/11 14:32, Anatoly Sokolov wrote:
> Hi.
> 
>   In config/h8300/h8300.h file both ASM_OUTPUT_BSS and ASM_OUTPUT_ALIGNED_BSS 
> macros is defined, but the ASM_OUTPUT_BSS macro is not used when 
> ASM_OUTPUT_ALIGNED_BSS is defined. This patch remove ASM_OUTPUT_BSS macro 
> from H8300 target. 
> 
>   Regression tested on h8300-unknown-elf with no new failure.
> 
>   OK to install? 
> 
>         * config/h8300/h8300.h (ASM_OUTPUT_BSS): Remove macro.

OK.  Please install.

Thanks,
jeff
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/

iQEcBAEBAgAGBQJNkJu+AAoJEBRtltQi2kC7h5wIALuAx8X1VFeEHYzrlwBJA1nO
GXK8DCcs6vRQvkvoNLXH2juQSis8DeSbFBc+LH8ut3f/cepB7RZs4SKS9RgkdfzZ
UZhZdB2qLKUCfq6UNqRgmvmGFYUqI+tlv+GH9lJzzX3M6KmxW/XgPiqg8xHUSWo0
8HGoP3W9dsY6+Fpxjx0nf3dIykp4SMIWj1O9FUkkUF+nVxcB63AuIhCIyIY9z+O8
ihgiJFnVnugoMkpxH+K9fLxz+HnAKG2ltVP7Hb7h3QxfKuDv5hGR77uu8BpA//sI
2tQdPhOD3u9kb2CVDN7fcc3wScpcjN7xl3/YL9gSI0iLJawhYryD7HMFwD1H5CA=
=4nDS
-----END PGP SIGNATURE-----

Patch

Index: gcc/config/h8300/h8300.h
===================================================================
--- gcc/config/h8300/h8300.h    (revision 171427)
+++ gcc/config/h8300/h8300.h    (working copy)
@@ -1015,13 +987,6 @@ 
   assemble_name ((FILE), (NAME)),                      \
   fprintf ((FILE), ",%lu\n", (unsigned long)(SIZE)))
 
-/* This says how to output the assembler to define a global
-   uninitialized but not common symbol.
-   Try to use asm_output_bss to implement this macro.  */
-
-#define ASM_OUTPUT_BSS(FILE, DECL, NAME, SIZE, ROUNDED)                \
-  asm_output_bss ((FILE), (DECL), (NAME), (SIZE), (ROUNDED))
-
 #define ASM_OUTPUT_ALIGNED_BSS(FILE, DECL, NAME, SIZE, ALIGN) \
   asm_output_aligned_bss (FILE, DECL, NAME, SIZE, ALIGN)