> -----Original Message----- > From: > [EMAIL PROTECTED] > [mailto:[EMAIL PROTECTED] > org] On Behalf Of Eric Weddington > Sent: Monday, August 28, 2006 2:26 PM > To: 'Joerg Wunsch'; [email protected]; 'Bernd Trog' > Subject: RE: [avr-libc-dev] Re: [avr-libc-commit] > avr-libcChangeLoginclude/avr/interrupt.h incl... > > > > > -----Original Message----- > > From: > > [EMAIL PROTECTED] > > [mailto:[EMAIL PROTECTED] > > org] On Behalf Of Joerg Wunsch > > Sent: Monday, August 28, 2006 1:51 PM > > To: [email protected] > > Subject: Re: [avr-libc-dev] Re: [avr-libc-commit] avr-libc > > ChangeLoginclude/avr/interrupt.h incl... > > > > As Bernd Trog wrote: > > > > > > * include/avr/interrupt.h: Add the > > > > 'externally_visible' attribute on all interrupt > > > > service routine macros. > > > > > Note that 'externally_visible' is a new attribute. Introduced with > > > 4.1.0, unknown to versions <4.1.0. > > > > Thanks for the heads-up. That means we have to encapsulate that > > using some preprocessor magic like: > > > > #if __GNUC__ >= 4 > > # define __INTR_ATTRS used, externally_visible > > #else /* GCC < 4.x */ > > # define __INTR_ATTRS used > > #endif > > > > ... > > #define ISR(vector) \ > > void vector (void) __attribute__ ((signal, __INTR_ATTRS)); \ > > void vector (void) > > > > etc. (Also needed in <compat/deprecated.h> and in the > documentation.) > > Do we really need that? Or will previous versions just > silently ignore this? > > I'm going to be finding out soon...
Bah. Previous versions of gcc (3.4.6) gives a warning about an unused attribute. I'll take care of it. Eric _______________________________________________ AVR-libc-dev mailing list [email protected] http://lists.nongnu.org/mailman/listinfo/avr-libc-dev
