Re: RFC: Add ADDR_EXPR lowering (PR tree-optimization/66718)

2015-07-04 Thread Jakub Jelinek
On Fri, Jul 03, 2015 at 03:21:47PM +0200, Marek Polacek wrote: > This patch implements a new pass, called laddress, which deals with > lowering ADDR_EXPR assignments. Such lowering ought to help the > vectorizer, but it also could expose more CSE opportunities, maybe > help reassoc, etc. It's onl

[PATCH] Fix mips-{mti,img}-linux-gnu boot-strap

2015-07-04 Thread Bernd Edlinger
Hi,this patch fixes a regression that was triggered by commit r225260.See pr66747 for details.Is it OK for trunk?ThanksBernd. 2015-07-04 Bernd Edlinger PR target/66747 * config/mips/mips.c (mips_cfun_has_inflexible_gp_ref_p): Handle

RE: [PATCH] Fix mips-{mti,img}-linux-gnu boot-strap

2015-07-04 Thread Matthew Fortune
Bernd Edlinger writes: > Hi,this patch fixes a regression that was triggered by commit > r225260.See > pr66747 for details.Is it OK for > trunk?ThanksBernd. Thanks Bernd. I was just reviewing this PR. I think it will probably be safer to move the fix up to the mips_find_gp_ref and consider eac

Re: [PATCH] Fix mips-{mti,img}-linux-gnu boot-strap

