This is the mail archive of the mailing list for the binutils project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [RFA] Fix broken .linkonce

On Thu, Jul 15, 2004 at 09:23:31AM -0400, Christopher Faylor wrote:
> On Mon, Jul 05, 2004 at 09:55:51AM -0700, H. J. Lu wrote:
> >On Mon, Jul 05, 2004 at 12:50:52PM -0400, Christopher Faylor wrote:
> >> On Sun, Jul 04, 2004 at 07:48:08AM -0700, H. J. Lu wrote:
> >> >On Sat, Jul 03, 2004 at 11:11:37PM -0400, Christopher Faylor wrote:
> >> >>H.J.  Lu's 2004-05-11 change to subsegs.c caused .linkonce symbols to
> >> >>be treated as undeclared, at least for PE.
> >> >
> >> >Can you add a testcase for it?
> >> 
> >> Can I get confirmation that this is actually the right fix before we
> >> start talking about test cases?
> >
> >I have used the new binutils to build gcc, glibc and kernel on Linux. I
> >don't have any problems. I am trying to understand why/how it is broken
> >on PE. A testcase will be very helpful.
> Sorry for the delay.  Real world intrusions.
> Here's a very simple test case.  It fails with the new version of gas
> and succeeds with older versions and after the application of the
> previously mentioned patch.
>   as -o linkonce.o linkonce.s
>   gcc -o linkonce linkonce.o -lcygwin
>   linkonce.o(.txt+0x0):fake: undefined reference to `.foo'
>   collect2: ld returned 1 exit status
> cgf
> 	 .section .foo,"w" 
> 	 .linkonce 
> 	 .long	27 
> 	 .section .txt,"wx" 
> 	 .long .foo 

This is a known issue:

Where does this code come from? If you can change your code, we can
make it as error. Or you should somehow revert my patch for PE only
and add your teestcase to gas testsuite.

> 	.global	_WinMain@16
> _WinMain@16:


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]