Home Original page

[PATCH] ld: Failing to copy processor specific flags for split-by-file

Nick Clifton nickc@redhat.com
Tue Sep 7 15:11:00 GMT 2010 
Hi Andrew,

> I was wondering if there was anything I could do that would
> help get these patches merged?

Pinging the list like that was just the thing.

> ...or should I just be more patient :)

Actually it would also have helped if you had filed a bug report for 
this problem with the bugzilla system.  That way there would be a PR 
number that we could refer to in changelog entries and comments in the code.

 > I've attached two patches, split-test.patch, contains the test that
 > highlights this problem.

Thanks.  I have applied both patches.  I also had to make up some 
changelog entries for them (see below).  If you can include such entries 
yourself when submit patches in the future it will make things a little 
bit easier.

 > I'll be honest, I don't really understand
 > what the "l" flag is used for, but I don't *think* that it makes any
 > difference for the purpose of this test.

It doesn't make a huge difference, although there is a need for the 
presence of an unusual flag, such as "l".

Using "l" did point out one other small problem to me, which was that 
readelf was not explaining what "l" stands for in the "Key to Flags" 
part of its output.  So I created a small additional patch to do this.

Cheers
   Nick

ld/ChangeLog
2010-09-07  Andrew Burgess  <aburgess@broadcom.com>

	* ldwrite.c (clone_section): Call bfd_copy_private_section_data on
	newly cloned section.

ld/testsuite/ChangeLog
2010-09-07  Andrew Burgess  <aburgess@broadcom.com>

	* ld-x86-64/split-by-file.rd: New test.
	* ld-x86-64/split-by-file1.s: New test source code.
	* ld-x86-64/split-by-file2.s: New test source code.
	* ld-x86-64/x86-64.exp: Run the new test.

readelf/ChangeLog
2010-09-07  Nick Clifton  <nickc@redhat.com>

	* readelf.c (process_section_headers): Mention meaning of 'l'
	section flag for x86-64 targets.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: readelf.c.patch
Type: text/x-diff
Size: 1447 bytes
Desc: not available
URL: <https://sourceware.org/pipermail/binutils/attachments/20100907/7f0088c3/attachment.bin>
More information about the Binutils mailing list