On Sun, Jul 28, 2013 at 3:53 AM, Venkataramanan Kumar
<venkataramanan.ku...@linaro.org> wrote:
> Hi Maintainers,
>
> This patch adds supports to handle return address via. frame pointer.

I noticed this patch causes undefined behavior when
-fomit-frame-pointer and __builtin_return_address(1) is used.  On
PowerPC it is defined correctly that is it generates a frame for that
case.  Now on x86_64 it might be undefined but I think that is just
wrong since __builtin_return_address is just used for debugging
anyways.

Thanks,
Andrew

>
> gcc/ChangeLog
> ---------------------
>
> 2013-07-28  Venkataramanan Kumar  <venkataramanan.ku...@linaro.org>
>
>        *  config/aarch64/aarch64.c (aarch64_return_addr): Handle returning
>           address from a frame.
>
>
> Regression tested with aarch64-none-elf with V8 foundation model.
>
> regards,
> Venkat.

Reply via email to