public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
From: Ian Lance Taylor <ian@wasabisystems.com>
To: Richard Earnshaw <rearnsha@gcc.gnu.org>
Cc: Mark Mitchell <mark@codesourcery.com>, binutils@sources.redhat.com
Subject: Re: PATCH: Reduce size of SymbianOS DLLs
Date: Tue, 02 Nov 2004 14:56:00 -0000	[thread overview]
Message-ID: <m3d5ywi9kf.fsf@gossamer.airs.com> (raw)
In-Reply-To: <1099391341.11478.87.camel@pc960.cambridge.arm.com>

Richard Earnshaw <rearnsha@gcc.gnu.org> writes:

> On Mon, 2004-11-01 at 21:10, Mark Mitchell wrote:
> > The linker believed that SymbianOS BPABI DLLs were demand-paged, which
> > meant that it wanted to align segments on page-size boundaries.
> > However, all BPABI DLLs are post-processed before becoming something
> > that can actually be executed, so this is just wasted space in the
> > file.  I realized that when D_PAGED is clear, the linker will not try
> > to map the program headers, which means that some special-case code I
> > added for SymbianOS can be removed.
> 
> Mark,
> 
> I've probably missed something subtle here, but these changes are all
> being made to elfarm-nabi.c, a generic ARM file.  What is it that
> prevents all these symbian-specific hooks from being called when it's
> not targeting symbianos?

It's the usual BFD crock.  elf32-target.h is included twice--once by
elf32-arm.h, and once, for the Symbian stuff, by elfarm-nabi.c itself.

(I haven't tried to verify that Mark has done everything correctly,
but certainly the right framework is there).

(It would not be unreasonable to create a new file elfarm-symbian.c,
and make the appropriate functions and data structures in
elfarm-nabi.c non-static.)

Ian

  reply	other threads:[~2004-11-02 14:56 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-11-01 21:10 Mark Mitchell
2004-11-02 10:29 ` Richard Earnshaw
2004-11-02 14:56   ` Ian Lance Taylor [this message]
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                       ` Obsoleting elfarm-oabi [was Re: PATCH: Reduce size of SymbianOS DLLs] Richard Earnshaw
2004-11-03 15:25                         ` 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=m3d5ywi9kf.fsf@gossamer.airs.com \
    --to=ian@wasabisystems.com \
    --cc=binutils@sources.redhat.com \
    --cc=mark@codesourcery.com \
    --cc=rearnsha@gcc.gnu.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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).