public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug target/55897] New: [avr] Allocate __memx data to .progmemx.data
@ 2013-01-07 14:19 gjl at gcc dot gnu.org
2013-01-07 18:35 ` [Bug target/55897] " gjl at gcc dot gnu.org
` (3 more replies)
0 siblings, 4 replies; 5+ messages in thread
From: gjl at gcc dot gnu.org @ 2013-01-07 14:19 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55897
Bug #: 55897
Summary: [avr] Allocate __memx data to .progmemx.data
Classification: Unclassified
Product: gcc
Version: 4.8.0
Status: UNCONFIRMED
Severity: normal
Priority: P3
Component: target
AssignedTo: unassigned@gcc.gnu.org
ReportedBy: gjl@gcc.gnu.org
Target: avr
Currently, data in address space __memx is allocated to .progmem.data, i.e. the
same section as for __flash resp. progmem. However, there os no need that
__memx goes onto the same section.
In order to support better data layout, __memx should go in a different section
than .progmem, e.g. progmemx.
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Bug target/55897] [avr] Allocate __memx data to .progmemx.data
2013-01-07 14:19 [Bug target/55897] New: [avr] Allocate __memx data to .progmemx.data gjl at gcc dot gnu.org
@ 2013-01-07 18:35 ` gjl at gcc dot gnu.org
2013-01-07 18:38 ` gjl at gcc dot gnu.org
` (2 subsequent siblings)
3 siblings, 0 replies; 5+ messages in thread
From: gjl at gcc dot gnu.org @ 2013-01-07 18:35 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55897
--- Comment #1 from Georg-Johann Lay <gjl at gcc dot gnu.org> 2013-01-07 18:34:50 UTC ---
Author: gjl
Date: Mon Jan 7 18:34:40 2013
New Revision: 194991
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=194991
Log:
PR target/55897
* config/avr/avr.h (ADDR_SPACE_COUNT): New enum.
(avr_addrspace_t): Add .section_name field.
* config/avr/avr.c (progmem_section): Use ADDR_SPACE_COUNT as
array size.
(avr_addrspace): Same. Initialize .section_name. Remove last
NULL entry. Put __memx into .progmemx.data.
(progmem_section_prefix): Remove.
(avr_asm_init_sections): No need to initialize progmem_section.
(avr_asm_named_section): Use avr_addrspace[].section_name to get
section name prefix.
(avr_asm_select_section): Ditto. And use get_unnamed_section to
retrieve the progmem section.
* avr-c.c (avr_cpu_cpp_builtins): Use ADDR_SPACE_COUNT as loop
boundary to run over avr_addrspace[].
(avr_register_target_pragmas): Ditto.
Modified:
trunk/gcc/ChangeLog
trunk/gcc/config/avr/avr-c.c
trunk/gcc/config/avr/avr.c
trunk/gcc/config/avr/avr.h
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Bug target/55897] [avr] Allocate __memx data to .progmemx.data
2013-01-07 14:19 [Bug target/55897] New: [avr] Allocate __memx data to .progmemx.data gjl at gcc dot gnu.org
2013-01-07 18:35 ` [Bug target/55897] " gjl at gcc dot gnu.org
@ 2013-01-07 18:38 ` gjl at gcc dot gnu.org
2013-01-07 18:51 ` gjl at gcc dot gnu.org
2013-01-07 18:54 ` gjl at gcc dot gnu.org
3 siblings, 0 replies; 5+ messages in thread
From: gjl at gcc dot gnu.org @ 2013-01-07 18:38 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55897
--- Comment #2 from Georg-Johann Lay <gjl at gcc dot gnu.org> 2013-01-07 18:37:57 UTC ---
Author: gjl
Date: Mon Jan 7 18:37:47 2013
New Revision: 194992
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=194992
Log:
PR target/55897
* doc/extend.texi (AVR Named Address Spaces): __memx goes into
.progmemx.data now.
Modified:
trunk/gcc/ChangeLog
trunk/gcc/doc/extend.texi
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Bug target/55897] [avr] Allocate __memx data to .progmemx.data
2013-01-07 14:19 [Bug target/55897] New: [avr] Allocate __memx data to .progmemx.data gjl at gcc dot gnu.org
2013-01-07 18:35 ` [Bug target/55897] " gjl at gcc dot gnu.org
2013-01-07 18:38 ` gjl at gcc dot gnu.org
@ 2013-01-07 18:51 ` gjl at gcc dot gnu.org
2013-01-07 18:54 ` gjl at gcc dot gnu.org
3 siblings, 0 replies; 5+ messages in thread
From: gjl at gcc dot gnu.org @ 2013-01-07 18:51 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55897
--- Comment #3 from Georg-Johann Lay <gjl at gcc dot gnu.org> 2013-01-07 18:51:38 UTC ---
Author: gjl
Date: Mon Jan 7 18:51:33 2013
New Revision: 194993
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=194993
Log:
Backport from 2013-01-07 trunk r194991, r194992.
PR target/55897
* doc/extend.texi (AVR Named Address Spaces): __memx goes into
.progmemx.data now.
* config/avr/avr.h (ADDR_SPACE_COUNT): New enum.
(avr_addrspace_t): Add .section_name field.
* config/avr/avr.c (progmem_section): Use ADDR_SPACE_COUNT as
array size.
(avr_addrspace): Same. Initialize .section_name. Remove last
NULL entry. Put __memx into .progmemx.data.
(progmem_section_prefix): Remove.
(avr_asm_init_sections): No need to initialize progmem_section.
(avr_asm_named_section): Use avr_addrspace[].section_name to get
section name prefix.
(avr_asm_select_section): Ditto. And use get_unnamed_section to
retrieve the progmem section.
* avr-c.c (avr_cpu_cpp_builtins): Use ADDR_SPACE_COUNT as loop
boundary to run over avr_addrspace[].
(avr_register_target_pragmas): Ditto.
Modified:
branches/gcc-4_7-branch/gcc/ChangeLog
branches/gcc-4_7-branch/gcc/config/avr/avr-c.c
branches/gcc-4_7-branch/gcc/config/avr/avr.c
branches/gcc-4_7-branch/gcc/config/avr/avr.h
branches/gcc-4_7-branch/gcc/doc/extend.texi
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Bug target/55897] [avr] Allocate __memx data to .progmemx.data
2013-01-07 14:19 [Bug target/55897] New: [avr] Allocate __memx data to .progmemx.data gjl at gcc dot gnu.org
` (2 preceding siblings ...)
2013-01-07 18:51 ` gjl at gcc dot gnu.org
@ 2013-01-07 18:54 ` gjl at gcc dot gnu.org
3 siblings, 0 replies; 5+ messages in thread
From: gjl at gcc dot gnu.org @ 2013-01-07 18:54 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55897
Georg-Johann Lay <gjl at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Keywords| |addr-space
Priority|P3 |P4
Status|UNCONFIRMED |RESOLVED
Resolution| |FIXED
Target Milestone|--- |4.7.3
--- Comment #4 from Georg-Johann Lay <gjl at gcc dot gnu.org> 2013-01-07 18:53:45 UTC ---
Fixed in 4.7.3
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2013-01-07 18:54 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-01-07 14:19 [Bug target/55897] New: [avr] Allocate __memx data to .progmemx.data gjl at gcc dot gnu.org
2013-01-07 18:35 ` [Bug target/55897] " gjl at gcc dot gnu.org
2013-01-07 18:38 ` gjl at gcc dot gnu.org
2013-01-07 18:51 ` gjl at gcc dot gnu.org
2013-01-07 18:54 ` gjl at gcc dot gnu.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).