[PATCH, GCC/THUMB1] New define_insn_and_split pattern to enable optimizing out certain unnecessary uxtb instruction

2014-03-16 Thread Terry Guo
Hi The existing test case gcc.target/arm/unsigned-extend-1.c fails for Thumb1 target like cortex-m0 because the thumb1_addsi3_addgeu insn pattern isn't friendly to gcc combine pass. Before combine pass, we have such insn for this test case: (insn 10 9 12 2 (set (reg:SI 118) (plus:SI (plus

[PATCH,GCC/Thumb1] Correctly reset the variable after_arm_reorg for Thumb1 target

2014-03-16 Thread Terry Guo
Hi, I am working on another patch and found this per-function variable isn't correctly reset for Thumb1 target. Currently no ICE will be triggered because we don't call function arm_split_constants for Thumb1 target. This patch intends to define this variable in machine_function struct in arm.h. I

Re: libgo patch committed: Compile math library with -ffp-contract=off

2014-03-16 Thread Michael Hudson-Doyle
Michael Hudson-Doyle writes: > Ian Lance Taylor writes: > >> On Thu, Mar 13, 2014 at 6:27 PM, Michael Hudson-Doyle >> wrote: >>> Ian Lance Taylor writes: >>> The bug report http://golang.org/issue/7074 shows that math.Log2(1) produces the wrong result on Aarch64, because the Go math

[patch, libgfortran] PR46800 Handle CTRL-D correctly with STDIN

