Re: [PATCH] fix PR51910, take 2

2012-02-11 Thread Jason Merrill
On 02/08/2012 03:01 AM, Jakub Jelinek wrote: On Tue, Feb 07, 2012 at 11:41:03PM -1000, Jason Merrill wrote: Hmm, I wrote up something quite similar on the plane. One difference is that mine synchronizes .rpo files that start with some variants chosen and others not. Does this make sense to you

Re: [PATCH] fix PR51910, take 2

2012-02-08 Thread Jakub Jelinek
On Tue, Feb 07, 2012 at 11:41:03PM -1000, Jason Merrill wrote: > Hmm, I wrote up something quite similar on the plane. One > difference is that mine synchronizes .rpo files that start with some > variants chosen and others not. Does this make sense to you? You mean the maybe_tweak change? Not s

Re: [PATCH] fix PR51910, take 2

2012-02-08 Thread Jason Merrill
On 02/02/2012 11:27 PM, Jakub Jelinek wrote: Anyway, here is the tweaking all the symbols that demangle the same equally patch (untested so far) as an alternative. On the example it unfortunately causes also the D2 dtor into the link which wasn't there otherwise (and with -Wl,--no-demangle). Wh

Re: [PATCH] fix PR51910, take 2

2012-02-03 Thread Jakub Jelinek
On Fri, Feb 03, 2012 at 10:27:17AM +0100, Jakub Jelinek wrote: > Anyway, here is the tweaking all the symbols that demangle the same > equally patch (untested so far) as an alternative. On the example it Now bootstrapped/regtested on x86_64-linux and i686-linux. Jakub

Re: [PATCH] fix PR51910, take 2

2012-02-03 Thread Jakub Jelinek
On Thu, Feb 02, 2012 at 09:10:50PM -0700, Sandra Loosemore wrote: > --- gcc/testsuite/g++.dg/torture/pr51910.C(revision 0) > +++ gcc/testsuite/g++.dg/torture/pr51910.C(revision 0) > @@ -0,0 +1,19 @@ > +// PR c++/51910 > +// Check that -frepo works in the presence of linker symbol demanglin

[PATCH] fix PR51910, take 2

2012-02-02 Thread Sandra Loosemore
Here is another attempt to fix the bad interaction between --with-demangler-in-ld and -frepo processing. This version of the patch always disables demangling in ld when repository files are present for the link. If demangling is requested (either by an explicit -Wl,--demangle option, by using