public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug c/14047] New: __progmem__ attribute doesn't work
@ 2004-02-06 12:45 LAL at pro dot onet dot pl
  2004-02-06 15:09 ` [Bug target/14047] " pinskia at gcc dot gnu dot org
                   ` (19 more replies)
  0 siblings, 20 replies; 21+ messages in thread
From: LAL at pro dot onet dot pl @ 2004-02-06 12:45 UTC (permalink / raw)
  To: gcc-bugs

It seems that this version of the GCC does ignore __progmem__ attribute.

Following code (but the same result for other vriable types) produces elf image
with rotor array placed in .data section

const uint8_t rotor[ANIMSTEPS] __attribute__((__progmem__)) = {
        ~(SEG_A)&SEG_MASK, ~(SEG_B)&SEG_MASK, ~(SEG_C)&SEG_MASK,
        ~(SEG_D)&SEG_MASK, ~(SEG_E)&SEG_MASK, ~(SEG_F)&SEG_MASK };

Where SEG_* are simple numeric defines.

Below a fragment of the map file:
.data           0x00800060       0x2c load address 0x00000604
                0x00800060                PROVIDE (__data_start, .)
 *(.data)
 .data          0x00800060       0x2b display.o
                0x00800078                rotor

(AVR)Binutils version 2.14

GCC version:
Reading specs from /usr/lib/gcc/avr/3.4.0/specs
Configured with: ../configure --prefix=/usr --mandir=/usr/share/man
--infodir=/usr/share/info --enable-languages=c --disable-nls --target=avr
i686-redhat-linux-gnu
Thread model: single
gcc version 3.4.0 20040206 (prerelease)

-- 
           Summary: __progmem__ attribute doesn't work
           Product: gcc
           Version: 3.4.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P1
         Component: c
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: LAL at pro dot onet dot pl
                CC: gcc-bugs at gcc dot gnu dot org
 GCC build triplet: i686-redhat-linux-gnu
  GCC host triplet: i686-redhat-linux-gnu
GCC target triplet: avr-unknown-elf


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14047


^ permalink raw reply	[flat|nested] 21+ messages in thread
* [Bug target/14047] __progmem__ attribute doesn't work
  2004-02-06 12:45 [Bug c/14047] New: __progmem__ attribute doesn't work LAL at pro dot onet dot pl
@ 2004-02-06 15:09 ` pinskia at gcc dot gnu dot org
  2004-02-07  9:27 ` berndtrog at yahoo dot com
                   ` (18 subsequent siblings)
  19 siblings, 0 replies; 21+ messages in thread
From: pinskia at gcc dot gnu dot org @ 2004-02-06 15:09 UTC (permalink / raw)
  To: gcc-bugs



-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|c                           |target


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14047


^ permalink raw reply	[flat|nested] 21+ messages in thread
* [Bug target/14047] __progmem__ attribute doesn't work
  2004-02-06 12:45 [Bug c/14047] New: __progmem__ attribute doesn't work LAL at pro dot onet dot pl
  2004-02-06 15:09 ` [Bug target/14047] " pinskia at gcc dot gnu dot org
@ 2004-02-07  9:27 ` berndtrog at yahoo dot com
  2004-02-10  7:43 ` olischulz at web dot de
                   ` (17 subsequent siblings)
  19 siblings, 0 replies; 21+ messages in thread
From: berndtrog at yahoo dot com @ 2004-02-07  9:27 UTC (permalink / raw)
  To: gcc-bugs



-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |berndtrog at yahoo dot com


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14047


^ permalink raw reply	[flat|nested] 21+ messages in thread
* [Bug target/14047] __progmem__ attribute doesn't work
  2004-02-06 12:45 [Bug c/14047] New: __progmem__ attribute doesn't work LAL at pro dot onet dot pl
  2004-02-06 15:09 ` [Bug target/14047] " pinskia at gcc dot gnu dot org
  2004-02-07  9:27 ` berndtrog at yahoo dot com
@ 2004-02-10  7:43 ` olischulz at web dot de
  2004-02-10 16:21 ` eric at ecentral dot com
                   ` (16 subsequent siblings)
  19 siblings, 0 replies; 21+ messages in thread
