RFA: broken strip/objcopy

[Kai Tietz]

Hello,

2009/4/24 Kai Tietz <ktietz70@googlemail.com>:
> Hello,
>
> The current implementation of objcopy/strip in binutils has problems,
> because for pe-coff targets. It assumes at the moment that it is an
> i386 version. This leads to issues (especially on strip) for PE+
> target that binaries are getting corrupted. So I think we have here to
> move those target specific pe-coff parts into seperate files. (I
> assume that possibly for othe pe-coff targets then i386 the same
> problem exists). Of course we could use here target specific macros to
> change internal structures, but for a mulilib version of them this
> isn't suitable anymore.
> So possible somebody of you have a strong opinion about how this issue
> could be solved. I'll prepare a patch for it and post it soon.

Ah, I found the major issue here

2009-04-17  H.J. Lu  <hongjiu.lu@intel.com>

        PR binutils/10074

This patch made objcopy and strip for PE-Coff target 32-bit PE only.
This patch should fix it.

ChangeLog

2009-04-24  Kai Tietz  <kai.tietz@onevision.com>

	* objcopy.c (pep_objcopy.h): Add include.
	(copy_object): Add check for PE+ x86_64
	and call for it pep_copy_object.
	* pep_objcopy.c: New file.
	(pep_copy_object): New implementation.
	* pep_objcopy.h: New include.
	(pep_copy_object): New declaration.
	* Makefile.am: Add pep_objcopy.c to make.
	* Makefile.in: Regenerated.

Tested for x86_64-pc-mingw32. Ok for apply?

Cheers,
Kai

-- 
|  (\_/) This is Bunny. Copy and paste
| (='.'=) Bunny into your signature to help
| (")_(") him gain world domination
-------------- next part --------------
A non-text attachment was scrubbed...
Name: fix_strip.diff
Type: application/octet-stream
Size: 13708 bytes
Desc: not available
URL: <https://sourceware.org/pipermail/binutils/attachments/20090424/2ebdb596/attachment.obj>