From: Sergio Durigan Junior <sergiodj@redhat.com>
To: Joel Brobecker <brobecker@adacore.com>
Cc: gdb-patches@sourceware.org
Subject: Re: [RFA/commit 1/2] Regenerate the features/i386 target description .c files
Date: Wed, 02 May 2012 06:34:00 -0000 [thread overview]
Message-ID: <m3ehr3ytve.fsf@redhat.com> (raw)
In-Reply-To: <1335913461-1628-2-git-send-email-brobecker@adacore.com> (Joel Brobecker's message of "Tue, 1 May 2012 16:04:20 -0700")
On Tuesday, May 01 2012, Joel Brobecker wrote:
> There's been a minor change in the code generator that affects
> how a couple of local variables are being declared. This patch
> regenerates the .c files in features/i386 to bring them up to
> date.
Hi Joel,
Sorry, I only read your message after committing my patch to clean more
bits in the features/ directory.
http://sourceware.org/ml/gdb-patches/2012-05/msg00010.html
Anyway, the only file my commit touched was `i386/i386-mmx-linux.c'. If
it's not too much troube for you, I think you can remove this specific
diff from the patch, and apply the rest safely.
Thanks,
> gdb/ChangeLog:
>
> * features/i386/amd64-avx-linux.c, features/i386/amd64-avx.c,
> features/i386/amd64-linux.c, features/i386/amd64.c,
> features/i386/i386-avx-linux.c, features/i386/i386-avx.c,
> features/i386/i386-linux.c, features/i386/i386-mmx-linux.c,
> features/i386/i386.c, features/i386/x32-avx-linux.c,
> features/i386/x32-avx.c, features/i386/x32-linux.c,
> features/i386/x32.c: Regenerate.
>
> Will commit in a day or two pending comments.
>
> ---
> gdb/features/i386/amd64-avx-linux.c | 3 ++-
> gdb/features/i386/amd64-avx.c | 3 ++-
> gdb/features/i386/amd64-linux.c | 3 ++-
> gdb/features/i386/amd64.c | 3 ++-
> gdb/features/i386/i386-avx-linux.c | 3 ++-
> gdb/features/i386/i386-avx.c | 3 ++-
> gdb/features/i386/i386-linux.c | 3 ++-
> gdb/features/i386/i386-mmx-linux.c | 2 +-
> gdb/features/i386/i386.c | 3 ++-
> gdb/features/i386/x32-avx-linux.c | 3 ++-
> gdb/features/i386/x32-avx.c | 3 ++-
> gdb/features/i386/x32-linux.c | 3 ++-
> gdb/features/i386/x32.c | 3 ++-
> 13 files changed, 25 insertions(+), 13 deletions(-)
>
> diff --git a/gdb/features/i386/amd64-avx-linux.c b/gdb/features/i386/amd64-avx-linux.c
> index 73392d3..81e8ca7 100644
> --- a/gdb/features/i386/amd64-avx-linux.c
> +++ b/gdb/features/i386/amd64-avx-linux.c
> @@ -10,7 +10,8 @@ initialize_tdesc_amd64_avx_linux (void)
> {
> struct target_desc *result = allocate_target_description ();
> struct tdesc_feature *feature;
> - struct tdesc_type *field_type, *type;
> + struct tdesc_type *field_type;
> + struct tdesc_type *type;
>
> set_tdesc_architecture (result, bfd_scan_arch ("i386:x86-64"));
>
> diff --git a/gdb/features/i386/amd64-avx.c b/gdb/features/i386/amd64-avx.c
> index 05c60ff..d69d427 100644
> --- a/gdb/features/i386/amd64-avx.c
> +++ b/gdb/features/i386/amd64-avx.c
> @@ -10,7 +10,8 @@ initialize_tdesc_amd64_avx (void)
> {
> struct target_desc *result = allocate_target_description ();
> struct tdesc_feature *feature;
> - struct tdesc_type *field_type, *type;
> + struct tdesc_type *field_type;
> + struct tdesc_type *type;
>
> set_tdesc_architecture (result, bfd_scan_arch ("i386:x86-64"));
>
> diff --git a/gdb/features/i386/amd64-linux.c b/gdb/features/i386/amd64-linux.c
> index 71efcbe..db97118 100644
> --- a/gdb/features/i386/amd64-linux.c
> +++ b/gdb/features/i386/amd64-linux.c
> @@ -10,7 +10,8 @@ initialize_tdesc_amd64_linux (void)
> {
> struct target_desc *result = allocate_target_description ();
> struct tdesc_feature *feature;
> - struct tdesc_type *field_type, *type;
> + struct tdesc_type *field_type;
> + struct tdesc_type *type;
>
> set_tdesc_architecture (result, bfd_scan_arch ("i386:x86-64"));
>
> diff --git a/gdb/features/i386/amd64.c b/gdb/features/i386/amd64.c
> index 154e8df..951b36d 100644
> --- a/gdb/features/i386/amd64.c
> +++ b/gdb/features/i386/amd64.c
> @@ -10,7 +10,8 @@ initialize_tdesc_amd64 (void)
> {
> struct target_desc *result = allocate_target_description ();
> struct tdesc_feature *feature;
> - struct tdesc_type *field_type, *type;
> + struct tdesc_type *field_type;
> + struct tdesc_type *type;
>
> set_tdesc_architecture (result, bfd_scan_arch ("i386:x86-64"));
>
> diff --git a/gdb/features/i386/i386-avx-linux.c b/gdb/features/i386/i386-avx-linux.c
> index 1aa939b..038073e 100644
> --- a/gdb/features/i386/i386-avx-linux.c
> +++ b/gdb/features/i386/i386-avx-linux.c
> @@ -10,7 +10,8 @@ initialize_tdesc_i386_avx_linux (void)
> {
> struct target_desc *result = allocate_target_description ();
> struct tdesc_feature *feature;
> - struct tdesc_type *field_type, *type;
> + struct tdesc_type *field_type;
> + struct tdesc_type *type;
>
> set_tdesc_architecture (result, bfd_scan_arch ("i386"));
>
> diff --git a/gdb/features/i386/i386-avx.c b/gdb/features/i386/i386-avx.c
> index 1e74ed5..cdfebf8 100644
> --- a/gdb/features/i386/i386-avx.c
> +++ b/gdb/features/i386/i386-avx.c
> @@ -10,7 +10,8 @@ initialize_tdesc_i386_avx (void)
> {
> struct target_desc *result = allocate_target_description ();
> struct tdesc_feature *feature;
> - struct tdesc_type *field_type, *type;
> + struct tdesc_type *field_type;
> + struct tdesc_type *type;
>
> set_tdesc_architecture (result, bfd_scan_arch ("i386"));
>
> diff --git a/gdb/features/i386/i386-linux.c b/gdb/features/i386/i386-linux.c
> index cc1fb99..cdc41a8 100644
> --- a/gdb/features/i386/i386-linux.c
> +++ b/gdb/features/i386/i386-linux.c
> @@ -10,7 +10,8 @@ initialize_tdesc_i386_linux (void)
> {
> struct target_desc *result = allocate_target_description ();
> struct tdesc_feature *feature;
> - struct tdesc_type *field_type, *type;
> + struct tdesc_type *field_type;
> + struct tdesc_type *type;
>
> set_tdesc_architecture (result, bfd_scan_arch ("i386"));
>
> diff --git a/gdb/features/i386/i386-mmx-linux.c b/gdb/features/i386/i386-mmx-linux.c
> index 364ade4..872826a 100644
> --- a/gdb/features/i386/i386-mmx-linux.c
> +++ b/gdb/features/i386/i386-mmx-linux.c
> @@ -10,7 +10,7 @@ initialize_tdesc_i386_mmx_linux (void)
> {
> struct target_desc *result = allocate_target_description ();
> struct tdesc_feature *feature;
> - struct tdesc_type *field_type, *type;
> + struct tdesc_type *field_type;
>
> set_tdesc_architecture (result, bfd_scan_arch ("i386"));
>
> diff --git a/gdb/features/i386/i386.c b/gdb/features/i386/i386.c
> index 7f7beb3..b1de96f 100644
> --- a/gdb/features/i386/i386.c
> +++ b/gdb/features/i386/i386.c
> @@ -10,7 +10,8 @@ initialize_tdesc_i386 (void)
> {
> struct target_desc *result = allocate_target_description ();
> struct tdesc_feature *feature;
> - struct tdesc_type *field_type, *type;
> + struct tdesc_type *field_type;
> + struct tdesc_type *type;
>
> set_tdesc_architecture (result, bfd_scan_arch ("i386"));
>
> diff --git a/gdb/features/i386/x32-avx-linux.c b/gdb/features/i386/x32-avx-linux.c
> index d2e5ce9..b01013a 100644
> --- a/gdb/features/i386/x32-avx-linux.c
> +++ b/gdb/features/i386/x32-avx-linux.c
> @@ -10,7 +10,8 @@ initialize_tdesc_x32_avx_linux (void)
> {
> struct target_desc *result = allocate_target_description ();
> struct tdesc_feature *feature;
> - struct tdesc_type *field_type, *type;
> + struct tdesc_type *field_type;
> + struct tdesc_type *type;
>
> set_tdesc_architecture (result, bfd_scan_arch ("i386:x64-32"));
>
> diff --git a/gdb/features/i386/x32-avx.c b/gdb/features/i386/x32-avx.c
> index 47d7305..af235cd 100644
> --- a/gdb/features/i386/x32-avx.c
> +++ b/gdb/features/i386/x32-avx.c
> @@ -10,7 +10,8 @@ initialize_tdesc_x32_avx (void)
> {
> struct target_desc *result = allocate_target_description ();
> struct tdesc_feature *feature;
> - struct tdesc_type *field_type, *type;
> + struct tdesc_type *field_type;
> + struct tdesc_type *type;
>
> set_tdesc_architecture (result, bfd_scan_arch ("i386:x64-32"));
>
> diff --git a/gdb/features/i386/x32-linux.c b/gdb/features/i386/x32-linux.c
> index b42287d..d34a2a9 100644
> --- a/gdb/features/i386/x32-linux.c
> +++ b/gdb/features/i386/x32-linux.c
> @@ -10,7 +10,8 @@ initialize_tdesc_x32_linux (void)
> {
> struct target_desc *result = allocate_target_description ();
> struct tdesc_feature *feature;
> - struct tdesc_type *field_type, *type;
> + struct tdesc_type *field_type;
> + struct tdesc_type *type;
>
> set_tdesc_architecture (result, bfd_scan_arch ("i386:x64-32"));
>
> diff --git a/gdb/features/i386/x32.c b/gdb/features/i386/x32.c
> index 6342ddd..755ec69 100644
> --- a/gdb/features/i386/x32.c
> +++ b/gdb/features/i386/x32.c
> @@ -10,7 +10,8 @@ initialize_tdesc_x32 (void)
> {
> struct target_desc *result = allocate_target_description ();
> struct tdesc_feature *feature;
> - struct tdesc_type *field_type, *type;
> + struct tdesc_type *field_type;
> + struct tdesc_type *type;
>
> set_tdesc_architecture (result, bfd_scan_arch ("i386:x64-32"));
>
> --
> 1.7.1
--
Sergio
next prev parent reply other threads:[~2012-05-02 6:34 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-05-01 23:04 Invalid segment resister value on x86_64-windows Joel Brobecker
2012-05-01 23:04 ` [RFA/commit 1/2] Regenerate the features/i386 target description .c files Joel Brobecker
2012-05-02 6:34 ` Sergio Durigan Junior [this message]
2012-05-01 23:05 ` [RFA 2/2] [x86/x86_64] Segment registers are 16 bits long (not 32bits) Joel Brobecker
2012-05-02 0:01 ` [RFA 3/2(+)] Test size of x86/x86_64 segment registers Joel Brobecker
2012-05-02 10:10 ` Invalid segment resister value on x86_64-windows Mark Kettenis
2012-05-02 17:57 ` Joel Brobecker
2012-05-02 20:45 ` Mark Kettenis
2012-05-02 21:26 ` Joel Brobecker
2012-05-02 21:27 ` Joel Brobecker
2012-05-02 21:50 ` Mark Kettenis
2012-05-02 21:58 ` [WINDOWS/RFC] " Joel Brobecker
2012-05-02 22:10 ` Christopher Faylor
2012-05-02 22:16 ` Christopher Faylor
2012-05-04 18:38 ` checked in: " Joel Brobecker
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=m3ehr3ytve.fsf@redhat.com \
--to=sergiodj@redhat.com \
--cc=brobecker@adacore.com \
--cc=gdb-patches@sourceware.org \
/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