From: olischulz at web dot de @ 2004-02-10  7:43 UTC (permalink / raw)
  To: gcc-bugs



-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |olischulz at web dot de


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14047


^ permalink raw reply	[flat|nested] 21+ messages in thread
* [Bug target/14047] __progmem__ attribute doesn't work
  2004-02-06 12:45 [Bug c/14047] New: __progmem__ attribute doesn't work LAL at pro dot onet dot pl
                   ` (2 preceding siblings ...)
  2004-02-10  7:43 ` olischulz at web dot de
@ 2004-02-10 16:21 ` eric at ecentral dot com
  2004-02-11  8:02 ` LAL at pro dot onet dot pl
                   ` (15 subsequent siblings)
  19 siblings, 0 replies; 21+ messages in thread
From: eric at ecentral dot com @ 2004-02-10 16:21 UTC (permalink / raw)
  To: gcc-bugs



-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |eric at ecentral dot com


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14047


^ permalink raw reply	[flat|nested] 21+ messages in thread
* [Bug target/14047] __progmem__ attribute doesn't work
  2004-02-06 12:45 [Bug c/14047] New: __progmem__ attribute doesn't work LAL at pro dot onet dot pl
                   ` (3 preceding siblings ...)
  2004-02-10 16:21 ` eric at ecentral dot com
@ 2004-02-11  8:02 ` LAL at pro dot onet dot pl
  2004-02-11  8:56 ` [Bug target/14047] [3.4/3.5 Regression] " pinskia at gcc dot gnu dot org
                   ` (14 subsequent siblings)
  19 siblings, 0 replies; 21+ messages in thread
From: LAL at pro dot onet dot pl @ 2004-02-11  8:02 UTC (permalink / raw)
  To: gcc-bugs


------- Additional Comments From LAL at pro dot onet dot pl  2004-02-11 08:02 -------
Created an attachment (id=5722)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=5722&action=view)
preprocessed testcase

Command line was:
avr-gcc -g -Wall -Os -mmcu=attiny26  -Wl,-Map,pgmtst.map -save-temps -o pgmtst
test.c


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14047


^ permalink raw reply	[flat|nested] 21+ messages in thread
* [Bug target/14047] [3.4/3.5 Regression] __progmem__ attribute doesn't work
  2004-02-06 12:45 [Bug c/14047] New: __progmem__ attribute doesn't work LAL at pro dot onet dot pl
                   ` (4 preceding siblings ...)
  2004-02-11  8:02 ` LAL at pro dot onet dot pl
@ 2004-02-11  8:56 ` pinskia at gcc dot gnu dot org
  2004-03-01  2:00 ` mmitchel at gcc dot gnu dot org
                   ` (13 subsequent siblings)
  19 siblings, 0 replies; 21+ messages in thread
From: pinskia at gcc dot gnu dot org @ 2004-02-11  8:56 UTC (permalink / raw)
  To: gcc-bugs


------- Additional Comments From pinskia at gcc dot gnu dot org  2004-02-11 08:56 -------
Confirmed.  The problem is that avr was depending on the order of the calls to avr_insert_attributes 
and the actually adding the attribute to the decl.  It really should not depending on them at all.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|                            |1
           Keywords|                            |wrong-code
   Last reconfirmed|0000-00-00 00:00:00         |2004-02-11 08:56:03
               date|                            |
            Summary|__progmem__ attribute       |[3.4/3.5 Regression]
                   |doesn't work                |__progmem__ attribute
                   |                            |doesn't work
   Target Milestone|---                         |3.4.0


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14047


^ permalink raw reply	[flat|nested] 21+ messages in thread
* [Bug target/14047] [3.4/3.5 Regression] __progmem__ attribute doesn't work
  2004-02-06 12:45 [Bug c/14047] New: __progmem__ attribute doesn't work LAL at pro dot onet dot pl
                   ` (5 preceding siblings ...)
  2004-02-11  8:56 ` [Bug target/14047] [3.4/3.5 Regression] " pinskia at gcc dot gnu dot org
@ 2004-03-01  2:00 ` mmitchel at gcc dot gnu dot org
  2004-03-10  5:33 ` eric at ecentral dot com
                   ` (12 subsequent siblings)
  19 siblings, 0 replies; 21+ messages in thread
