What does this do ?
Danny
On Mon, 2009-05-18 at 19:16 +0200, Vincent R. wrote:
> Transferring a mail from Dave Korn about PE change in binutils.
> Dave or Danny could you have a look ?
>
> Dave Korn wrote:
> > Dave Korn wrote:
> >
> >> But it's quite a big one, and the tests still have 24 hours or so to run
> >
> > The tests completed, and showed up a couple of problems;
>
> > So, I'm now putting this
> > revised version through the full tests again. Results tomorrow sometime,
> > along with an updated changelog.
>
> The tests completed with no regressions on
>
> i686-pc-cygwin native and x {i686-pc-mingw32, x86_64-pc-mingw32} and
> i686-pc-linux-gnu x {usual long list of 71 random targets, see footer of
> post[*]}
>
> I also temporarily hacked in enough basic support for cegcc targets to
> give
> them a quick cross-test and verify nothing significant broke, although they
> don't pass the tests because they have some kind of file-alignment limited
> to
> 8 which the compiler warns about:
>
> /gnu/binutils/src/ld/testsuite/ld-pe/aligncomm-1.c:5: warning: alignment of
> 'r' is greater than maximum object file alignment. Using 8
> /gnu/binutils/src/ld/testsuite/ld-pe/aligncomm-1.c:5: warning: alignment of
> 'r' is greater than maximum object file alignment. Using 8
>
> ... so they show as UNRESOLVED. We could XFAIL, skip, or add the required
> compiler flags as desired whenever the new cegcc target names get added to
> config (not sure what's happening on that front), or potentially the
> compiler
> could be changed to take advantage of the new support.
>
>
>
> Because of this minor potential hiccup, I'll wait 24 hours for comments
> before committing the final version of the patch (unchanged since previous
> posting at http://sourceware.org/ml/binutils/2009-05/msg00271.html). (Also
> in
> case anyone wants to say I should separate out the refactoring of the ld-pe
> testcases into a separate patch, which I would have done in the first place
> if
> I'd known in advance I was going to end up doing it.... but it's been
> pretty
> thoroughly tested, so I reckon it'll be ok.)
>
>
> bfd/ChangeLog:
>
> * cofflink.c (process_embedded_commands): Ignore "-aligncomm".
>
> gas/ChangeLog:
>
> * NEWS: Mention new feature.
> * config/obj-coff.c (obj_coff_common_parse): New function.
> (obj_coff_comm): Likewise.
> (coff_pseudo_table): Override default ".comm" definition on PE.
> * doc/as.texinfo: Document new feature.
>
> gas/testsuite/ChangeLog
>
> * gas/pe/: New directory for PE format-specific tests.
> * gas/pe/aligncomm-a.d: New test pattern file.
> * gas/pe/aligncomm-a.s: New test source file.
> * gas/pe/aligncomm-b.d: New test pattern file.
> * gas/pe/aligncomm-b.s: New test source file.
> * gas/pe/aligncomm-c.d: New test pattern file.
> * gas/pe/aligncomm-c.s: New test source file.
> * gas/pe/aligncomm-d.d: New test pattern file.
> * gas/pe/aligncomm-d.s: New test source file.
> * gas/pe/pe.exp: New test control script.
> * lib/gas-defs.exp (is_pecoff_format): New function.
>
> ld/ChangeLog:
>
> * NEWS: Mention new feature.
> * deffile.h (def_file_aligncomm): Add new struct definition.
> (def_file): Add new def_file_aligncomm member.
> * deffilep.y (%token): Add new ALIGNCOMM token.
> (command): Add production rule for ALIGNCOMM.
> (def_file_free): Free any chained def_file_aligncomm structs.
> (diropts[]): Add entry for '-aligncomm' .drectve command.
> (def_aligncomm): New grammar function.
> * ld.texinfo: Document new feature.
> * pe-dll.c (process_def_file): Rename from this ...
> (process_def_file_and_drectve): ... to this, updating all callers,
> and process any aligncomms chained to the def file after scanning
> all .drectve sections.
> (generate_edata): Updated to match.
> (pe_dll_build_sections): Likewise.
>
> ld/testsuite/ChangeLog:
>
> * ld-pe/aligncomm-1.c: New test source file.
> * ld-pe/aligncomm-2.c: Likewise.
> * ld-pe/aligncomm-3.c: Likewise.
> * ld-pe/aligncomm-4.c: Likewise.
> * ld-pe/aligncomm.d: New test pattern file.
>
> * ld-pe/direct.exp: Deleted, and content moved into ...
> * ld-pe/pe-run.exp: ... New common file for all PE run tests.
>
> * ld-pe/vers-script.exp: Deleted, and content merged into ...
> * ld-pe/pe-compile.exp: ... New common file for PE tests needing
> a compiler, adding aligned common tests.
>
> * ld-pe/pe.exp: Update header comment.
>
> cheers,
> DaveK
>
--
Danny Backx ; danny.backx - at - scarlet.be ; http://danny.backx.info
------------------------------------------------------------------------------
Crystal Reports - New Free Runtime and 30 Day Trial
Check out the new simplified licensing option that enables
unlimited royalty-free distribution of the report engine
for externally facing server and web deployment.
http://p.sf.net/sfu/businessobjects
_______________________________________________
Cegcc-devel mailing list
Cegcc-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/cegcc-devel
