public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
From: Richard Earnshaw <rearnsha@gcc.gnu.org>
To: Mark Mitchell <mark@codesourcery.com>
Cc: Daniel Jacobowitz <drow@false.org>,
	Nick Clifton <nickc@redhat.com>,
	Ian Lance Taylor <ian@wasabisystems.com>,
	binutils@sources.redhat.com
Subject: Obsoleting elfarm-oabi [was Re: PATCH: Reduce size of SymbianOS DLLs]
Date: Wed, 03 Nov 2004 15:05:00 -0000	[thread overview]
Message-ID: <1099494297.6939.106.camel@pc960.cambridge.arm.com> (raw)
In-Reply-To: <4187B8FC.7070407@codesourcery.com>

On Tue, 2004-11-02 at 16:42, Mark Mitchell wrote:
> OK.  In that case, I'd say we could just remove the code, then.  What's 
> the point in pretending we might keep it, if we've already made up our 
> minds?

Let's try and bring this to a rational conclusion.

We have three possible courses of action

1.  Obsolete elfarm-oabi.c, but change nothing for now

2.  Obsolete elfarm-oabi.c and split the code common with the current
ABI into separate files.

3.  Skip obsoleting elfarm-oabi.c and proceed straight to deletion.

I've done some archaeology, and it turns out that the old abi was named
so in February 1999, ie 5+ years ago.  That appears to correspond with
the binutils 2.10 release (we're now at 2.15).  The old ABI was only
ever added as a stop-gap because at the time it was first created ARM
hadn't published an ELF specification for ARM.

Daniel asserts that it isn't really possible to test the oabi code
because the assembler always defaults to the 'new' ABI (even in the oabi
configuration :-) ; this implies that there are almost certainly bugs
(untested code is almost always buggy or bit-rotten in my experience).

A final note is that there is no gcc configuration for the old abi, the
only way it would be even remotely possible to invoke it would be to
pass -Wa,-moabi to every possible invocation of the compiler.   I find
it hard to believe that has been happening for 5 years and yet that
nobody has proposed creating a configuration of gcc that does that by
default.

So my opinion is that we should go straight to option 3.  The 'risk' is
minimal, and if it does materialise we could push back by insisting
somebody else step forward to maintain that specific code if it is
resurrected: once it is separate from the current ARM code that is far
more feasible.

Nick, of course, has a veto; but he doesn't have to use it...

R.

  reply	other threads:[~2004-11-03 15:05 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-11-01 21:10 PATCH: Reduce size of SymbianOS DLLs Mark Mitchell
2004-11-02 10:29 ` Richard Earnshaw
2004-11-02 14:56   ` Ian Lance Taylor
2004-11-02 15:04     ` Richard Earnshaw
2004-11-02 15:08       ` Ian Lance Taylor
2004-11-02 15:11         ` Richard Earnshaw
2004-11-02 15:17       ` Daniel Jacobowitz
2004-11-02 15:21         ` Richard Earnshaw
2004-11-02 15:29           ` Nick Clifton
2004-11-02 15:33             ` Richard Earnshaw
2004-11-02 15:43               ` Nick Clifton
2004-11-02 16:24                 ` Mark Mitchell
2004-11-02 16:31                   ` Daniel Jacobowitz
2004-11-02 16:42                     ` Mark Mitchell
2004-11-03 15:05                       ` Richard Earnshaw [this message]
2004-11-03 15:25                         ` Obsoleting elfarm-oabi [was Re: PATCH: Reduce size of SymbianOS DLLs] Nick Clifton
2004-11-03 15:57                         ` Richard Earnshaw
2004-11-04  8:52                           ` Nick Clifton
2004-11-02 17:44                     ` PATCH: Reduce size of SymbianOS DLLs Mark Mitchell
2004-11-02 15:59               ` Daniel Jacobowitz
2004-11-02 16:01                 ` Richard Earnshaw
2004-11-03 17:24 ` Richard Earnshaw

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=1099494297.6939.106.camel@pc960.cambridge.arm.com \
    --to=rearnsha@gcc.gnu.org \
    --cc=binutils@sources.redhat.com \
    --cc=drow@false.org \
    --cc=ian@wasabisystems.com \
    --cc=mark@codesourcery.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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).