From: mmitchel at gcc dot gnu dot org @ 2004-03-01  2:00 UTC (permalink / raw)
  To: gcc-bugs


------- Additional Comments From mmitchel at gcc dot gnu dot org  2004-03-01 02:00 -------
AVR is not a primary platform, so I have postponed this until 3.4.1.

The patch here is not quite right.  I believe a correct patch would be to modify
avr_insert_attributes to use 

  avr_progrmem_p (node) || lookup_attribute ("progmem", *attributes)

That would still find a progmem attribute already in the DECL_ATTRIBUTES, which
the patch would not.

If someone will test that change, such a patch would be OK for 3.4.0.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|3.4.0                       |3.4.1


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14047


^ permalink raw reply	[flat|nested] 21+ messages in thread
* [Bug target/14047] [3.4/3.5 Regression] __progmem__ attribute doesn't work
  2004-02-06 12:45 [Bug c/14047] New: __progmem__ attribute doesn't work LAL at pro dot onet dot pl
                   ` (6 preceding siblings ...)
  2004-03-01  2:00 ` mmitchel at gcc dot gnu dot org
@ 2004-03-10  5:33 ` eric at ecentral dot com
  2004-03-11  4:30 ` mmitchel at gcc dot gnu dot org
                   ` (11 subsequent siblings)
  19 siblings, 0 replies; 21+ messages in thread
From: eric at ecentral dot com @ 2004-03-10  5:33 UTC (permalink / raw)
  To: gcc-bugs


------- Additional Comments From eric at ecentral dot com  2004-03-10 05:33 -------
According to comment #4 (by Mark Mitchell) he pre-approved a patch for this for 
3.4 if tested ok. Oliver Schulz tested the patch and confirmed that this works 
on March 2.

Mark, can this be approved and rolled into 3.4 please?

Currently the AVR port is only good up to 3.3.2 (There's a showstopper for 
3.3.3, see #14149). It would be disappointing if we had to wait for 3.4.1 to 
upgrade.

Thanks
Eric

-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14047


^ permalink raw reply	[flat|nested] 21+ messages in thread
* [Bug target/14047] [3.4/3.5 Regression] __progmem__ attribute doesn't work
  2004-02-06 12:45 [Bug c/14047] New: __progmem__ attribute doesn't work LAL at pro dot onet dot pl
                   ` (7 preceding siblings ...)
  2004-03-10  5:33 ` eric at ecentral dot com
@ 2004-03-11  4:30 ` mmitchel at gcc dot gnu dot org
  2004-03-11  4:52 ` kazu at cs dot umass dot edu
                   ` (10 subsequent siblings)
  19 siblings, 0 replies; 21+ messages in thread
From: mmitchel at gcc dot gnu dot org @ 2004-03-11  4:30 UTC (permalink / raw)
  To: gcc-bugs


------- Additional Comments From mmitchel at gcc dot gnu dot org  2004-03-11 04:30 -------
The patch needs to go through a full "make bootstrap && make check" process.  I
can't tell if that's been done, or whether just the small program has been
tested.  If the full test process has been done, and it's been confirmed that
there are no regressions, then it's OK to check in this change for GCC 3.4.

-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14047


^ permalink raw reply	[flat|nested] 21+ messages in thread
* [Bug target/14047] [3.4/3.5 Regression] __progmem__ attribute doesn't work
  2004-02-06 12:45 [Bug c/14047] New: __progmem__ attribute doesn't work LAL at pro dot onet dot pl
                   ` (8 preceding siblings ...)
  2004-03-11  4:30 ` mmitchel at gcc dot gnu dot org