2014-03-16 Thread Jerry DeLisle
Hi all. The problem here was that when reading a value from STDIN and the user just entered an empty entry (LF), we would end up getting nested into a second read (via next_char) and the user would have to press CTRL-D twice to get out of the read. (The correct behavior is to only hit CTRL-D once

Re: [patch, fortran] Fix for PR 60522

2014-03-16 Thread Mikael Morin
Le 16/03/2014 11:41, Thomas Koenig a écrit : > Hello world, > > the attached patch fixes PR 60522, a regresseion where temporary > variables were incorrectly introduced in a BLOCK within a WHERE > statement. > > Regression-tested on x86_64-unknown-linux-gnu. > > OK for trunk and the other open

[PATCH] Fix PR c++/60390

2014-03-16 Thread Adam Butcher
PR c++/60390 * parser.c (cp_parser_member_declaration): Don't allow finish_fully_implicit_template to consider friend declarations to be class member templates. (synthesize_implicit_template_parm): Handling winding back through class scope to the clas

Re: [PATCH 1/4] [GOMP4] [Fortran] OpenACC 1.0+ support in fortran front-end

2014-03-16 Thread Thomas Schwinge
Hi! On Sun, 16 Mar 2014 20:23:48 +0100, Tobias Burnus wrote: > Can you or Thomas apply the following patch after my patch* has > been committed to the trunk and the branch has been updated? Please > mention PR fortran/60283 in the ChangeLog. > > * http://gcc.gnu.org/ml/gcc-patches/2014-03/msg0

Re: [PATCH 1/4] [GOMP4] [Fortran] OpenACC 1.0+ support in fortran front-end

2014-03-16 Thread Tobias Burnus
Ilmir Usmanov wrote: Committed as r208541. Thanks. Can you or Thomas apply the following patch after my patch* has been committed to the trunk and the branch has been updated? Please mention PR fortran/60283 in the ChangeLog. * http://gcc.gnu.org/ml/gcc-patches/2014-03/msg00782.html Tobias

[Patch, Fortran] PRs 60283/60543: Fix two wrong-code bugs related for implicit pure

2014-03-16 Thread Tobias Burnus
This patch fixes two issues, where gfortran claims that a function is implicit pure, but it is not. That will cause a wrong-code optimization in the middle end. First problem, cf. PR60543, is that implicit pure was not set to 0 for calls to impure intrinsic subroutines. (BTW: There are no impu

Re: Try to catch up _GLIBCXX_RESOLVE_LIB_DEFECTS comments and documentation.

2014-03-16 Thread Ed Smith-Rowland
On 03/16/2014 08:43 AM, Jonathan Wakely wrote: On 15 March 2014 14:46, Ed Smith-Rowland wrote: I'm resending this because I forgot to dupe to gcc-patches and I'd like one thread. This should be pure commentary and documentation. I hope I got all these. I grepped for DR and added _GLIBCXX_RESO

[build]-PING - Re: [Build, Driver] Add -lcilkrts for -fcilkplus

2014-03-16 Thread Tobias Burnus
PING for the build part; Joseph has already approved the driver part. Tobias Burnus wrote: When using Cilk Plus (-fcilkplus), it makes sense to automatically link the run-time library (-lcilkrts). This patch mimics libgomp by adding a .spec file; I am not 100% sure whether the .spec file is n

Re: Try to catch up _GLIBCXX_RESOLVE_LIB_DEFECTS comments and documentation.

2014-03-16 Thread Jonathan Wakely
On 15 March 2014 14:46, Ed Smith-Rowland wrote: > I'm resending this because I forgot to dupe to gcc-patches and I'd like one > thread. > > This should be pure commentary and documentation. > > I hope I got all these. I grepped for DR and added > _GLIBCXX_RESOLVE_LIB_DEFECTS where it seemed needed

Re: [ada, build] host/target configuration

2014-03-16 Thread Andreas Schwab
Thomas Schwinge writes: > No, you did not, but I had said: »not obvious [...] where the > gcc/ada/gcc-interface/ code is using [target_cpu_default]«. Why do you think this directory is called gcc-interface??? Andreas. -- Andreas Schwab, sch...@linux-m68k.org GPG Key fingerprint = 58CA 54C7 6D

[PATCH][AARCH64]combine "ubfiz" and "orr" with bfi when certain condition meets.

2014-03-16 Thread Renlin Li
Hi all, Thank you for your suggestions, Richard. I have updated the patch accordingly. This is an optimization patch which will combine "ubfiz" and "orr" insns with a single "bfi" when certain conditions meet. tmp = (x & m) | ( (y & n) << lsb) can be presented using and tmp, x, m

[PATCH v2] SPARC: Clarify -mapp-regs option

2014-03-16 Thread Sebastian Huber
gcc/ChangeLog 2014-03-16 Sebastian Huber * doc/invoke.texi (mapp-regs): Clarify. --- gcc/doc/invoke.texi | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi index 24bd76e..485867d 100644 --- a/gcc/doc/invoke.texi +++ b/gcc/d

Re: [ada, build] host/target configuration

2014-03-16 Thread Thomas Schwinge
Hi! On Sun, 16 Mar 2014 12:09:59 +0100, Andreas Schwab wrote: > Thomas Schwinge writes: > > On Sun, 16 Mar 2014 11:30:06 +0100, Andreas Schwab > > wrote: > >> Thomas Schwinge writes: > >> > >> > It is not obvious to me (and neither was to Olivier as well as the patch > >> > reviewers, as it

[PATCH][AARCH64]Amend AArch64 frame layout comment.

2014-03-16 Thread Renlin Li
Hi all, This is a simple patch to update the AArch64 frame layout comment in the source code. frame_pointer should point above the local_variables section as we define FRAME_GROWS_DOWNWARD = 1. Is this Okay for stage-4? Regards, Renlin Li gcc/ChangeLog: 2014-03-16 Renlin Li * con

Re: [ada, build] host/target configuration

2014-03-16 Thread Andreas Schwab
Thomas Schwinge writes: > Hi! > > On Sun, 16 Mar 2014 11:30:06 +0100, Andreas Schwab > wrote: >> Thomas Schwinge writes: >> >> > It is not obvious to me (and neither was to Olivier as well as the patch >> > reviewers, as it seems) where the gcc/ada/gcc-interface/ code is using >> > this varia

Re: [ada, build] host/target configuration

2014-03-16 Thread Thomas Schwinge
Hi! On Sun, 16 Mar 2014 11:30:06 +0100, Andreas Schwab wrote: > Thomas Schwinge writes: > > > It is not obvious to me (and neither was to Olivier as well as the patch > > reviewers, as it seems) where the gcc/ada/gcc-interface/ code is using > > this variable. Is there a way for this to be ma

[patch, fortran] Fix for PR 60522

2014-03-16 Thread Thomas Koenig
Hello world, the attached patch fixes PR 60522, a regresseion where temporary variables were incorrectly introduced in a BLOCK within a WHERE statement. Regression-tested on x86_64-unknown-linux-gnu. OK for trunk and the other open branches? Thomas 2014-04-16 Thomas Koenig

Re: [ada, build] host/target configuration

2014-03-16 Thread Andreas Schwab
Thomas Schwinge writes: > It is not obvious to me (and neither was to Olivier as well as the patch > reviewers, as it seems) where the gcc/ada/gcc-interface/ code is using > this variable. Is there a way for this to be made more explicit, so in > the future it won't get removed as unused, again?

Re: [ada, build] host/target configuration

2014-03-16 Thread Thomas Schwinge
Hi! On Sun, 16 Mar 2014 09:32:33 +0100, Andreas Schwab wrote: > Thomas Schwinge writes: was the final version of the patch, after Olivier had taken it over from me. > > --- gcc/ada/gcc-i

New Swedish PO file for 'gcc' (version 4.9-b20140202)

2014-03-16 Thread Translation Project Robot
Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'gcc' has been submitted by the Swedish team of translators. The file is available at: http://translationproject.org/latest/gcc/sv.po (This file, 'gcc-4.9-b20140202.sv.po',

Re: [ada, build] host/target configuration

2014-03-16 Thread Andreas Schwab
Thomas Schwinge writes: > diff --git gcc/ada/gcc-interface/Makefile.in gcc/ada/gcc-interface/Makefile.in > index eeb8c7f..c07722b 100644 > --- gcc/ada/gcc-interface/Makefile.in > +++ gcc/ada/gcc-interface/Makefile.in > @@ -151,12 +151,15 @@ GCC_CFLAGS = $(INTERNAL_CFLAGS) $(T_CFLAGS) $(CFLAGS) >