[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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

From: Anatoly Sokolov
Subject: Re: [avr-gcc-list] Add builtins in avr target.
Date: Sun, 6 Apr 2008 18:01:06 +0400


My patch adds avr specific builtins, and has no attitude to GCC standard 

> 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.

 As far as I can tell, __builtin_return_address is used entirely for
tracing, profiling, and error reporting. And practically it is not useful for 
target. More, this function returns the pointer which is 16-bits on AVR 
target, hence its value for devices with more than 64KB FLASH will be incorrect.

The only practical advantage of this patch to remove 2-3 FALL in GCC testsuite. 
To make truly a working '__builtin_return_address' now it is not possible.

Easier to document '__builtin_return_address' as not supported feature and 
this testcases in GGG testsuite. And to not complicate AVR backend.


reply via email to

[Prev in Thread] Current Thread [Next in Thread]