From: Pedro Alves <palves@redhat.com>
To: Jeff Kenton <jkenton@tilera.com>
Cc: Joel Brobecker <brobecker@adacore.com>, gdb-patches@sourceware.org
Subject: Re: [PATCH] Add support for Tilera TILE-Gx processor (part 1/2: gdb)
Date: Tue, 24 Apr 2012 16:22:00 -0000 [thread overview]
Message-ID: <4F96D15B.9090506@redhat.com> (raw)
In-Reply-To: <4F95B46D.1070101@tilera.com>
On 04/23/2012 08:58 PM, Jeff Kenton wrote:
> diff -r -u -N /home/packages/gdb-7.4x/gdb/config/tilegx/tm-linux.h ./gdb/config/tilegx/tm-linux.h
> --- /home/packages/gdb-7.4x/gdb/config/tilegx/tm-linux.h 1969-12-31 19:00:00.000000000 -0500
> +++ ./gdb/config/tilegx/tm-linux.h 2012-04-23 13:01:20.342909000 -0400
> @@ -0,0 +1,29 @@
> +/* Target-dependent definitions for GNU/Linux TILE.
> +
> + Copyright (C) 2012 Free Software Foundation, Inc.
> +
> + This file is part of GDB.
> +
> + This program is free software; you can redistribute it and/or modify
> + it under the terms of the GNU General Public License as published by
> + the Free Software Foundation; either version 3 of the License, or
> + (at your option) any later version.
> +
> + This program is distributed in the hope that it will be useful,
> + but WITHOUT ANY WARRANTY; without even the implied warranty of
> + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> + GNU General Public License for more details.
> +
> + You should have received a copy of the GNU General Public License
> + along with this program. If not, see <http://www.gnu.org/licenses/>. */
> +
> +#ifndef TM_TILELINUX_H
> +#define TM_TILELINUX_H
> +
> +/* Pull in GNU/Linux generic defs. */
> +#include "config/tm-linux.h"
> +
> +/* Integer capable of holding an instruction bundle. */
> +typedef unsigned long long t_bundle;
> +
> +#endif /* TM_TILELINUX_H */
tm files are long gone from GDB's sources. This whole file is dead -- note
config/tm-linux.h doesn't exist.
Could you please split the series further one more time (and resubmit
as a new thread, not as a reply to this one)? I'd like to see the host-independent
tdep bits split out from the linux native debugger support (the nat bits). So you'll
end up with 3 patches (you should be able to debug against gdbserver with just
the tdep bits, and the gdbserver bits, for example.). That'll make it easier to see
such issues and what else might need adjustment to current GDB's way of doing things.
--
Pedro Alves
next prev parent reply other threads:[~2012-04-24 16:14 UTC|newest]
Thread overview: 33+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-04-19 19:26 Jeff Kenton
2012-04-20 1:58 ` Yao Qi
2012-04-20 12:54 ` Joel Brobecker
2012-04-20 14:03 ` Jeff Kenton
2012-04-21 0:34 ` Joel Brobecker
2012-04-23 19:59 ` Jeff Kenton
2012-04-24 16:07 ` Pedro Alves
2012-04-24 16:07 ` Pedro Alves
2012-04-24 16:07 ` Pedro Alves
2012-04-24 16:07 ` Pedro Alves
2012-04-24 16:08 ` Pedro Alves
2012-04-24 16:11 ` Pedro Alves
2012-04-24 16:14 ` Pedro Alves
2012-04-24 16:22 ` Pedro Alves [this message]
2012-04-24 18:47 ` Jeff Kenton
2012-04-24 20:21 ` Jeff Kenton
2012-04-24 22:40 ` Pedro Alves
2012-04-24 16:23 ` Pedro Alves
2012-04-24 16:27 ` Pedro Alves
2012-04-24 16:38 ` Pedro Alves
2012-04-24 16:40 ` Pedro Alves
2012-04-24 16:41 ` Pedro Alves
2012-04-24 16:41 ` Pedro Alves
2012-04-24 16:59 ` Jeff Kenton
2012-04-24 16:42 ` Pedro Alves
2012-04-24 16:54 ` Pedro Alves
2012-04-25 2:18 ` Yao Qi
2012-04-23 20:12 ` [PATCH] Add support for Tilera TILE-Gx processor (part 2/2: gdbserver) Jeff Kenton
2012-04-24 16:54 ` Pedro Alves
2012-04-24 16:55 ` Pedro Alves
2012-04-24 17:34 ` Pedro Alves
2012-05-03 14:28 ` Jeff Kenton
2012-05-03 14:44 ` Pedro Alves
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4F96D15B.9090506@redhat.com \
--to=palves@redhat.com \
--cc=brobecker@adacore.com \
--cc=gdb-patches@sourceware.org \
--cc=jkenton@tilera.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox