Re: [avr-gcc-list] Add builtins in avr target.

[Andrew Hutchinson]

Unless the builtins expose more information to gcc, I cannot see any 
benefit from the addins - with the possible exception of FMUL.

The result will be the same. No better code, no fewer bugs.

Yes there are lots of outstanding patches that fix real bugs. It would 
seem to me that these are more critical.


Andy


Weddington, Eric wrote:
>  
>
>   
> > -----Original Message-----
> > From: 
> > address@hidden 
> > [mailto:address@hidden
> > org] On Behalf Of Anatoly Sokolov
> > Sent: Friday, April 04, 2008 2:51 PM
> > To: address@hidden
> > Subject: [avr-gcc-list] Add builtins in avr target.
> >
> > Hello.
> >
> >
> >   This patch add '__builtin_avr_swap', '__builtin_avr_sei' and
> > '__builtin_avr_cli' builtins in avr backend. It will allow to 
> > use 'SWAP',
> > 'SEI' and 'CLI' instructions, without use inline assembly.
> >
> >   I wish to add:
> >     1. builtins for 'SLEEP' and 'WDR' instructions;
> >     2. builtins for 'FMUL*' instructions;
> >     3. builtin similarly to IAR '__delay_cycles';
> >     4. builtin for 'DES' xmega instruction.
> >
> >   Your suggestions?
> >
> >     
>
> See GCC bug #21080:
> <http://gcc.gnu.org/bugzilla/show_bug.cgi?id=21080>
>
> Andy Hutchinson wrote a patch (attached to bug report) to fix this bug
> which adds support for __builtin_return_address.
>
> BTW, Andy has a *lot* of outstanding patches that needs review and
> possible commit to AVR port. He has been fixing a lot of bugs. I have a
> list if you need it.
>
> Eric Weddington
>
>