@ 2004-03-11  4:52 ` kazu at cs dot umass dot edu
  2004-03-11  4:53 ` mark at codesourcery dot com
                   ` (9 subsequent siblings)
  19 siblings, 0 replies; 21+ messages in thread
From: kazu at cs dot umass dot edu @ 2004-03-11  4:52 UTC (permalink / raw)
  To: gcc-bugs


------- Additional Comments From kazu at cs dot umass dot edu  2004-03-11 04:52 -------
Being so tiny a microcontroller, I don't think "make bootstrap" is possible.
I've heard that some AVR simulator is available, but I don't know if
it's known to work with dejagnu to run gcc testsuite.
(Well, I guess one could at least test gcc.c-torture/compile and such.)


-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |kazu at cs dot umass dot edu


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14047


^ permalink raw reply	[flat|nested] 21+ messages in thread
* [Bug target/14047] [3.4/3.5 Regression] __progmem__ attribute doesn't work
  2004-02-06 12:45 [Bug c/14047] New: __progmem__ attribute doesn't work LAL at pro dot onet dot pl
                   ` (9 preceding siblings ...)
  2004-03-11  4:52 ` kazu at cs dot umass dot edu
@ 2004-03-11  4:53 ` mark at codesourcery dot com
  2004-03-11  4:54 ` eric at ecentral dot com
                   ` (8 subsequent siblings)
  19 siblings, 0 replies; 21+ messages in thread
From: mark at codesourcery dot com @ 2004-03-11  4:53 UTC (permalink / raw)
  To: gcc-bugs


------- Additional Comments From mark at codesourcery dot com  2004-03-11 04:53 -------
Subject: Re:  [3.4/3.5 Regression] __progmem__ attribute
 doesn't work

kazu at cs dot umass dot edu wrote:

> ------- Additional Comments From kazu at cs dot umass dot edu  2004-03-11 04:52 -------
> Being so tiny a microcontroller, I don't think "make bootstrap" is possible.
> I've heard that some AVR simulator is available, but I don't know if
> it's known to work with dejagnu to run gcc testsuite.
> (Well, I guess one could at least test gcc.c-torture/compile and such.)

If people using this tareget don't normally run DejaGNU, then the patch 
can be checked in with whatever testing would normally be done.  If no 
testing is normally done, then just go ahead and commit the patch. :-)



-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14047


^ permalink raw reply	[flat|nested] 21+ messages in thread
* [Bug target/14047] [3.4/3.5 Regression] __progmem__ attribute doesn't work
  2004-02-06 12:45 [Bug c/14047] New: __progmem__ attribute doesn't work LAL at pro dot onet dot pl
                   ` (10 preceding siblings ...)
  2004-03-11  4:53 ` mark at codesourcery dot com
@ 2004-03-11  4:54 ` eric at ecentral dot com
  2004-03-12  4:13 ` eric at ecentral dot com
                   ` (7 subsequent siblings)
  19 siblings, 0 replies; 21+ messages in thread
From: eric at ecentral dot com @ 2004-03-11  4:54 UTC (permalink / raw)
  To: gcc-bugs


------- Additional Comments From eric at ecentral dot com  2004-03-11 04:54 -------
The patch only affects 2 files in the config/avr subdir. Doesn't "make 
bootstrap" only apply to building native compilers? The avr port is always 
built as a cross compiler. ... Or do you need the "make bootstrap ..." to test 
that the changes done in config/avr don't affect any of the other ports (that 
are primary targets)?

Also, a simulator is available, but it IIRC it doesn't run with dejagnu.

-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14047


^ permalink raw reply	[flat|nested] 21+ messages in thread
* [Bug target/14047] [3.4/3.5 Regression] __progmem__ attribute doesn't work
  2004-02-06 12:45 [Bug c/14047] New: __progmem__ attribute doesn't work LAL at pro dot onet dot pl
                   ` (11 preceding siblings ...)
  2004-03-11  4:54 ` eric at ecentral dot com
@ 2004-03-12  4:13 ` eric at ecentral dot com
  2004-03-13  6:24 ` eric at ecentral dot com
                   ` (6 subsequent siblings)
  19 siblings, 0 replies; 21+ messages in thread
