This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [patch] Support bionic's jmp_buf.
- From: Thiago Jung Bauermann <thiago dot bauermann at linaro dot org>
- To: Yao Qi <yao at codesourcery dot com>
- Cc: gdb-patches at sourceware dot org, Pedro Alves <palves at redhat dot com>
- Date: Mon, 25 Jun 2012 14:54:33 -0300
- Subject: Re: [patch] Support bionic's jmp_buf.
- References: <1337560528.4363.13.camel@hactar> <1337623164.22073.9.camel@hactar> <1340574375.31308.7.camel@hactar> <2077698.dhnsbKdz32@qiyao.dyndns.org>
Hello,
On Mon, 2012-06-25 at 21:32 +0800, Yao Qi wrote:
> On Sunday, June 24, 2012 06:46:15 PM Thiago Jung Bauermann wrote:
> > +/* Determines whether the inferior is an Android application. */
> > +
> > +static int
> > +is_target_linux_android (void)
>
> Looks this function is not arm-specific, so we can move it to linux-tdep.c.
Thanks for your review.
Android is officially supported on ARM and x86. I don't know how GDB
fares on Android on x86, and at least for now it will only be used on
ARM (I don't have plans to work on Android on x86).
With that said, I'm happy to move to linux-tdep.c if that makes more
sense. I'll do the change before I commit or on the next iteration of
the patch if there is one.
--
[]'s
Thiago Jung Bauermann
Linaro Toolchain Working Group