Mailing-List: contact cygwin-apps-help AT sourceware DOT cygnus DOT com; run by ezmlm
Sender: cygwin-apps-owner AT sourceware DOT cygnus DOT com
List-Subscribe: <mailto:cygwin-apps-subscribe AT sources DOT redhat DOT com>
List-Archive: <http://sources.redhat.com/ml/cygwin-apps/>
List-Post: <mailto:cygwin-apps AT sources DOT redhat DOT com>
List-Help: <mailto:cygwin-apps-help AT sources DOT redhat DOT com>, <http://sources.redhat.com/lists.html#faqs>
Delivered-To: mailing list cygwin-apps AT sources DOT redhat DOT com
Message-ID: <3B699A88.7060904@ece.gatech.edu>
Date: Thu, 02 Aug 2001 14:23:04 -0400
From: Charles Wilson <cwilson AT ece DOT gatech DOT edu>
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:0.9.2) Gecko/20010713
X-Accept-Language: en-us
MIME-Version: 1.0
To: DJ Delorie <dj AT delorie DOT com>
CC: binutils AT sources DOT redhat DOT com, cygwin-apps AT cygwin DOT com
Subject: Re: [RFA] pei386 dll: auto-import patch
References: <3B670087 DOT 7090102 AT ece DOT gatech DOT edu> <200108011735 DOT NAA32231 AT envy DOT delorie DOT com> <3B6846D2 DOT 9040206 AT ece DOT gatech DOT edu> <200108011847 DOT OAA32757 AT envy DOT delorie DOT com> <3B68C879 DOT 1070809 AT ece DOT gatech DOT edu> <200108020347 DOT XAA05289 AT envy DOT delorie DOT com> <3B68ED4B DOT 4080405 AT ece DOT gatech DOT edu> <200108020610 DOT CAA06564 AT envy DOT delorie DOT com> <3B68F80C DOT 7030207 AT ece DOT gatech DOT edu> <200108020700 DOT DAA06930 AT envy DOT delorie DOT com>
Content-Type: text/plain; charset=us-ascii; format=flowed
Content-Transfer-Encoding: 7bit

DJ Delorie wrote:

>> > Sorry, can't use snprintf.
>>
>>Why not?
>>
> 
> Not all systems have it.  Think "cross compilers".


Ah.


>>Like this?  ('course, you have to #include <math.h> for the ceil 
>>function.  Perhaps multiples of 128 or 256 would be better...bit 
>>shifting and masking is easy -- but not really portable...)
>>
> 
> Hmmm... You can avoid float math if you use a power of two:
> 
> 	size = (size + 127) & ~127;


Okay, new binaries and source tarball (plus the patch, ChangeLog, TODO 
list, etc) are at:

http://www.neuro.gatech.edu/users/cwilson/cygutils/robert-collins/latest/binutils/

This version supersedes the one I posted late last night, and includes 
all suggestions so far from DJ (including the non-float math above) and 
has been (minimally) tested by me.  Open issues (other than more 
thorough testing) are:

   make data_import_dll static?
     No, used in both pe.em and pe-dll.c
     I suppose we could rename it pe_data_import_dll.

   Ralf's bug report.  My intuition sez this is
     related to the data_import_dll thing.
     http://sources.redhat.com/ml/binutils/2001-06/msg00742.html

I need to slow my breakneck pace here, and get some "real work" done. 
If somebody (Robert?  Paul?  Ralf?) could take a look at these two 
issues I'd appreciate it.

--Chuck

Done (in 20010801-2):
----------------------

ld.texinfo: @cindex stuff
pe-dll.c (auto_export): replace #if 0 block with textual comment
pe-dll.c (make_import_fixup_mark): make static
pe-dll.c (add_bfd_to_link): revert to static
pe-dll.c (make_import_fixup_mark): dynamically size the
   fixup_name buffer. Use DJ's (x + 127) & ~127 logic.
pe.em: make __FUNCTION__ safe for %

Done (in 20010801-1):
-----------------------

1) --enable-gory-debug ==> --enable-extra-pe-debug
2) added --enable-auto-import option
3) made --disable-auto-import the default
4) encapsulated pe_dll_auto_import into bfd_link_info structure: new 
field "pei386_auto_import
5) changed pe_em(auto_export) filters to be table-based
6) moved detailed docs into ldint.texi
7) removed "bugger"
8) followed GNU coding style
9) removed ldlang.c patch
10) removed C++ comments