From: Yann E. MORIN <yann.morin.1998@free.fr>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 1/1] package/gdb: add patch for fixing build gdb 9.2 with gcc 10 on ppc arch
Date: Sun, 27 Sep 2020 09:11:23 +0200 [thread overview]
Message-ID: <20200927071123.GF11621@scaer> (raw)
In-Reply-To: <20200911112240.3222531-1-fido_max@inbox.ru>
Maxim, All,
On 2020-09-11 14:22 +0300, Maxim Kochetkov spake thusly:
> Signed-off-by: Maxim Kochetkov <fido_max@inbox.ru>
Applied to master (I justslightly tweaaked the commit title for purely
personal tastes). Thanks!
Regards,
Yann E. MORIN.
> ---
> ...ppc-Fix-linker-error-with-fno-common.patch | 101 ++++++++++++++++++
> 1 file changed, 101 insertions(+)
> create mode 100644 package/gdb/9.2/0006-sim-ppc-Fix-linker-error-with-fno-common.patch
>
> diff --git a/package/gdb/9.2/0006-sim-ppc-Fix-linker-error-with-fno-common.patch b/package/gdb/9.2/0006-sim-ppc-Fix-linker-error-with-fno-common.patch
> new file mode 100644
> index 0000000000..813a1054a5
> --- /dev/null
> +++ b/package/gdb/9.2/0006-sim-ppc-Fix-linker-error-with-fno-common.patch
> @@ -0,0 +1,101 @@
> +From ad8464f799a4c96c7ab8bdfec3f95846cf54f9b0 Mon Sep 17 00:00:00 2001
> +From: Sebastian Huber <sebastian.huber@embedded-brains.de>
> +Date: Wed, 1 Jul 2020 19:29:55 +0200
> +Subject: [PATCH] sim/ppc: Fix linker error with -fno-common
> +
> +GCC 10 enables -fno-common by default. This resulted in a multiple
> +definition linker error since global variables were declared and defined
> +in a header file:
> +
> + ld: ld-insn.o:sim/ppc/ld-insn.h:221: multiple definition of
> + `max_model_fields_len'; igen.o:sim/ppc/ld-insn.h:221: first defined here
> +
> +sim/ppc
> +
> + * ld-insn.h (last_model, last_model_data, last_model_function,
> + last_model_internal, last_model_macro, last_model_static):
> + Delete.
> + (max_model_fields_len, model_data, model_functions,
> + model_internal, model_macros, model_static, models): Declare, but do not
> + define.
> + * ld-insn.c (last_model, last_model_data, last_model_function,
> + last_model_internal, last_model_macro, last_model_static,
> + max_model_fields_len, model_data, model_functions,
> + model_internal, model_macros, model_static, models): Define.
> +
> +Signed-off-by: Maxim Kochetkov <fido_max@inbox.ru>
> +Fetch from: https://sourceware.org/git/?p=binutils-gdb.git;a=commitdiff;h=ad8464f799a4c96c7ab8bdfec3f95846cf54f9b0
> +---
> + sim/ppc/ld-insn.c | 18 ++++++++++++++++++
> + sim/ppc/ld-insn.h | 24 +++++++-----------------
> + 2 files changed, 25 insertions(+), 17 deletions(-)
> +
> +diff --git a/sim/ppc/ld-insn.c b/sim/ppc/ld-insn.c
> +index e39131ca133..585071a861f 100644
> +--- a/sim/ppc/ld-insn.c
> ++++ b/sim/ppc/ld-insn.c
> +@@ -28,6 +28,24 @@
> +
> + #include "igen.h"
> +
> ++static model *last_model;
> ++
> ++static insn *last_model_macro;
> ++static insn *last_model_function;
> ++static insn *last_model_internal;
> ++static insn *last_model_static;
> ++static insn *last_model_data;
> ++
> ++model *models;
> ++
> ++insn *model_macros;
> ++insn *model_functions;
> ++insn *model_internal;
> ++insn *model_static;
> ++insn *model_data;
> ++
> ++int max_model_fields_len;
> ++
> + static void
> + update_depth(insn_table *entry,
> + lf *file,
> +diff --git a/sim/ppc/ld-insn.h b/sim/ppc/ld-insn.h
> +index 88318ffa2b3..52baeaa2d84 100644
> +--- a/sim/ppc/ld-insn.h
> ++++ b/sim/ppc/ld-insn.h
> +@@ -200,25 +200,15 @@ extern insn_table *load_insn_table
> + table_include *includes,
> + cache_table **cache_rules);
> +
> +-model *models;
> +-model *last_model;
> ++extern model *models;
> +
> +-insn *model_macros;
> +-insn *last_model_macro;
> ++extern insn *model_macros;
> ++extern insn *model_functions;
> ++extern insn *model_internal;
> ++extern insn *model_static;
> ++extern insn *model_data;
> +
> +-insn *model_functions;
> +-insn *last_model_function;
> +-
> +-insn *model_internal;
> +-insn *last_model_internal;
> +-
> +-insn *model_static;
> +-insn *last_model_static;
> +-
> +-insn *model_data;
> +-insn *last_model_data;
> +-
> +-int max_model_fields_len;
> ++extern int max_model_fields_len;
> +
> + extern void insn_table_insert_insn
> + (insn_table *table,
> +--
> +2.18.4
> +
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
prev parent reply other threads:[~2020-09-27 7:11 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-09-11 11:22 [Buildroot] [PATCH 1/1] package/gdb: add patch for fixing build gdb 9.2 with gcc 10 on ppc arch Maxim Kochetkov
2020-09-27 7:11 ` Yann E. MORIN [this message]
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=20200927071123.GF11621@scaer \
--to=yann.morin.1998@free.fr \
--cc=buildroot@busybox.net \
/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