> -----Original Message----- > From: Georg-Johann Lay > Sent: Monday, January 14, 2013 11:15 AM > To: Weddington, Eric > Cc: gcc-patches@gcc.gnu.org; Denis Chertykov > Subject: Re: [patch,avr]: Minor fixes to some built-in macros > > Weddington, Eric wrote: > >> > >> This patch does: > >> > >> 1) With 8-bit int, __INT24_MAX__ and __UINT24_MAX__ need a long long > >> suffix. > >> > >> 2) Defines like __FLASH that indicate if address space is available > should > >> be like a predicate, i.e. 1 if the space is available. > > > > I'm not seeing #2 being done in the patch... Am I missing something? > > The current "#define __FLASH=__flash" is accomplished by the > cpp_define_formatted with "%s=%s". The new is just a cpp_define for > __FLASH, > retrieved by avr_toupper.
Thanks for the explanation. Please commit. Eric
