Re: [patch] make default linker --hash-style configurable option

2011-05-10 Thread Ian Lance Taylor
Richard Guenther richard.guent...@gmail.com writes: On Mon, May 9, 2011 at 7:11 PM, Paul Pluzhnikov ppluzhni...@google.com wrote: On Mon, May 9, 2011 at 9:51 AM, Joseph S. Myers jos...@codesourcery.com wrote: On Mon, 9 May 2011, Paolo Bonzini wrote: Uhm, so we deadlocked, I thought the

Re: [patch] make default linker --hash-style configurable option

2011-05-10 Thread Jakub Jelinek
On Tue, May 10, 2011 at 07:02:44AM -0700, Ian Lance Taylor wrote: Otherwise if people link with ld they suddenly get different hash-style. That looks wrong to me. That turns out not to be the case. Both gold and GNU ld accept the same set of --hash-style options. And we keep telling

Re: [patch] make default linker --hash-style configurable option

2011-05-10 Thread Paul Pluzhnikov
On Tue, May 10, 2011 at 7:02 AM, Ian Lance Taylor i...@google.com wrote: Richard Guenther richard.guent...@gmail.com writes: I wonder why this is a GCC specific patch and not a linker patch.  Why not change the linker(s) to accept such configure option that changes its default behavior? It

Re: [patch] make default linker --hash-style configurable option

2011-05-10 Thread Richard Guenther
On Tue, May 10, 2011 at 4:24 PM, Paul Pluzhnikov ppluzhni...@google.com wrote: On Tue, May 10, 2011 at 7:02 AM, Ian Lance Taylor i...@google.com wrote: Richard Guenther richard.guent...@gmail.com writes: I wonder why this is a GCC specific patch and not a linker patch.  Why not change the

Re: [patch] make default linker --hash-style configurable option

2011-05-09 Thread Paul Pluzhnikov
Ping? Ping? Ping? Ping? Ping? http://gcc.gnu.org/ml/gcc-patches/2011-04/msg00246.html CC'ing the rest of build system maintainers. On Mon, May 2, 2011 at 8:56 AM, Paul Pluzhnikov ppluzhni...@google.com wrote: On Mon, May 2, 2011 at 7:59 AM, Joseph S. Myers jos...@codesourcery.com wrote:

Re: [patch] make default linker --hash-style configurable option

2011-05-09 Thread Paolo Bonzini
On 05/09/2011 05:59 PM, Paul Pluzhnikov wrote: Ping? Ping? Ping? Ping? Ping? http://gcc.gnu.org/ml/gcc-patches/2011-04/msg00246.html CC'ing the rest of build system maintainers. None of the build system maintainers can approve gcc.c changes. And those can be approved only by either a

Re: [patch] make default linker --hash-style configurable option

2011-05-09 Thread Joseph S. Myers
On Mon, 9 May 2011, Paolo Bonzini wrote: On 05/09/2011 05:59 PM, Paul Pluzhnikov wrote: Ping? Ping? Ping? Ping? Ping? http://gcc.gnu.org/ml/gcc-patches/2011-04/msg00246.html CC'ing the rest of build system maintainers. None of the build system maintainers can approve gcc.c

Re: [patch] make default linker --hash-style configurable option

2011-05-09 Thread Paolo Bonzini
On Mon, May 9, 2011 at 18:45, Joseph S. Myers jos...@codesourcery.com wrote: On Mon, 9 May 2011, Paolo Bonzini wrote: On 05/09/2011 05:59 PM, Paul Pluzhnikov wrote: Ping? Ping? Ping? Ping? Ping? http://gcc.gnu.org/ml/gcc-patches/2011-04/msg00246.html CC'ing the rest of build system

Re: [patch] make default linker --hash-style configurable option

2011-05-09 Thread Joseph S. Myers
On Mon, 9 May 2011, Paolo Bonzini wrote: On Mon, May 9, 2011 at 18:45, Joseph S. Myers jos...@codesourcery.com wrote: On Mon, 9 May 2011, Paolo Bonzini wrote: On 05/09/2011 05:59 PM, Paul Pluzhnikov wrote: Ping? Ping? Ping? Ping? Ping?

