This is the mail archive of the gcc-bugs@gcc.gnu.org mailing list for the GCC project.


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

Re: g++/cygwin-B19 linkage problem, multiple definitions


Jason Merrill <jason@cygnus.com> writes:
> >>>>> Mumit Khan <khan@xraylith.wisc.edu> writes:
> 
>  > Also, I may be completely off the base here, but wasn't there something
>  > about MIPS port not liking this change (and hence Jason's Oct 13 change 
>  > to mips/iris6.h and varasm.c)?
> 
> Yes.  And you'll also need
> 
>         * decl2.c (finish_vtable_vardecl): Check DECL_INTERFACE_KNOWN
>         instead of linkage.
> 
> I wasn't planning to put this change in because of the follow-on patches it
> required, but I could be convinced if it's a serious problem.
> 

Well, I have a folder full of bug reports that all lead to this. However,
it has a trivial workaround (define a destructor, even if empty), so it's
not crucial. The trouble is when you're using a library with such cases,
and then you're stuck. 

I did run the testsuite and no regressions on x86-win32 and ix86-linux. 
Of course, the testsuite doesn't really test for lots of things that can 
go wrong here.

Mumit



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