This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [patch] disable AVR libiberty, libstdcxx, libgcj


Marek Michalkiewicz wrote:-

> Neil Booth wrote:
> > Marek Michalkiewicz wrote:-
> > 
> > > > http://gcc.gnu.org/ml/gcc-patches/2002-05/msg01583.html
> > > 
> > > Thanks for the patch - looks fine, but I'll try to simplify it a little
> > > more: one enum type instead of two (enum avr_arch, enum avr_arch_t) for
> > > the same purpose.  I'll test it and apply it tomorrow.
> > 
> > Great, thanks.
> 
> Now that this is applied (in slightly modified form) - I have a question:
> what exactly is builtin_define_std ("AVR") supposed to define?  It seems
> it defines AVR, _AVR and _AVR__ as 1, but no __AVR__ (note two leading
> underscores), which was defined in the past as well.  Is this intentional
> (then the AVR port needs to define __AVR__ too), or a bug?  Comments in
> c-common.c (builtin_define_std) indicate the latter ("if it was in user's
> namespace... Define the macro with leading and following __.").

If you refresh c-common.c, this should be fixed.

Neil.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]