Re: [RFC PATCH, i386]: Use %r15 for REAL_PIC_OFFSET_TABLE_REGNUM on x86_64

2012-12-24 Thread Mike Frysinger
On Monday 24 December 2012 17:26:47 Leif Ekblad wrote: > In the case of cpuid, the code is hardly performance sensitive, and > probably runs only at startup. An alternative solution for the broken code > here is to move the result from rbx to another register, and to > save/restore rbx. Currently,

Re: [RFC PATCH, i386]: Use %r15 for REAL_PIC_OFFSET_TABLE_REGNUM on x86_64

2012-12-24 Thread Leif Ekblad
In the case of cpuid, the code is hardly performance sensitive, and probably runs only at startup. An alternative solution for the broken code here is to move the result from rbx to another register, and to save/restore rbx. Currently, this is the only place in libgcc and newlib affected by this

Re: [RFC PATCH, i386]: Use %r15 for REAL_PIC_OFFSET_TABLE_REGNUM on x86_64

2012-12-24 Thread Andi Kleen
Uros Bizjak writes: > Hello! > > Currently, we use %rbx as REAL_PIC_OFFSET_TABLE_REGNUM on x86_64. > Since this register gets marked as fixed reg in > ix86_conditional_register_usage, we get into troubles with insns that > use %rbx (cmpxchg, cpuid). According to x86_64 psABI, we are free to > use

Re: [PATCH] PR c++/52343 - error with alias template as template template argument

2012-12-24 Thread Gabriel Dos Reis
On Mon, Dec 24, 2012 at 7:57 AM, Dodji Seketeli wrote: > Gabriel Dos Reis writes: > > G> On Sun, Dec 23, 2012 at 11:04 PM, Jason Merrill wrote: >>> On 12/21/2012 07:35 AM, Dodji Seketeli wrote: else if (TREE_TYPE (t) && INTEGRAL_OR_ENUMERATION_TYPE_P (TREE_TYPE (t))

Re: [Patch, libgomp] libgomp.fortran/fortran.exp - add -fintrinsic-modules-path ${blddir}

2012-12-24 Thread David Edelsohn
On Fri, Dec 21, 2012 at 11:07 AM, Jakub Jelinek wrote: > Do you happen to use spaces or similar problematic characters in the build > directory path? > Can you paste some command line from libgomp.log? Something is definitely broken. When I run the fortran tests by themselves, it is okay, but in

[RFC PATCH, i386]: Use %r15 for REAL_PIC_OFFSET_TABLE_REGNUM on x86_64

2012-12-24 Thread Uros Bizjak
Hello! Currently, we use %rbx as REAL_PIC_OFFSET_TABLE_REGNUM on x86_64. Since this register gets marked as fixed reg in ix86_conditional_register_usage, we get into troubles with insns that use %rbx (cmpxchg, cpuid). According to x86_64 psABI, we are free to use any register, so attached patch ch

Re: fix libquadmath build regression

2012-12-24 Thread Paolo Bonzini
Il 21/12/2012 06:17, Alexandre Oliva ha scritto: > Revision 193063 brought in calls to fraiseexcept() into libquadmath, > which caused a build regression on Fedora 16 (BLAG 160k actually) x86_64 > while building an i686-linux-gnu native toolchain. > > The problem is that glibc has an extern inline

Re: [PATCH] PR c++/55311 - Cannot specialize alias template with arg of type array of char

2012-12-24 Thread Dodji Seketeli
Jason Merrill writes: > On 12/22/2012 11:03 AM, Dodji Seketeli wrote: >> [1]: The relacement of the VAR_DECL by its initializer is done by >> decay_conversion by callig decl_constant_value_safe. That replacement >> doesn't happen if processing_template_decl is not set. That's why it >> doesn't

Re: [PATCH] PR c++/52343 - error with alias template as template template argument

2012-12-24 Thread Dodji Seketeli
Gabriel Dos Reis writes: G> On Sun, Dec 23, 2012 at 11:04 PM, Jason Merrill wrote: >> On 12/21/2012 07:35 AM, Dodji Seketeli wrote: >>> >>> else if (TREE_TYPE (t) >>>&& INTEGRAL_OR_ENUMERATION_TYPE_P (TREE_TYPE (t)) >>> - && !TREE_CONSTANT (t)) >>> + && !TREE_CO

Re: [PATCH] PR c++/52343 - error with alias template as template template argument

2012-12-24 Thread Dodji Seketeli
Jason Merrill writes: > On 12/21/2012 07:35 AM, Dodji Seketeli wrote: >> else if (TREE_TYPE (t) >> && INTEGRAL_OR_ENUMERATION_TYPE_P (TREE_TYPE (t)) >> - && !TREE_CONSTANT (t)) >> + && !TREE_CONSTANT (t) >> + /* Class template and alias template arguments should be O

Re: C++ PATCH for c++/54325 (wrong error initializing abstract base class)

2012-12-24 Thread Paolo Carlini
... to explain more concretely what I mean, if I *brutally* hack mainline per the below, then the testcase is accepted. Paolo. // Index: call.c === --- call.c (revision 194659) +++ call.c (working copy) @@

Re: C++ PATCH for c++/54325 (wrong error initializing abstract base class)

2012-12-24 Thread Paolo Carlini
On 12/24/2012 05:56 AM, Jason Merrill wrote: On 12/22/2012 06:02 PM, Paolo Carlini wrote: Well, we still reject it after the patch My 4.7 and trunk compilers both accept it (with -std=c++11, of course). I just updated and rebuilt my 4.7 for you (which definitely I didn't hack over the next da