------- Additional Comments From nickc at redhat dot com  2007-09-28 11:57 
-------
Hi Ralf,

  OK, please could you try out the uploaded patch.  There are several points
that need to be looked at:

  * The fix is only enabled for x86 PE ports.  Does it need to be enabled for
other PE ports ?  (ARM, MIPS, SH, etc).

  * The new script is only used when --enable-auto-import is present on the
linker command line, not when the linker magically enables auto import for
itself.  (There appears to be code in pe{p}.em:...find_data_imports() that
magically enables auto importing.  I was not sure what should be done in this
situation).

  * There are still some input sections that are mapped to the .rdata output
section, even with the new script active:
 .rdata BLOCK(__section_alignment__) :
  {
     *(.eh_frame)
    ___RUNTIME_PSEUDO_RELOC_LIST__ = .;
    __RUNTIME_PSEUDO_RELOC_LIST__ = .;
    *(.rdata_runtime_pseudo_reloc)
    ___RUNTIME_PSEUDO_RELOC_LIST_END__ = .;
    __RUNTIME_PSEUDO_RELOC_LIST_END__ = .;
  }

  Should these input sections (and symbols) be moved to the .data section as 
well ?


Most importantly, I have not checked the patched linker when used to link real
applications.  I am not really set up here to do that, so I hope that you will
be able to test it for me.

Cheers
  Nick


-- 


http://sourceware.org/bugzilla/show_bug.cgi?id=4844

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.


_______________________________________________
bug-binutils mailing list
bug-binutils@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-binutils

Reply via email to