Re: [patch, libgfortran] PR48960 OPEN statement modifies NEWUNIT variable on error

2012-12-26 Thread Janne Blomqvist
On Wed, Dec 26, 2012 at 9:46 AM, Jerry DeLisle wrote: > The attached patch fixes this bug by not assigning the new unit value to the > user variable until after the OPEN is successful. > > Regression tested on Linux X86-64. > > OK for trunk with the test case from the PR? Ok. Thanks for the patch

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

2012-12-26 Thread Uros Bizjak
On Tue, Dec 25, 2012 at 8:27 PM, Mike Frysinger 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/restor

[RFC PATCH] Implementing ifunc target hook

2012-12-26 Thread Alexander Ivchenko
Hi, Currently Android dynamic loader does not support indirect functions (And I don't think that it will someday). But there is no way for us to specify that for gcc, and for example, tests like gcc.dg/attr-ifunc-* are failing on android right now. The attached patch is indended to add the target

Re: [Patch, libffi] libffi merge

2012-12-26 Thread Anthony Green
I've committed this patch. Thanks, Anthony Green On Sun, Dec 23, 2012 at 4:37 PM, Anthony Green wrote: > The attached patch is fairly complete merge of the libffi side-stream > git repo. Highlights include aarch64 support, blackfin support, and > additional windows ABI support. I will commi

Re: [RFC PATCH] Implementing ifunc target hook

2012-12-26 Thread Joseph S. Myers
On Wed, 26 Dec 2012, Alexander Ivchenko wrote: > The attached patch is indended to add the target hook for indicating > the support of ifunc on target. That's not a hook, it's a target macro. What is the rationale for this needing to be a target macro instead of a target hook? -- Joseph S. My

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

2012-12-26 Thread Jason Merrill
On 12/24/2012 09:23 AM, Dodji Seketeli wrote: + /* For and array decl decay_conversion should not try to return its "an array..." OK with that change. Jason

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

2012-12-26 Thread Jason Merrill
OK. Jason

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

2012-12-26 Thread Jason Merrill
On 12/24/2012 03:29 AM, Paolo Carlini wrote: Are you sure your patch handles the access control issue too?? (isn't obvious to me that it does, looking at the patch itself and your comments) Nope, you're right. I put the testcase in one file and then compiled a different one. /facepalm Jaso

[committed] Fix ICE in gen_reg_rtx, at emit-rtl.c:864/865 compiling GNU MPFR

2012-12-26 Thread John David Anglin
The attached change fixes PR target/5379. ICE occurs when reload tries to emit a move instruction containing a TLS symbol reference as the source operand. This requires several scratch registers. As a result, it isn't possible for a reload pattern to handle this case. So, the best solution was

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

2012-12-26 Thread Andi Kleen
> Can you please post a real-world example, where using %r15 would break > existing code? I used to run into problems like this when porting code to gcc from icc or VC. A lot of hyper optimized inline assembler snippets wants to use all registers and icc/VC support that. With gcc usually had to ad

Re: RFA: Fix ICE on PARALLEL returns when expand builtins

2012-12-26 Thread Andrew Pinski
On Sun, Dec 23, 2012 at 1:43 AM, Richard Sandiford wrote: > Some of the maths builtins can expand to a call followed by a bit > of postprocessing. With 4.8's PARALLEL return optimisations, these > embedded calls might return a PARALLEL of pseudos, but the postprocessing > isn't prepared to deal w

[PATCH] Support OpenMP for task parallelism on Android-ICS/GCC-4.6.3

2012-12-26 Thread Geunsik Lim
From: Geunsik Lim The patch supports OpenMP library to parallelize the existing C/C++ code rapidly/easily without any modification of android platform on multicore embedded devices. The original patch is made by Keith Obenschain (using GCC 4.4.3). . Archive - https://android-review.googlesource.

[PATCH] Support OpenMP for task parallelism on Android-ICS/GCC-4.7.2

2012-12-26 Thread Geunsik Lim
From: Geunsik Lim The patch supports OpenMP library to parallelize the existing C/C++ code (e.g: for task parallelism) rapidly/easily without any modification of android platform on multicore embedded devices. The original patch is made by Keith Obenschain (using GCC 4.4.3). . Archive - https://

Re: [PATCH] Function Multiversioning Bug, checking for function versions

2012-12-26 Thread Sriraman Tallam
I committed this patch with the fix. Thanks, -Sri. On Tue, Dec 18, 2012 at 4:13 PM, Diego Novillo wrote: > On Tue, Dec 18, 2012 at 6:38 PM, Sriraman Tallam wrote: > >> The function versions are now determined purely based on the string >> value and not on DECL_FUNCTION_SPECIFIC_TARGET fields. S

[PATCH, ARM, iWMMXT] Fix define_constants for WCGR

2012-12-26 Thread Xinyu Qi
Hi, It is necessary to sync the constants WCGR0 to WCGR3 in iwmmxt.md with the IWMMXT_GR_REGNUM in arm.h. ChangeLog * config/arm/arm.h (FIRST_IWMMXT_GR_REGNUM): Add comment. * config/arm/iwmmxt.md (WCGR0, WCGR1): Update. * config/arm/iwmmxt.md (WCGR2, WCGR3): Likewise.

[patch, fortran] PR48976 INQUIRE with STREAM= not supported

2012-12-26 Thread Jerry DeLisle
Greetings, The attached patch implements the missing INQUIRE(99, STREAM=str) functionality required by the Fortran 2008 Standard. Regression tested on x86-64. OK for trunk with test case from the PR? Regards, Jerry 2012-12-27 Jerry DeLisle PR fortran/48976 * gfortran.h