From: eric at ecentral dot com @ 2004-03-12  4:13 UTC (permalink / raw)
  To: gcc-bugs


------- Additional Comments From eric at ecentral dot com  2004-03-12 04:13 -------
Denis and Marek:

Could one of the AVR port maintainers please commit this approved patch for 3.4 
as soon as possible, please?

Thanks
Eric Weddington

-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14047


^ permalink raw reply	[flat|nested] 21+ messages in thread
* [Bug target/14047] [3.4/3.5 Regression] __progmem__ attribute doesn't work
  2004-02-06 12:45 [Bug c/14047] New: __progmem__ attribute doesn't work LAL at pro dot onet dot pl
                   ` (12 preceding siblings ...)
  2004-03-12  4:13 ` eric at ecentral dot com
@ 2004-03-13  6:24 ` eric at ecentral dot com
  2004-03-13  6:38 ` cvs-commit at gcc dot gnu dot org
                   ` (5 subsequent siblings)
  19 siblings, 0 replies; 21+ messages in thread
From: eric at ecentral dot com @ 2004-03-13  6:24 UTC (permalink / raw)
  To: gcc-bugs


------- Additional Comments From eric at ecentral dot com  2004-03-13 06:23 -------
Would *anyone* with CVS write permissions please commit this approved patch for 
3.4?

Thanks
Eric

-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14047


^ permalink raw reply	[flat|nested] 21+ messages in thread
* [Bug target/14047] [3.4/3.5 Regression] __progmem__ attribute doesn't work
  2004-02-06 12:45 [Bug c/14047] New: __progmem__ attribute doesn't work LAL at pro dot onet dot pl
                   ` (13 preceding siblings ...)
  2004-03-13  6:24 ` eric at ecentral dot com
@ 2004-03-13  6:38 ` cvs-commit at gcc dot gnu dot org
  2004-03-13  6:39 ` denisc at overta dot ru
                   ` (4 subsequent siblings)
  19 siblings, 0 replies; 21+ messages in thread
From: cvs-commit at gcc dot gnu dot org @ 2004-03-13  6:38 UTC (permalink / raw)
  To: gcc-bugs


------- Additional Comments From cvs-commit at gcc dot gnu dot org  2004-03-13 06:38 -------
Subject: Bug 14047

CVSROOT:	/cvs/gcc
Module name:	gcc
Branch: 	gcc-3_4-branch
Changes by:	denisc@gcc.gnu.org	2004-03-13 06:38:12

Modified files:
	gcc            : ChangeLog 
	gcc/config/avr : avr.c avr-protos.h 

Log message:
	PR target/14047
	* config/avr/avr.c (avr_progmem_p): Add "attributes" parameter.
	(avr_insert_attributes): Pass "attributes" to avr_progmem_p.
	* config/avr/avr-protos.h (avr_progmem_p): Change prototype.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/ChangeLog.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=2.2326.2.336&r2=2.2326.2.337
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/config/avr/avr.c.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=1.108.4.1&r2=1.108.4.2
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/config/avr/avr-protos.h.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=1.31&r2=1.31.10.1



-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14047


^ permalink raw reply	[flat|nested] 21+ messages in thread
* [Bug target/14047] [3.4/3.5 Regression] __progmem__ attribute doesn't work
  2004-02-06 12:45 [Bug c/14047] New: __progmem__ attribute doesn't work LAL at pro dot onet dot pl
                   ` (14 preceding siblings ...)
  2004-03-13  6:38 ` cvs-commit at gcc dot gnu dot org
