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:
[EMAIL PROTECTED]
[mailto:[EMAIL PROTECTED]
org] On Behalf Of Anatoly Sokolov
Sent: Friday, April 04, 2008 2:51 PM
To: avr-gcc-list@nongnu.org
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
_______________________________________________
AVR-GCC-list mailing list
AVR-GCC-list@nongnu.org
http://lists.nongnu.org/mailman/listinfo/avr-gcc-list