Re: [patch] make default linker --hash-style configurable option

2011-05-09 Thread Paul Pluzhnikov
On Mon, May 9, 2011 at 9:51 AM, Joseph S. Myers jos...@codesourcery.com wrote: On Mon, 9 May 2011, Paolo Bonzini wrote: Uhm, so we deadlocked, I thought the other way.  I cannot really express any opinion about the desirability of the feature, but the configure syntax is certainly okay with

Re: [patch] make default linker --hash-style configurable option

2011-05-02 Thread Paul Pluzhnikov
Ping? Ping? Ping? Ping? This is getting ridiculous. Would someone please accept the patch, tell me what to fix in it to make it acceptable, or explain why it is a bad idea? Thanks! On Mon, Apr 25, 2011 at 9:08 AM, Paul Pluzhnikov ppluzhni...@google.com wrote: Ping? Ping? Ping? On Mon, Apr

Re: [patch] make default linker --hash-style configurable option

2011-04-25 Thread Paul Pluzhnikov
Ping? Ping? Ping? On Mon, Apr 18, 2011 at 9:45 AM, Paul Pluzhnikov ppluzhni...@google.com wrote: Ping? Ping? On Mon, Apr 11, 2011 at 11:00 AM, Paul Pluzhnikov ppluzhni...@google.com wrote: Ping? -- Paul Pluzhnikov

Re: [patch] make default linker --hash-style configurable option

2011-04-18 Thread Paul Pluzhnikov
Ping? Ping? On Mon, Apr 11, 2011 at 11:00 AM, Paul Pluzhnikov ppluzhni...@google.com wrote: Ping? -- Paul Pluzhnikov

Re: [patch] make default linker --hash-style configurable option

2011-04-11 Thread Paul Pluzhnikov
Ping? On Mon, Apr 4, 2011 at 5:01 PM, Joseph S. Myers jos...@codesourcery.com wrote: On Tue, 5 Apr 2011, Matthias Klose wrote: ... Linux distributions pass more than that by default to the linker, e.g. --as-needed and --no-copy-dt-needed-entries.  Wouldn't it make more sense to add

[patch] make default linker --hash-style configurable option

2011-04-04 Thread Paul Pluzhnikov
Greetings, Several Linux distributions (e.g. Fedora) carry local patches that turn on --hash-style=gnu for all links. Attached is a proposed patch (originally by Satoru Takabayashi) that makes default hash style a configure option. Tested by doing native bootstrap and verifying that no

Re: [patch] make default linker --hash-style configurable option

2011-04-04 Thread Rainer Orth
ppluzhni...@google.com (Paul Pluzhnikov) writes: Several Linux distributions (e.g. Fedora) carry local patches that turn on --hash-style=gnu for all links. Shouldn't configure verify that the linker used actually understands that option? Rainer --

Re: [patch] make default linker --hash-style configurable option

2011-04-04 Thread Ian Lance Taylor
Rainer Orth r...@cebitec.uni-bielefeld.de writes: ppluzhni...@google.com (Paul Pluzhnikov) writes: Several Linux distributions (e.g. Fedora) carry local patches that turn on --hash-style=gnu for all links. Shouldn't configure verify that the linker used actually understands that option?

Re: [patch] make default linker --hash-style configurable option

2011-04-04 Thread Matthias Klose
On 04.04.2011 20:17, Paul Pluzhnikov wrote: Greetings, Several Linux distributions (e.g. Fedora) carry local patches that turn on --hash-style=gnu for all links. Attached is a proposed patch (originally by Satoru Takabayashi) that makes default hash style a configure option. Tested by

Re: [patch] make default linker --hash-style configurable option

2011-04-04 Thread Joseph S. Myers
On Tue, 5 Apr 2011, Matthias Klose wrote: On 04.04.2011 20:17, Paul Pluzhnikov wrote: Greetings, Several Linux distributions (e.g. Fedora) carry local patches that turn on --hash-style=gnu for all links. Attached is a proposed patch (originally by Satoru Takabayashi) that makes