@ 2004-03-13  6:39 ` denisc at overta dot ru
  2004-03-13  6:43 ` cvs-commit at gcc dot gnu dot org
                   ` (3 subsequent siblings)
  19 siblings, 0 replies; 21+ messages in thread
From: denisc at overta dot ru @ 2004-03-13  6:39 UTC (permalink / raw)
  To: gcc-bugs


------- Additional Comments From denisc at overta dot ru  2004-03-13 06:39 -------
Subject: Re:  [3.4/3.5 Regression] __progmem__ attribute doesn't work

"eric at ecentral dot com" <gcc-bugzilla@gcc.gnu.org> writes:

> ------- Additional Comments From eric at ecentral dot com  2004-03-12 04:13 -------
> Denis and Marek:
> 
> Could one of the AVR port maintainers please commit this approved patch for 3.4 
> as soon as possible, please?

Committed as:

2004-03-13  Dean Ferreyra <dferreyra@igc.org>

	PR target/14047
	* config/avr/avr.c (avr_progmem_p): Add "attributes" parameter.
	(avr_insert_attributes): Pass "attributes" to avr_progmem_p.
	* config/avr/avr-protos.h (avr_progmem_p): Change prototype.

Denis.



-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14047


^ permalink raw reply	[flat|nested] 21+ messages in thread
* [Bug target/14047] [3.4/3.5 Regression] __progmem__ attribute doesn't work
  2004-02-06 12:45 [Bug c/14047] New: __progmem__ attribute doesn't work LAL at pro dot onet dot pl
                   ` (15 preceding siblings ...)
  2004-03-13  6:39 ` denisc at overta dot ru
@ 2004-03-13  6:43 ` cvs-commit at gcc dot gnu dot org
  2004-03-13  6:45 ` pinskia at gcc dot gnu dot org
                   ` (2 subsequent siblings)
  19 siblings, 0 replies; 21+ messages in thread
From: cvs-commit at gcc dot gnu dot org @ 2004-03-13  6:43 UTC (permalink / raw)
  To: gcc-bugs


------- Additional Comments From cvs-commit at gcc dot gnu dot org  2004-03-13 06:43 -------
Subject: Bug 14047

CVSROOT:	/cvs/gcc
Module name:	gcc
Changes by:	denisc@gcc.gnu.org	2004-03-13 06:43:31

Modified files:
	gcc/config/avr : avr.c avr-protos.h 

Log message:
	PR target/14047
	* config/avr/avr.c (avr_progmem_p): Add "attributes" parameter.
	(avr_insert_attributes): Pass "attributes" to avr_progmem_p.
	* config/avr/avr-protos.h (avr_progmem_p): Change prototype.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/config/avr/avr.c.diff?cvsroot=gcc&r1=1.117&r2=1.118
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/config/avr/avr-protos.h.diff?cvsroot=gcc&r1=1.31&r2=1.32



-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14047


^ permalink raw reply	[flat|nested] 21+ messages in thread
* [Bug target/14047] [3.4/3.5 Regression] __progmem__ attribute doesn't work
  2004-02-06 12:45 [Bug c/14047] New: __progmem__ attribute doesn't work LAL at pro dot onet dot pl
                   ` (16 preceding siblings ...)
  2004-03-13  6:43 ` cvs-commit at gcc dot gnu dot org
@ 2004-03-13  6:45 ` pinskia at gcc dot gnu dot org
  2004-03-13  6:52 ` eric at ecentral dot com
  2004-03-13  6:52 ` cvs-commit at gcc dot gnu dot org
  19 siblings, 0 replies; 21+ messages in thread
From: pinskia at gcc dot gnu dot org @ 2004-03-13  6:45 UTC (permalink / raw)
  To: gcc-bugs


------- Additional Comments From pinskia at gcc dot gnu dot org  2004-03-13 06:45 -------
Fixed for 3.4.0.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|3.4.1                       |3.4.0


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14047


^ permalink raw reply	[flat|nested] 21+ messages in thread
* [Bug target/14047] [3.4/3.5 Regression] __progmem__ attribute doesn't work
  2004-02-06 12:45 [Bug c/14047] New: __progmem__ attribute doesn't work LAL at pro dot onet dot pl
                   ` (18 preceding siblings ...)
  2004-03-13  6:52 ` eric at ecentral dot com
@ 2004-03-13  6:52 ` cvs-commit at gcc dot gnu dot org
  19 siblings, 0 replies; 21+ messages in thread