2015-07-04 Thread Richard Sandiford
Bernd Edlinger writes: > --- gcc/config/mips/mips.c.jj 2015-06-08 23:06:50.0 +0200 > +++ gcc/config/mips/mips.c2015-07-03 13:16:02.637293167 +0200 > @@ -9902,8 +9902,11 @@ mips_cfun_has_inflexible_gp_ref_p (void) > static bool > mips_insn_has_flexible_gp_ref_p (rtx_insn *insn) > { >

Re: [PR66726] Factor conversion out of COND_EXPR

2015-07-04 Thread Bernhard Reutner-Fischer
On Sat, Jul 04, 2015 at 12:58:58PM +1000, Kugan wrote: > Please find a patch that attempt to FIX PR66726 by factoring conversion > out of COND_EXPR as explained in the PR. > > Bootstrapped and regression tested on x86-64-none-linux-gnu with no new > regressions. Is this OK for trunk? > > Thanks,

New Swedish PO file for 'gcc' (version 5.1.0)

2015-07-04 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-5.1.0.sv.po', has just

Re: [PATCH 1/2][ARM] PR/65956 AAPCS update for alignment attribute

2015-07-04 Thread Richard Biener
On July 3, 2015 10:43:30 PM GMT+02:00, Richard Earnshaw wrote: >On 03/07/15 19:24, Richard Biener wrote: >> On July 3, 2015 6:11:13 PM GMT+02:00, Richard Earnshaw > wrote: >>> On 03/07/15 16:26, Alan Lawrence wrote: These include tests of structs, scalars, and vectors - only general-pur

Re: [PATCH 1/2][ARM] PR/65956 AAPCS update for alignment attribute

2015-07-04 Thread Jakub Jelinek
On Sat, Jul 04, 2015 at 12:57:36PM +0200, Richard Biener wrote: > + if (!AGGREGATE_TYPE_P (type)) > +return TYPE_ALIGN (TYPE_MAIN_VARIANT (type)) > PARM_BOUNDARY; > + > + for (tree field = TYPE_FIELDS (type); field; field = DECL_CHAIN > >>> (field)) > +if (DECL_AL

RE: [PATCH] Fix mips-{mti,img}-linux-gnu boot-strap

2015-07-04 Thread Bernd Edlinger
Hi, On Sat, 4 Jul 2015 09:04:41, Richard Sandiford wrote: > > The final return here would also mishandle SEQUENCE PATTERNs. > The idea was that this function would only see "real" instructions, > so I think instead the FOR_EACH_SUBINSN should be here: > > static bool > mips_find_gp_ref (bool *cach

Re: [PR66726] Factor conversion out of COND_EXPR

2015-07-04 Thread Kugan
On 04/07/15 18:51, Bernhard Reutner-Fischer wrote: > On Sat, Jul 04, 2015 at 12:58:58PM +1000, Kugan wrote: >> Please find a patch that attempt to FIX PR66726 by factoring conversion >> out of COND_EXPR as explained in the PR. >> >> Bootstrapped and regression tested on x86-64-none-linux-gnu with n

RE: [Patch,tree-optimization]: Add new path Splitting pass on tree ssa representation

2015-07-04 Thread Ajit Kumar Agarwal
-Original Message- From: Richard Biener [mailto:richard.guent...@gmail.com] Sent: Tuesday, June 30, 2015 4:42 PM To: Ajit Kumar Agarwal Cc: l...@redhat.com; GCC Patches; Vinod Kathail; Shail Aditya Gupta; Vidhumouli Hunsigida; Nagaraju Mekala Subject: Re: [Patch,tree-optimization]: Add

Re: [PR66726] Factor conversion out of COND_EXPR

2015-07-04 Thread Bernhard Reutner-Fischer
On July 4, 2015 2:32:11 PM GMT+02:00, Kugan wrote: >On 04/07/15 18:51, Bernhard Reutner-Fischer wrote: >> On Sat, Jul 04, 2015 at 12:58:58PM +1000, Kugan wrote: >>> Please find a patch that attempt to FIX PR66726 by factoring >conversion >>> out of COND_EXPR as explained in the PR. >>> >>> Bootst

[committed] Remove condition from PA indirect_jump

2015-07-04 Thread John David Anglin
The attached change removes the C condition from the PA indirect jump since it depended on operands[] and this isn't generally allowed in named patterns. Tested on hppa-unknown-linux-gnu and hppa2.0w-hp-hpux11.11. Committed to trunk and active branches. Dave -- John David Anglin dave.ang

Re: RFC: Add ADDR_EXPR lowering (PR tree-optimization/66718)

2015-07-04 Thread Jakub Jelinek
On Sat, Jul 04, 2015 at 09:20:04AM +0200, Jakub Jelinek wrote: > On Fri, Jul 03, 2015 at 03:21:47PM +0200, Marek Polacek wrote: > > This patch implements a new pass, called laddress, which deals with > > lowering ADDR_EXPR assignments. Such lowering ought to help the > > vectorizer, but it also co

Re: RFC: Add ADDR_EXPR lowering (PR tree-optimization/66718)

2015-07-04 Thread Jakub Jelinek
On Fri, Jul 03, 2015 at 04:06:26PM +0200, Jakub Jelinek wrote: > In the pr59984.c testcase, with Marek's patch and this patch, one loop in > test is already vectorized (the ICE was on the other one), I'll work on > recognizing multiples of GOMP_SIMD_LANE () as linear next, so that we > vectorize al

Clean-ups in match.pd

2015-07-04 Thread Marc Glisse
Hello, these are just some minor changes. I believe I had already promised a build_ function to match integer_each_onep. Bootstrap+testsuite on powerpc64le-unknown-linux-gnu (it looks like *-match.c takes about 10 minutes to compile in stage2 these days). 2015-07-06 Marc Glisse *

Re: RFC: Add ADDR_EXPR lowering (PR tree-optimization/66718)

2015-07-04 Thread Richard Biener
On July 4, 2015 4:17:02 PM GMT+02:00, Jakub Jelinek wrote: >On Sat, Jul 04, 2015 at 09:20:04AM +0200, Jakub Jelinek wrote: >> On Fri, Jul 03, 2015 at 03:21:47PM +0200, Marek Polacek wrote: >> > This patch implements a new pass, called laddress, which deals with >> > lowering ADDR_EXPR assignments.

Re: [Ping, Patch, Fortran, PR58586, v5] ICE with derived type with allocatable component passed by value

2015-07-04 Thread Steve Kargl
On Fri, Jul 03, 2015 at 11:29:00AM +0200, Andre Vehreschild wrote: > Ping! > (Un)fortnuately you're working on an area of Fortran that I don't know and in parts of the tree that takes me a long time to decipher (aka, trans-*.c files). I applied your patch and see several failures. I'll note th

RE: [PATCH] Fix mips-{mti,img}-linux-gnu boot-strap

2015-07-04 Thread Matthew Fortune
Bernd Edlinger writes: > On Sat, 4 Jul 2015 09:04:41, Richard Sandiford wrote: > > > > The final return here would also mishandle SEQUENCE PATTERNs. > > The idea was that this function would only see "real" instructions, > > so I think instead the FOR_EACH_SUBINSN should be here: > > > > static bo

Re: [Ping, Patch, Fortran, PR58586, v5] ICE with derived type with allocatable component passed by value

2015-07-04 Thread Andre Vehreschild
Hi Steve, Thanks for looking at the code. The error you experience is known to me. The bug is present in gfortran and only exposed by this patch. Unfortunately is the pr58586 not addressing this specific error. It may be in the bugtracker under a different number already. Furthermore did I not

Re: [gomp] Move openacc vector& worker single handling to RTL

2015-07-04 Thread Nathan Sidwell
On 07/03/15 19:11, Jakub Jelinek wrote: On Fri, Jul 03, 2015 at 06:51:57PM -0400, Nathan Sidwell wrote: IMHO this is a step towards putting target-dependent handling in the target compiler and out of the more generic host-side compiler. The changelog is separated into 3 parts - a) general infra

Re: [PATCH] c/66516 - missing diagnostic on taking the address of a builtin function

2015-07-04 Thread Martin Sebor
I don't think c_validate_addressable is a good name - given that it's called for lots of things that aren't addressable, in contexts in which there is no need for them to be addressable, and doesn't do checks of addressability in contexts where they are actually needed and done elsewhere (e.g. che

[patch, committed] minor cleanup in nios2 backend

2015-07-04 Thread Sandra Loosemore
I've checked in the attached patch to clean up a few stylistic issues in the nios2 back end, e.g. switching to the new "rtx_insn *" type instead of "rtx" for things that are explicitly insns. There's no change in functionality. -Sandra 2015-07-04 Sandra Loosemore gcc/ * config/nios2/ni

[patch] [fixincludes] Ignore .DS_Store junk files when running make check

2015-07-04 Thread Eric Gallager
I've attached trivial, 1-line patch to fixincludes_check.tpl; it allows 'make check' to succeed on OS X, by ignoring the files that Finder creates to keep track of the status of directories. -Eric Gallager fixincludes/check.tpl | 1 + 1 file changed, 1 insertion(+) diff --git a/fixincludes/check