This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
RE: [Patch,AVR]: Rename address spaces
- From: "Weddington, Eric" <Eric dot Weddington at atmel dot com>
- To: "Georg-Johann Lay" <avr at gjlay dot de>, <gcc-patches at gcc dot gnu dot org>
- Cc: "Denis Chertykov" <chertykov at gmail dot com>, "Joerg Wunsch" <joerg_wunsch at uriah dot heep dot sax dot de>
- Date: Wed, 25 Jan 2012 10:13:34 -0700
- Subject: RE: [Patch,AVR]: Rename address spaces
- References: <4F1FEF3E.8030404@gjlay.de>
> -----Original Message-----
> From: Georg-Johann Lay [mailto:avr@gjlay.de]
> Sent: Wednesday, January 25, 2012 4:02 AM
> To: gcc-patches@gcc.gnu.org
> Cc: Denis Chertykov; Weddington, Eric; Joerg Wunsch
> Subject: [Patch,AVR]: Rename address spaces
>
> This patch renames the named address spaces from their initial draft
names
> to
> their final names as proposed by several developers, amongst them
Eric.
>
> * Using __flash will make porting to/from other compilers easier.
> * "__flash" is more descriptive than the meaningless "__pgm".
> * __pgm might lead to confusion with the pgm_ macros from avr-libc.
>
> The 24-bit address space is named differently: It's renamed to __memx
to
> point
> out its difference to the other spaces that only address flash.
>
> The patch not only renames the address spaces (which would just change
the
> 7
> strings in avr.c:avr_addrspace) but also renames some identifiers,
> function
> names, enums etc. so that they are kept in sync with the new names is
most
> places.
>
> Ok for trunk?
Please commit, and thanks for renaming the spaces. :-)
Eric Weddington