From: cvs-commit at gcc dot gnu dot org @ 2004-03-13  6:52 UTC (permalink / raw)
  To: gcc-bugs


------- Additional Comments From cvs-commit at gcc dot gnu dot org  2004-03-13 06:51 -------
Subject: Bug 14047

CVSROOT:	/cvs/gcc
Module name:	gcc
Changes by:	denisc@gcc.gnu.org	2004-03-13 06:51:56

Modified files:
	gcc            : ChangeLog 

Log message:
	PR target/14047
	* config/avr/avr.c (avr_progmem_p): Add "attributes" parameter.
	(avr_insert_attributes): Pass "attributes" to avr_progmem_p.
	* config/avr/avr-protos.h (avr_progmem_p): Change prototype.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/ChangeLog.diff?cvsroot=gcc&r1=2.3154&r2=2.3155



-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14047


^ permalink raw reply	[flat|nested] 21+ messages in thread
* [Bug target/14047] [3.4/3.5 Regression] __progmem__ attribute doesn't work
  2004-02-06 12:45 [Bug c/14047] New: __progmem__ attribute doesn't work LAL at pro dot onet dot pl
                   ` (17 preceding siblings ...)
  2004-03-13  6:45 ` pinskia at gcc dot gnu dot org
@ 2004-03-13  6:52 ` eric at ecentral dot com
  2004-03-13  6:52 ` cvs-commit at gcc dot gnu dot org
  19 siblings, 0 replies; 21+ messages in thread
From: eric at ecentral dot com @ 2004-03-13  6:52 UTC (permalink / raw)
  To: gcc-bugs


------- Additional Comments From eric at ecentral dot com  2004-03-13 06:52 -------
Thanks, Denis!

Eric

-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14047


^ permalink raw reply	[flat|nested] 21+ messages in thread
end of thread, other threads:[~2004-03-13  6:52 UTC | newest]

Thread overview: 21+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2004-02-06 12:45 [Bug c/14047] New: __progmem__ attribute doesn't work LAL at pro dot onet dot pl
2004-02-06 15:09 ` [Bug target/14047] " pinskia at gcc dot gnu dot org
2004-02-07  9:27 ` berndtrog at yahoo dot com
2004-02-10  7:43 ` olischulz at web dot de
2004-02-10 16:21 ` eric at ecentral dot com
2004-02-11  8:02 ` LAL at pro dot onet dot pl
2004-02-11  8:56 ` [Bug target/14047] [3.4/3.5 Regression] " pinskia at gcc dot gnu dot org
2004-03-01  2:00 ` mmitchel at gcc dot gnu dot org
2004-03-10  5:33 ` eric at ecentral dot com
2004-03-11  4:30 ` mmitchel at gcc dot gnu dot org
2004-03-11  4:52 ` kazu at cs dot umass dot edu
2004-03-11  4:53 ` mark at codesourcery dot com
2004-03-11  4:54 ` eric at ecentral dot com
2004-03-12  4:13 ` eric at ecentral dot com
2004-03-13  6:24 ` eric at ecentral dot com
2004-03-13  6:38 ` cvs-commit at gcc dot gnu dot org
2004-03-13  6:39 ` denisc at overta dot ru
2004-03-13  6:43 ` cvs-commit at gcc dot gnu dot org
2004-03-13  6:45 ` pinskia at gcc dot gnu dot org
2004-03-13  6:52 ` eric at ecentral dot com
2004-03-13  6:52 ` cvs-commit at gcc dot gnu dot org

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).