Re: c/6873: Improper Alignment Parameters Emitted by Compiler for .comm Section

Re: c/6873: Improper Alignment Parameters Emitted by Compiler for .comm Section

[Dara Hazeghi]

The following reply was made to PR c/6873; it has been noted by GNATS.

From: Dara Hazeghi <dhazeghi@yahoo.com>
To: gcc-gnats@gcc.gnu.org, mmitchel@qualcomm.com
Cc:  
Subject: Re: c/6873: Improper Alignment Parameters Emitted by Compiler for .comm Section
Date: Sat, 10 May 2003 16:37:18 -0700

 http://gcc.gnu.org/cgi-bin/gnatsweb.pl?cmd=view%20audit- 
 trail&database=gcc&pr=6873
 
 Hello,
 
 would it be possible for you to determine whether this bug still exists  
 in a more current version of gcc (ie 3.2.3)? Thanks,
 
 Dara
 

Re: c/6873: Improper Alignment Parameters Emitted by Compiler for .comm Section

[steven]

Synopsis: Improper Alignment Parameters Emitted by Compiler for .comm Section

State-Changed-From-To: open->feedback
State-Changed-By: steven
State-Changed-When: Mon May 12 10:45:52 2003
State-Changed-Why:
    Old bug. Dara asked for feedback.

http://gcc.gnu.org/cgi-bin/gnatsweb.pl?cmd=view%20audit-trail&database=gcc&pr=6873

c/6873: Improper Alignment Parameters Emitted by Compiler for .comm Section

[mmitchel]

>Number:         6873
>Category:       c
>Synopsis:       Improper Alignment Parameters Emitted by Compiler for .comm Section
>Confidential:   no
>Severity:       critical
>Priority:       medium
>Responsible:    unassigned
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Thu May 30 13:26:01 PDT 2002
>Closed-Date:
>Last-Modified:
>Originator:     mmitchel@qualcomm.com
>Release:        gcc version 3.1 arm-elf
>Organization:
>Environment:
cross compile, windows nt, cygwin

$ arm-elf-gcc -v
Reading specs from /gnude/lib/gcc-lib/arm-elf/3.1/specs
Configured with: ../src/configure --target=arm-elf --host=i686-pc-cygwin --prefi
x=/gnude --with-local-prefix=/gnude/local --with-gnu-as --with-gnu-ld
Thread model: single
gcc version 3.1
>Description:
The compiler not emitting correct assembly output for following example:

$ more test.c
int i;
int j;
$ arm-elf-gcc -S test.c
$ more test.s
        .file   "test.c"
        .comm   i, 4, 32
        .comm   j, 4, 32
        .ident  "GCC: (GNU) 3.1"

the third paramater, alignment parameter, to the .comm pseudo op is incorrect.

Incorrect output impacts memory use and requirements of generating small code for embedded targets.
>How-To-Repeat:

>Fix:
In the file gcc\config\arm\elf.h, at line 172, scale the parameter ALIGN by dividing by BITS_PER_UNIT. This is how this macro is implemented for other targets; this may be an undesired omission for this target.
>Release-Note:
>Audit-Trail:
>Unformatted: