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).