Mirror of the gdb mailing list
 help / color / mirror / Atom feed
From: Nick Clifton via Gdb <gdb@sourceware.org>
To: "Richard Earnshaw (lists)" <Richard.Earnshaw@arm.com>,
	"gdb@sourceware.org" <gdb@sourceware.org>
Subject: Re: RFC: Deprecate the ARM simulator
Date: Tue, 24 Sep 2024 15:18:14 +0100	[thread overview]
Message-ID: <d5a86d08-cc0f-480d-bb7d-72e8677d169d@redhat.com> (raw)
In-Reply-To: <7f447b75-b472-477f-9ab2-e0b0f17d03f3@arm.com>

Hi Richard,

>>    I would like to deprecate or even delete the ARM simulator.

> Won't this break builds for arm users unless they now explicitly specify --disable-sim?  That's a bit aggressive, IMO, and might cause problems for multi-target builds that 
> want some sims, if available.  We don't want to make life harder for those just trying to build gdb.

I am totally OK with just deleting the arm sim altogether.
I was just not sure if there was a GDB approved method for doing so.

So instead of the deprecating patch, how about this one to add the sim
directory to the top level configure script's noconfigdirs list for ARM
targets:

(Obviously a full patch would include the regenerated configure file
file.  I am just posting this patch for discussion purposes).

---------------------------------------------------------------------
diff --git a/configure.ac b/configure.ac
index 6fd9e77f20a..7d99cdda8fa 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1003,6 +1003,13 @@ case "${target}" in
      ;;
  esac

+# The ARM simulator has been deprecated for all ARM targets.
+case "${target}" in
+  arm*-*-*)
+    noconfigdirs="$noconfigdirs sim"
+    ;;
+esac
+
  case "${target}" in
    *-*-chorusos)
      ;;
@@ -1014,7 +1021,7 @@ case "${target}" in
      ;;
    arm-*-darwin*)
      noconfigdirs="$noconfigdirs ld gas gdb gprof"
-    noconfigdirs="$noconfigdirs sim target-rda"
+    noconfigdirs="$noconfigdirs target-rda"
      ;;
    powerpc-*-darwin*)
      noconfigdirs="$noconfigdirs ld gas gdb gprof"
--------------------------------------------------------------

I am also assuming that this patch, or something similar, should go in first
before any simulator sources are actually deleted...

Cheers
   Nick



      parent reply	other threads:[~2024-09-24 14:19 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-09-23 13:31 Nick Clifton via Gdb
2024-09-23 16:17 ` Simon Marchi via Gdb
2024-09-23 16:34   ` Nick Clifton via Gdb
2024-09-23 16:45 ` Richard Earnshaw (lists) via Gdb
2024-09-24 11:03   ` Maciej W. Rozycki
2024-09-24 14:18   ` Nick Clifton via Gdb [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=d5a86d08-cc0f-480d-bb7d-72e8677d169d@redhat.com \
    --to=gdb@sourceware.org \
    --cc=Richard.Earnshaw@arm.com \
    --cc=nickc@redhat.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