Re: OpenACC Firstprivate

2015-11-11 Thread Jakub Jelinek
On Tue, Nov 10, 2015 at 09:12:55AM -0500, Nathan Sidwell wrote: > + /* Create a local object to hold the instance > +value. */ > + tree inst = create_tmp_var > + (TREE_TYPE (TREE_TYPE (new_var)), > +IDENTIFIE

[PATCH] gcc: configure: fix test == bashisms

2015-11-11 Thread Mike Frysinger
--- gcc/ChangeLog| 5 + gcc/configure| 4 ++-- gcc/configure.ac | 4 ++-- 3 files changed, 9 insertions(+), 4 deletions(-) diff --git a/gcc/ChangeLog b/gcc/ChangeLog index ea15ada..e3a0432 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,8 @@ +2015-11-10 Mike Frysinger + +

Re: [OpenACC] declare directive

2015-11-11 Thread Jakub Jelinek
On Mon, Nov 09, 2015 at 05:11:44PM -0600, James Norris wrote: > diff --git a/gcc/c-family/c-pragma.h b/gcc/c-family/c-pragma.h > index 953c4e3..c6a2981 100644 > --- a/gcc/c-family/c-pragma.h > +++ b/gcc/c-family/c-pragma.h > @@ -30,6 +30,7 @@ enum pragma_kind { >PRAGMA_OACC_ATOMIC, >PRAGMA_

RE: [RFC][PATCH] Preferred rename register in regrename pass

2015-11-11 Thread Robert Suchanek
Hi, > I guess this is ok to stop the failures for now, but you may want to > move the check to the point where we set terminated_this_insn. Also, as > I pointed out earlier, clearing terminated_this_insn should probably > happen earlier. Here is the updated patch that I'm about to commit once the

[PATCH 03/N] Just another set of memory leaks

2015-11-11 Thread Martin Liška
Hi. There are new fixed for memory leaks, where the following: ==19826== 21 bytes in 1 blocks are definitely lost in loss record 16 of 625 ==19826==at 0x4C2A00F: malloc (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so) ==19826==by 0x16868D7: xmalloc (xmalloc.c:148) ==19826==

Re: [PATCH] Simple optimization for MASK_STORE.

2015-11-11 Thread Richard Biener
On Tue, Nov 10, 2015 at 3:56 PM, Ilya Enkovich wrote: > 2015-11-10 17:46 GMT+03:00 Richard Biener : >> On Tue, Nov 10, 2015 at 1:48 PM, Ilya Enkovich >> wrote: >>> 2015-11-10 15:33 GMT+03:00 Richard Biener : On Fri, Nov 6, 2015 at 2:28 PM, Yuri Rumyantsev wrote: > Richard, > >

Re: Enable pointer TBAA for LTO

2015-11-11 Thread Richard Biener
On Tue, 10 Nov 2015, Jan Hubicka wrote: > > > Index: tree.c > > > === > > > --- tree.c(revision 229968) > > > +++ tree.c(working copy) > > > @@ -13198,6 +13198,7 @@ gimple_canonical_types_compatible_p (con > > >/*

[Patch] PR tree-optimization/68234 Improve range info for loop Phi node

2015-11-11 Thread Jiong Wang
As discussed at https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68234, this patch haven't touch the existed code logic in vrp_visit_phi_node, it only entends SCEV check to those VR_VARYING loop PHI node. Previously, we only do this check if the PHI node is with valid range info but later dropped eit

[patch] libstdc++/56158 Extend valid values of iostream bitmask types

2015-11-11 Thread Jonathan Wakely
As described in the PR, we have operator~ overloads defined for enumeration types which produce values outside the range of valid values for the type. In C++11 that can be trivially solved by giving the enumeration types a fixed underlying type, but this code needs to be valid in C++03 too. This

Re: [OpenACC] declare directive

2015-11-11 Thread Thomas Schwinge
Hi! On Wed, 11 Nov 2015 09:32:33 +0100, Jakub Jelinek wrote: > On Mon, Nov 09, 2015 at 05:11:44PM -0600, James Norris wrote: > > diff --git a/gcc/c-family/c-pragma.h b/gcc/c-family/c-pragma.h > > index 953c4e3..c6a2981 100644 > > --- a/gcc/c-family/c-pragma.h > > +++ b/gcc/c-family/c-pragma.h > >

[patch] libstdc++/64651 allow rethrow_exception to be found by ADL

2015-11-11 Thread Jonathan Wakely
As I wrote in the PR, the standard doesn't require that std::rethrow_exception can be found by ADL, because exception_ptr is not necessarily defined in namespace std. This ensures it will be found. Tested powerpc64le-linux, committed to trunk. commit 920a18f991d3604bf2dfdf9679411b012964f23d Autho

Re: Short-cut generation of simple built-in functions

2015-11-11 Thread Richard Biener
On Tue, Nov 10, 2015 at 10:24 PM, Richard Sandiford wrote: > Richard Biener writes: >> On Sat, Nov 7, 2015 at 2:31 PM, Richard Sandiford >> wrote: >>> This patch short-circuits the builtins.c expansion code for a particular >>> gimple call if: >>> >>> - the function has an associated internal fu

Re: [PATCH 03/N] Just another set of memory leaks

2015-11-11 Thread Richard Biener
On Wed, Nov 11, 2015 at 10:04 AM, Martin Liška wrote: > Hi. > > There are new fixed for memory leaks, where the following: > > ==19826== 21 bytes in 1 blocks are definitely lost in loss record 16 of 625 > ==19826==at 0x4C2A00F: malloc (in > /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.s

Re: [Patch] PR tree-optimization/68234 Improve range info for loop Phi node

2015-11-11 Thread Richard Biener
On Wed, 11 Nov 2015, Jiong Wang wrote: > As discussed at https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68234, this > patch haven't touch the existed code logic in vrp_visit_phi_node, it > only entends SCEV check to those VR_VARYING loop PHI node. > > Previously, we only do this check if the PHI no

[PATCH, committed] PR67826 gcc/fortran/openmp.c:1808: bad test ?

2015-11-11 Thread Dominique d'Humières
I have committed on trunk the following patch as revision r230148 (preapproved by Jakub Jelinek and tested on x86_64-apple-darwin14) Index: gcc/fortran/ChangeLog === --- gcc/fortran/ChangeLog (revision 230147) +++ gcc/fortran/C

Re: [PATCH][AArch64][v2] Improve comparison with complex immediates followed by branch/cset

2015-11-11 Thread Kyrill Tkachov
Ping. https://gcc.gnu.org/ml/gcc-patches/2015-11/msg00233.html Thanks, Kyrill On 03/11/15 15:43, Kyrill Tkachov wrote: Hi all, This patch slightly improves sequences where we want to compare against a complex immediate and branch against the result or perform a cset on it. This means transform

Re: [PATCH, 1/16] Insert new exit block only when needed in transform_to_exit_first_loop_alt

2015-11-11 Thread Richard Biener
On Mon, 9 Nov 2015, Tom de Vries wrote: > On 09/11/15 16:35, Tom de Vries wrote: > > Hi, > > > > this patch series for stage1 trunk adds support to: > > - parallelize oacc kernels regions using parloops, and > > - map the loops onto the oacc gang dimension. > > > > The patch series contains thes

Re: [PATCH, 2/16] Make create_parallel_loop return void

2015-11-11 Thread Richard Biener
On Mon, 9 Nov 2015, Tom de Vries wrote: > On 09/11/15 16:35, Tom de Vries wrote: > > Hi, > > > > this patch series for stage1 trunk adds support to: > > - parallelize oacc kernels regions using parloops, and > > - map the loops onto the oacc gang dimension. > > > > The patch series contains thes

Re: [PATCH, 4/16] Implement -foffload-alias

2015-11-11 Thread Richard Biener
On Mon, 9 Nov 2015, Tom de Vries wrote: > On 09/11/15 16:35, Tom de Vries wrote: > > Hi, > > > > this patch series for stage1 trunk adds support to: > > - parallelize oacc kernels regions using parloops, and > > - map the loops onto the oacc gang dimension. > > > > The patch series contains thes

Re: [PATCH, 5/16] Add in_oacc_kernels_region in struct loop

2015-11-11 Thread Richard Biener
On Mon, 9 Nov 2015, Tom de Vries wrote: > On 09/11/15 16:35, Tom de Vries wrote: > > Hi, > > > > this patch series for stage1 trunk adds support to: > > - parallelize oacc kernels regions using parloops, and > > - map the loops onto the oacc gang dimension. > > > > The patch series contains thes

Re: [PATCH, 6/16] Add pass_oacc_kernels

2015-11-11 Thread Richard Biener
On Mon, 9 Nov 2015, Tom de Vries wrote: > On 09/11/15 16:35, Tom de Vries wrote: > > Hi, > > > > this patch series for stage1 trunk adds support to: > > - parallelize oacc kernels regions using parloops, and > > - map the loops onto the oacc gang dimension. > > > > The patch series contains thes

Re: [PATCH, 4/16] Implement -foffload-alias

2015-11-11 Thread Jakub Jelinek
On Wed, Nov 11, 2015 at 11:51:02AM +0100, Richard Biener wrote: > > The option -foffload-alias=pointer instructs the compiler to assume that > > objects references in an offload region do not alias. > > > > The option -foffload-alias=all instructs the compiler to make no > > assumptions about alia

Re: [PATCH, 10/16] Add pass_oacc_kernels pass group in passes.def

2015-11-11 Thread Richard Biener
On Mon, 9 Nov 2015, Tom de Vries wrote: > On 09/11/15 16:35, Tom de Vries wrote: > > Hi, > > > > this patch series for stage1 trunk adds support to: > > - parallelize oacc kernels regions using parloops, and > > - map the loops onto the oacc gang dimension. > > > > The patch series contains thes

Re: [PATCH, 11/16] Update testcases after adding kernels pass group

2015-11-11 Thread Richard Biener
On Mon, 9 Nov 2015, Tom de Vries wrote: > On 09/11/15 16:35, Tom de Vries wrote: > > Hi, > > > > this patch series for stage1 trunk adds support to: > > - parallelize oacc kernels regions using parloops, and > > - map the loops onto the oacc gang dimension. > > > > The patch series contains thes

Re: [PATCH, 7/16] Add pass_dominator_oacc_kernels

2015-11-11 Thread Richard Biener
On Mon, 9 Nov 2015, Tom de Vries wrote: > On 09/11/15 16:35, Tom de Vries wrote: > > Hi, > > > > this patch series for stage1 trunk adds support to: > > - parallelize oacc kernels regions using parloops, and > > - map the loops onto the oacc gang dimension. > > > > The patch series contains thes

[PATCH] Fix PR rtl-optimization/68287

2015-11-11 Thread Martin Liška
Hi. There's a fix for fallout of r230027. Patch can bootstrap and survives regression tests on x86_64-linux-gnu. Ready for trunk? Thanks, Martin >From 127d629991d92ea42a87b84e9d88612b84dbec03 Mon Sep 17 00:00:00 2001 From: marxin Date: Wed, 11 Nov 2015 10:11:20 +0100 Subject: [PATCH 1/2] Fix PR

Re: [PATCH 4b/4] [ARM] PR63870 Remove error for invalid lane numbers

2015-11-11 Thread Kyrill Tkachov
Hi Charles, On 08/11/15 00:26, charles.bay...@linaro.org wrote: From: Charles Baylis Charles Baylis * config/arm/neon.md (neon_vld1_lane): Remove error for invalid lane number. (neon_vst1_lane): Likewise. (neon_vld2_lane): Likewise. (neon_vst2_lane)

[patch] Fix PR target/67265

2015-11-11 Thread Eric Botcazou
Hi, this is an ICE on an asm statement requiring a lot of registers, when compiled in 32-bit mode on x86/Linux with -O -fstack-check -fPIC: pr67265.c:10:3: error: 'asm' operand has impossible constraints The issue is that, since stack checking defines STACK_CHECK_MOVING_SP on this platform, th

Re: libgo patch committed: Update to Go 1.5 release

2015-11-11 Thread Rainer Orth
Ian Lance Taylor writes: > On Sun, Nov 8, 2015 at 9:21 AM, Rainer Orth > wrote: >> >> There were two remaining problems: >> >> * Before Solaris 12, sendfile only lives in libsendfile. This lead to >> link failures in gotools. >> >> * Solaris 12 introduced a couple more types that use _in6_ad

[gomp4] Merge trunk r230082 (2015-11-10) into gomp-4_0-branch

2015-11-11 Thread Thomas Schwinge
Hi! Committed to gomp-4_0-branch in r230154: commit 1fe1fa3a7b9d4286630cd286e0a52abe2d11e955 Merge: 02d9df1 76e711b Author: tschwinge Date: Wed Nov 11 11:43:09 2015 + svn merge -r 230048:230082 svn+ssh://gcc.gnu.org/svn/gcc/trunk git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc

Re: [patch] Fix PR target/67265

2015-11-11 Thread Bernd Schmidt
On 11/11/2015 12:38 PM, Eric Botcazou wrote: this is an ICE on an asm statement requiring a lot of registers, when compiled in 32-bit mode on x86/Linux with -O -fstack-check -fPIC: pr67265.c:10:3: error: 'asm' operand has impossible constraints The issue is that, since stack checking defines ST

Re: [ptx] partitioning optimization

2015-11-11 Thread Bernd Schmidt
On 11/10/2015 11:33 PM, Nathan Sidwell wrote: I've committed this patch to trunk. It implements a partitioning optimization for a loop partitioned over both vector and worker axes. We can elide the inner vector partitioning state propagation, if there are no intervening instructions in the worke

Re: [PATCH 4b/4] [ARM] PR63870 Remove error for invalid lane numbers

2015-11-11 Thread Charles Baylis
On 11 November 2015 at 11:22, Kyrill Tkachov wrote: > Hi Charles, > > On 08/11/15 00:26, charles.bay...@linaro.org wrote: >> >> From: Charles Baylis >> >> Charles Baylis >> >> * config/arm/neon.md (neon_vld1_lane): Remove error for >> invalid >> lane number. >> (neon_v

Re: [PATCH] PR67305, tighten neon_vector_mem_operand on eliminable registers

2015-11-11 Thread Jiong Wang
On 04/11/15 09:45, Jiong Wang wrote: As discussed at the bugzilla https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67305 neon_vector_mem_operand is broken. As the comments says "/* Reject eliminable registers. */", the code block at the head of this function which checks eliminable registers i

Re: [PATCH 4b/4] [ARM] PR63870 Remove error for invalid lane numbers

2015-11-11 Thread Kyrill Tkachov
On 11/11/15 12:08, Charles Baylis wrote: On 11 November 2015 at 11:22, Kyrill Tkachov wrote: Hi Charles, On 08/11/15 00:26, charles.bay...@linaro.org wrote: From: Charles Baylis Charles Baylis * config/arm/neon.md (neon_vld1_lane): Remove error for invalid lane numbe

Re: [PATCH] Fix PR rtl-optimization/68287

2015-11-11 Thread Richard Biener
On Wed, Nov 11, 2015 at 12:18 PM, Martin Liška wrote: > Hi. > > There's a fix for fallout of r230027. > > Patch can bootstrap and survives regression tests on x86_64-linux-gnu. Hmm, but only the new elements are zeroed so this still is different from previous behavior. Note that the previous .cre

Re: [PATCH] PR67305, tighten neon_vector_mem_operand on eliminable registers

2015-11-11 Thread Ramana Radhakrishnan
On 04/11/15 09:45, Jiong Wang wrote: > As discussed at the bugzilla > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67305 > > neon_vector_mem_operand is broken. As the comments says > "/* Reject eliminable registers. */", the code block at the head > of this function which checks eliminabl

[Patch] Optimize condition reductions where the result is an integer induction variable

2015-11-11 Thread Alan Hayward
Hi, I hoped to post this in time for Monday’s cut off date, but circumstances delayed me until today. Hoping if possible this patch will still be able to go in. This patch builds upon the change for PR65947, and reduces the amount of code produced in a vectorized condition reduction where operand

Re: [C/C++ PATCH] Reject declarators with huge arrays (PR c/68107, c++/68266)

2015-11-11 Thread Marek Polacek
On Tue, Nov 10, 2015 at 06:38:31PM +0100, Paolo Carlini wrote: > Hi, > > On 11/10/2015 05:36 PM, Marek Polacek wrote: > >+ > >+/* Did array size calculations overflow or does the array > >+ cover more than half of the address-space? */ > >+if (COMPLETE_TYPE_

RE: [PATCH 1/2][ARC] Add support for ARCv2 CPUs

2015-11-11 Thread Claudiu Zissulescu
This patch is committed (without the gen_compare_reg change). Thanks Joern, Claudiu > Apart from the gen_compare_reg change, the patch is OK. > If the v2 support mostly works like support for the other subtargets, you may > check it in without the gen_compare_reg change. > If that change is requ

Re: [patch] Fix PR target/67265

2015-11-11 Thread Eric Botcazou
> This piece of code along doesn't tell me exactly why the frame pointer > is needed. I was looking for an explicit use, but I now guess that if > you have multiple adjusts of the [stack] pointer you can't easily undo > them in the error case (the function behaves as-if using alloca). Is > that it?

RE: [PATCH 2/2][ARC] Add support for ARCv2 CPUs

2015-11-11 Thread Claudiu Zissulescu
This patch is committed. Thanks Joern, Claudiu > -Original Message- > From: Joern Wolfgang Rennecke [mailto:g...@amylaar.uk] > Sent: Tuesday, November 10, 2015 3:02 PM > To: Claudiu Zissulescu; gcc-patches@gcc.gnu.org > Cc: Francois Bedard; jeremy.benn...@embecosm.com > Subject: Re: [PATC

Re: [patch] Fix PR target/67265

2015-11-11 Thread Bernd Schmidt
On 11/11/2015 01:31 PM, Eric Botcazou wrote: Yes, it probably should, thanks for spotting it, revised patch attached. PR target/67265 * ira.c (ira_setup_eliminable_regset): Do not necessarily create the frame pointer for stack checking if non-call exceptions aren't us

Re: [C/C++ PATCH] Reject declarators with huge arrays (PR c/68107, c++/68266)

2015-11-11 Thread Marek Polacek
On Tue, Nov 10, 2015 at 12:40:49PM -0700, Martin Sebor wrote: > On 11/10/2015 09:36 AM, Marek Polacek wrote: > >While both C and C++ FEs are able to reject e.g. > >int a[__SIZE_MAX__ / sizeof(int)]; > >they are accepting code such as > >int (*a)[__SIZE_MAX__ / sizeof(int)]; > > > >As Joseph pointed

Re: [PATCH] Fix PR rtl-optimization/68287

2015-11-11 Thread Martin Liška
On 11/11/2015 01:20 PM, Richard Biener wrote: > On Wed, Nov 11, 2015 at 12:18 PM, Martin Liška wrote: >> Hi. >> >> There's a fix for fallout of r230027. >> >> Patch can bootstrap and survives regression tests on x86_64-linux-gnu. > > Hmm, but only the new elements are zeroed so this still is diff

Re: [C/C++ PATCH] Reject declarators with huge arrays (PR c/68107, c++/68266)

2015-11-11 Thread Bernd Schmidt
On 11/11/2015 01:31 PM, Marek Polacek wrote: Certainly I'm in favor of sharing code between C and C++ FEs, though in this case it didn't seem too important/obvious, because of the extra != error_mark_node check + I don't really like the new function getting *type and setting it there. Make it

Re: Enable pointer TBAA for LTO

2015-11-11 Thread Bernd Schmidt
On 11/11/2015 10:21 AM, Richard Biener wrote: On Tue, 10 Nov 2015, Jan Hubicka wrote: The reason is that TYPE_CANONICAL is initialized in get_alias_set that may be called before we finish all merging and then it is more fine grained than what we need here (i.e. TYPE_CANONICAL of pointers to two

[PATCH] PR68271 [6 Regression] Boostrap fails on x86_64-apple-darwin14 at r230084

2015-11-11 Thread Dominique d'Humières
The following patch restore bootstrap on darwin --- ../_clean/gcc/cp/parser.h 2015-11-10 01:54:44.0 +0100 +++ gcc/cp/parser.h 2015-11-11 12:10:28.0 +0100 @@ -48,7 +48,7 @@ struct GTY (()) cp_token { /* Token flags. */ unsigned char flags; /* Identifier for the pragm

Re: [PATCH] Simple optimization for MASK_STORE.

2015-11-11 Thread Yuri Rumyantsev
Richard, What we should do to cope with this problem (structure size increasing)? Should we return to vector comparison version? Thanks. Yuri. 2015-11-11 12:18 GMT+03:00 Richard Biener : > On Tue, Nov 10, 2015 at 3:56 PM, Ilya Enkovich wrote: >> 2015-11-10 17:46 GMT+03:00 Richard Biener : >>> O

Re: [PATCH] PR68271 [6 Regression] Boostrap fails on x86_64-apple-darwin14 at r230084

2015-11-11 Thread Jakub Jelinek
On Wed, Nov 11, 2015 at 02:11:38PM +0100, Dominique d'Humières wrote: > The following patch restore bootstrap on darwin > > --- ../_clean/gcc/cp/parser.h 2015-11-10 01:54:44.0 +0100 > +++ gcc/cp/parser.h 2015-11-11 12:10:28.0 +0100 > @@ -48,7 +48,7 @@ struct GTY (()) cp_token { >

Re: libgo patch committed: Update to Go 1.5 release

2015-11-11 Thread Ian Lance Taylor
On Wed, Nov 11, 2015 at 3:48 AM, Rainer Orth wrote: > Ian Lance Taylor writes: > >> On Sun, Nov 8, 2015 at 9:21 AM, Rainer Orth >> wrote: >>> >>> There were two remaining problems: >>> >>> * Before Solaris 12, sendfile only lives in libsendfile. This lead to >>> link failures in gotools. >>>

Re: [Patch] Optimize condition reductions where the result is an integer induction variable

2015-11-11 Thread Richard Biener
On Wed, Nov 11, 2015 at 1:22 PM, Alan Hayward wrote: > Hi, > I hoped to post this in time for Monday’s cut off date, but circumstances > delayed me until today. Hoping if possible this patch will still be able > to go in. > > > This patch builds upon the change for PR65947, and reduces the amount

Re: [ptx] partitioning optimization

2015-11-11 Thread Nathan Sidwell
On 11/10/15 17:45, Ilya Verbin wrote: I've been unable to introduce a testcase for this. The difficulty is we want to check an rtl dump from the acceleration compiler, and there doesn't appear to be existing machinery for that in the testsuite. Perhaps something to be added later? I haven't tr

Re: [gomp4] Fix some broken tests

2015-11-11 Thread Nathan Sidwell
On 11/10/15 18:08, Cesar Philippidis wrote: On 11/10/2015 12:35 PM, Nathan Sidwell wrote: I've committed this to gomp4. In preparing the reworked firstprivate patch changes for gomp4's gimplify.c I discovered these testcases were passing by accident, and lacked a data clause. It used to be i

RE: [PATCH, Mips] Compact branch/delay slot optimization.

2015-11-11 Thread Simon Dardis
Committed as r230160. Thanks, Simon > -Original Message- > From: Moore, Catherine [mailto:catherine_mo...@mentor.com] > Sent: 28 October 2015 14:00 > To: Simon Dardis; Matthew Fortune > Cc: gcc-patches@gcc.gnu.org > Subject: RE: [PATCH, Mips] Compact branch/delay slot optimization. > >

[PATCH] Fix PR58497 testcase for SPARC

2015-11-11 Thread Richard Biener
SPARC doesn't have vector support in this testcase and no integer multiplication. The general scalarization support fails to fold generated stmts so the following just does what other parts of the lowering do - factor in constants/constructors. On another note I noticed a tree sharing issue (mit

Re: OpenACC Firstprivate

2015-11-11 Thread Nathan Sidwell
On 11/11/15 03:04, Jakub Jelinek wrote: On Tue, Nov 10, 2015 at 09:12:55AM -0500, Nathan Sidwell wrote: + /* Create a local object to hold the instance + value. */ + tree inst = create_tmp_var + (TREE_TYPE (TREE_TYPE (

[gomp4] Rework gimplifyier region flags

2015-11-11 Thread Nathan Sidwell
I've committed this patch to gomp4 to remove the openacc-specific enums from gimplify_omp_ctx. Instead extending the existing omp_region_type enum. A similar patch will shortly be applied to trunk, now Jakub.s approved it. If you had patches relying on the old scheme, you'll need to update

Re: [ptx] partitioning optimization

2015-11-11 Thread Nathan Sidwell
On 11/11/15 07:06, Bernd Schmidt wrote: On 11/10/2015 11:33 PM, Nathan Sidwell wrote: I've committed this patch to trunk. It implements a partitioning optimization for a loop partitioned over both vector and worker axes. We can elide the inner vector partitioning state propagation, if there are

[PATCH, HSA] fix emission of HSAIL for builtins

2015-11-11 Thread Martin Liška
Hello. Following patch has been just applied to HSA branch and is responsible for correct emission of builtins. As HSAIL can support approximation for builtins like 'sin', we emit these if unsafe_math_optimization flag is enabled. Otherwise direct call instructions are emitted. I would like to in

Re: [PATCH] PR68271 [6 Regression] Boostrap fails on x86_64-apple-darwin14 at r230084

2015-11-11 Thread Dominique d'Humières
Is the following OK? Index: gcc/ChangeLog === --- gcc/ChangeLog (revision 230162) +++ gcc/ChangeLog (working copy) @@ -1,3 +1,10 @@ +2015-11-11 Dominique d'Humieres + + PR bootstrap/68271 + * cp/parser.h (cp

Re: [PATCH 1/2] simplify-rtx: Simplify trunc of and of shiftrt

2015-11-11 Thread Segher Boessenkool
On Tue, Nov 10, 2015 at 10:04:30PM +0100, Bernd Schmidt wrote: > On 11/10/2015 06:44 PM, Segher Boessenkool wrote: > > >Yes I know. All the rest of the code around is it like this though. > >Do you want this written in a saner way? > > I won't object to leaving it as-is for now, but in the futur

Re: [v3 PATCH] LWG 2510, make the default constructors of library tag types explicit.

2015-11-11 Thread Jonathan Wakely
On 10/11/15 22:01 +0200, Ville Voutilainen wrote: LWG 2510, make the default constructors of library tag types explicit. * include/bits/mutex.h (defer_lock_t, try_lock_t, adopt_lock_t): Add an explicit default constructor. * include/bits/stl_pair.h (piecewise_construct_t): Likewise

Re: [PATCH] PR68271 [6 Regression] Boostrap fails on x86_64-apple-darwin14 at r230084

2015-11-11 Thread Jakub Jelinek
On Wed, Nov 11, 2015 at 03:10:37PM +0100, Dominique d'Humières wrote: > Is the following OK? > > Index: gcc/ChangeLog > === > --- gcc/ChangeLog (revision 230162) > +++ gcc/ChangeLog (working copy) > @@ -1,3 +1,10 @@ > +2015-11

Re: [ptx] partitioning optimization

2015-11-11 Thread Bernd Schmidt
On 11/11/2015 02:59 PM, Nathan Sidwell wrote: That's not the problem. How to conditionally enable the test is the difficulty. I suspect porting something concerning accel_compiler from the libgomp testsuite is needed? Maybe a check_effective_target_offload_nvptx which tries to see if -fofflo

[PATCH] More compile-time saving in BB vectorization

2015-11-11 Thread Richard Biener
This saves some more compile-time avoiding vector size iteration for trivial fails. It also improves time spent by not giving up completely for all SLP instances if one fails to vectorize because of alignment issues. And it sneaks in a correctness fix for a previous change. Bootstrapped on x86_

Re: [PATCH v4] SH FDPIC backend support

2015-11-11 Thread Oleg Endo
On Tue, 2015-11-10 at 15:07 -0500, Rich Felker wrote: > > The way libcalls are now emitted is a bit unhandy. If more special > > -ABI > > libcalls are to be added in the future, they all have to do the jsr > > vs. > > bsrf handling (some potential candidates for new libcalls are > > optimized > >

Re: [C/C++ PATCH] Reject declarators with huge arrays (PR c/68107, c++/68266)

2015-11-11 Thread Marek Polacek
On Tue, Nov 10, 2015 at 04:48:13PM -0700, Jeff Law wrote: > Someone (I can't recall who) suggested the overflow check ought to be > shared, I agree. Can you factor out that check, shove it into c-family/ and > call it from the C & C++ front-ends? > > Approved with that change. Please post it her

Re: [gomp4] Fix some broken tests

2015-11-11 Thread Cesar Philippidis
On 11/11/2015 05:40 AM, Nathan Sidwell wrote: > On 11/10/15 18:08, Cesar Philippidis wrote: >> On 11/10/2015 12:35 PM, Nathan Sidwell wrote: >>> I've committed this to gomp4. In preparing the reworked firstprivate >>> patch changes for gomp4's gimplify.c I discovered these testcases were >>> pass

Re: [C/C++ PATCH] Reject declarators with huge arrays (PR c/68107, c++/68266)

2015-11-11 Thread Marek Polacek
On Wed, Nov 11, 2015 at 01:42:04PM +0100, Bernd Schmidt wrote: > On 11/11/2015 01:31 PM, Marek Polacek wrote: > > >Certainly I'm in favor of sharing code between C and C++ FEs, though in > >this case it didn't seem too important/obvious, because of the extra != > >error_mark_node check + I don't r

Re: [PATCH v4] SH FDPIC backend support

2015-11-11 Thread Rich Felker
On Wed, Nov 11, 2015 at 11:36:26PM +0900, Oleg Endo wrote: > On Tue, 2015-11-10 at 15:07 -0500, Rich Felker wrote: > > > > The way libcalls are now emitted is a bit unhandy. If more special > > > -ABI > > > libcalls are to be added in the future, they all have to do the jsr > > > vs. > > > bsrf h

Re: improved RTL-level if conversion using scratchpads [half-hammock edition]

2015-11-11 Thread Bernd Schmidt
On 11/10/2015 10:35 PM, Abe wrote: I wrote: What I'm saying is I don't see a reason for a "definitely always unsafe" state. Why would any access not be made safe if a scratchpad is used? Because the RTL if-converter doesn`t "know" how to convert {everything that can be made safe using a scratc

Re: [gomp4] Fix some broken tests

2015-11-11 Thread Nathan Sidwell
On 11/11/15 09:50, Cesar Philippidis wrote: On 11/11/2015 05:40 AM, Nathan Sidwell wrote: On 11/10/15 18:08, Cesar Philippidis wrote: On 11/10/2015 12:35 PM, Nathan Sidwell wrote: I've committed this to gomp4. In preparing the reworked firstprivate patch changes for gomp4's gimplify.c I disc

Re: State of support for the ISO C++ Transactional Memory TS and remanining work

2015-11-11 Thread Szabolcs Nagy
On 10/11/15 18:29, Torvald Riegel wrote: On Tue, 2015-11-10 at 17:26 +, Szabolcs Nagy wrote: On 09/11/15 00:19, Torvald Riegel wrote: Hi, I'd like to summarize the current state of support for the TM TS, and outline the current plan for the work that remains to complete the support. ...

[HSA] support global variables

2015-11-11 Thread Martin Liška
Hi. Following patch adds support for global variables seen by an HSAIL executable. The HSA runtime can link a name of a global variable with pointer to the variable used by host. Installed to HSA branch. Martin >From de58711a6ddbb1e4558a9454d7aeb6d2b33861de Mon Sep 17 00:00:00 2001 From: marxin

Re: State of support for the ISO C++ Transactional Memory TS and remanining work

2015-11-11 Thread Jonathan Wakely
On 11/11/15 15:04 +, Szabolcs Nagy wrote: yes, non-experimental (since you were talking about libstdc++ changes to complete the support for the TS). The TS is experimental. That's the nature of all C++ TSs. Completing the TS support does not mean anything is non-experimental.

[PATCH][ARC] Fix ARC backend ICE on pr29921-2

2015-11-11 Thread Claudiu Zissulescu
Please find attached a patch that fixes the ARC backend ICE on pr29921-2 test from gcc.dg (dg.exp). The patch will allow generating conditional move also outside expand scope. The error was triggered during if-conversion. Ok to apply? Claudiu ChangeLog: 2015-11-11 Claudiu Zissulescu

[PATCH][ARM] Do not expand movmisalign pattern if not in 32-bit mode

2015-11-11 Thread Kyrill Tkachov
Hi all, The attached testcase ICEs when compiled with -march=armv6k -mthumb -Os or any march for which -mthumb gives Thumb1: error: unrecognizable insn: } ^ (insn 13 12 14 5 (set (reg:SI 116 [ x ]) (unspec:SI [ (mem:SI (reg/v/f:SI 112 [ s ]) [0 MEM[(unsigned char *)s_

[gomp4.5] Don't mark GOMP_MAP_FIRSTPRIVATE mapped vars addressable

2015-11-11 Thread Jakub Jelinek
Hi! Alex reported to me privately that with the OpenMP 4.5 handling of array section bases (that they are firstprivate instead of mapped) we unnecessarily mark the pointers addressable and that result in less efficient way of passing them as shared to inner constructs. They don't need to be made

[PATCH, alpha]: Add TARGET_PRINT_OPERAND and friends

2015-11-11 Thread Uros Bizjak
2015-11-11 Uros Bizjak * config/alpha/alpha-protos.h (print_operand): Remove. (print_operand_address): Remove. * config/alpha/alpha.h (PRINT_OPERAND): Remove. (PRINT_OPERAND_ADDRESS): Remove. (PRINT_OPERAND_PUNCT_VALID_P): Remove. * config/alpha/alpha.c (TARGET_PRINT_OPE

Re: [gomp4.5] Don't mark GOMP_MAP_FIRSTPRIVATE mapped vars addressable

2015-11-11 Thread Alexander Monakov
On Wed, 11 Nov 2015, Jakub Jelinek wrote: > Hi! > > Alex reported to me privately that with the OpenMP 4.5 handling of > array section bases (that they are firstprivate instead of mapped) > we unnecessarily mark the pointers addressable and that result > in less efficient way of passing them as s

Re: [gomp4.5] Don't mark GOMP_MAP_FIRSTPRIVATE mapped vars addressable

2015-11-11 Thread Jakub Jelinek
On Wed, Nov 11, 2015 at 07:27:51PM +0300, Alexander Monakov wrote: > > Alex reported to me privately that with the OpenMP 4.5 handling of > > array section bases (that they are firstprivate instead of mapped) > > we unnecessarily mark the pointers addressable and that result > > in less efficient w

Re: [PATCH v4] SH FDPIC backend support

2015-11-11 Thread Rich Felker
On Wed, Nov 11, 2015 at 09:56:42AM -0500, Rich Felker wrote: > > > I'm actually > > > trying to prepare a simpler FDPIC patch for other gcc versions we're > > > interested in that's not so invasive, and for now I'm just having > > > function_symbol replace SFUNC_STATIC with SFUNC_GOT on TARGET_FDPI

[gomp4.5] depend nowait support for target

2015-11-11 Thread Jakub Jelinek
Hi! On Mon, Oct 19, 2015 at 10:47:54PM +0300, Ilya Verbin wrote: > So, here is what I have for now. Attached target-29.c testcase works fine > with > MIC emul, however I don't know how to (and where) properly check for > completion > of async execution on target. And, similarly, where to do un

[PATCH] Fix detection of setrlimit in libstdc++ testsuite

2015-11-11 Thread Maxim Kuvyrkov
Hi, This patch fixes an obscure cross-testing problem that crashed (OOMed) our boards at Linaro. Several tests in libstdc++ (e.g., [1]) limit themselves to some reasonable amount of RAM and then try to allocate 32 gigs. Unfortunately, the configure test that checks presence of setrlimit is ra

[patch] libstdc++/60421 (again) Loop in std::this_thread sleep functions

2015-11-11 Thread Jonathan Wakely
This fixes part of PR 60421 by looping in this_thread::sleep_for when it is interrupted by a signal, and looping in this_thread::sleep_until to handle clock adjustments. There are still problems with integer overflow/wrapping in sleep_for, which still need to be addressed somehow. Maybe using the

Re: [ptx] partitioning optimization

2015-11-11 Thread Thomas Schwinge
Hi! On Wed, 11 Nov 2015 08:59:17 -0500, Nathan Sidwell wrote: > On 11/11/15 07:06, Bernd Schmidt wrote: > > On 11/10/2015 11:33 PM, Nathan Sidwell wrote: > >> I've been unable to introduce a testcase for this. (But you still committed an update to gcc/testsuite/ChangeLog.) You'll need to put su

Re: [v3 PATCH] LWG 2510, make the default constructors of library tag types explicit.

2015-11-11 Thread Dominique d'Humières
Revision r230175 > 2015-11-10 Ville Voutilainen > > LWG 2510, make the default constructors of library tag types > explicit. > * include/bits/mutex.h (defer_lock_t, try_lock_t, > adopt_lock_t): Add an explicit default constructor. > * include/bits/stl_pair.h (piecewise_const

open acc default data attribute

2015-11-11 Thread Nathan Sidwell
Jakub, this patch implements default data attribute determination. The current behaviour defaults to 'copy' and ignores 'default(none)'. The patch corrects that. 1) We emit a diagnostic when 'default(none)' is in effect. The fortran FE emits some artificial decls that it doesn't otherwise a

Re: [OpenACC] declare directive

2015-11-11 Thread Jakub Jelinek
On Wed, Nov 11, 2015 at 11:08:21AM +0100, Thomas Schwinge wrote: > Hi! > > On Wed, 11 Nov 2015 09:32:33 +0100, Jakub Jelinek wrote: > > On Mon, Nov 09, 2015 at 05:11:44PM -0600, James Norris wrote: > > > diff --git a/gcc/c-family/c-pragma.h b/gcc/c-family/c-pragma.h > > > index 953c4e3..c6a2981 1

Re: [v3 PATCH] LWG 2510, make the default constructors of library tag types explicit.

2015-11-11 Thread Jonathan Wakely
On 11/11/15 18:17 +0100, Dominique d'Humières wrote: Revision r230175 2015-11-10 Ville Voutilainen LWG 2510, make the default constructors of library tag types explicit. * include/bits/mutex.h (defer_lock_t, try_lock_t, adopt_lock_t): Add an explicit default constructor.

Re: [PR64164] drop copyrename, integrate into expand

2015-11-11 Thread Alexandre Oliva
On Nov 10, 2015, Jeff Law wrote: >> * function.c (assign_parm_setup_block): Right-shift >> upward-padded big-endian args when bypassing the stack slot. > Don't you need to check the value of BLOCK_REG_PADDING at runtime? > The padding is essentially allowed to vary. Well, yeah, it's the result o

Re: [PATCH][ARC] Fix ARC backend ICE on pr29921-2

2015-11-11 Thread Joern Wolfgang Rennecke
On 11/11/15 15:22, Claudiu Zissulescu wrote: Please find attached a patch that fixes the ARC backend ICE on pr29921-2 test from gcc.dg (dg.exp). The patch will allow generating conditional move also outside expand scope. The error was triggered during if-conversion. Ok to apply? OK.

[PATCH] PR fortran/68283 -- remove a rogue gfc_internal_error()

2015-11-11 Thread Steve Kargl
This probably falls under the "obviously correct" moniker. It has been built and tested on i386-*-freebsd. OK to commit? The patch removes a gfc_internal_error(). I suspect that it was originally put into gfortran to cover "correctly written valid Fortran code cannot possibly ever hit this line

[PATCH] [ARM/Aarch64] add initial Qualcomm support

2015-11-11 Thread Jim Wilson
This adds an option for the Qualcomm server parts, qdf24xx, just optimizing like a cortex-a57 for now, same as how the initial Samsung exynos-m1 support worked. This was tested with armv8 and aarch64 bootstraps and make check. I had to disable the cortex-a57 fma steering pass in the aarch64 port

Re: [Patch] Optimize condition reductions where the result is an integer induction variable

2015-11-11 Thread Alan Hayward
On 11/11/2015 13:25, "Richard Biener" wrote: >On Wed, Nov 11, 2015 at 1:22 PM, Alan Hayward >wrote: >> Hi, >> I hoped to post this in time for Monday’s cut off date, but >>circumstances >> delayed me until today. Hoping if possible this patch will still be able >> to go in. >> >> >> This patch

Re: [PATCH] PR47266

2015-11-11 Thread Jerry DeLisle
On 11/10/2015 06:41 AM, Dominique d'Humières wrote: > Index: gcc/testsuite/ChangeLog > === > --- gcc/testsuite/ChangeLog (revision 229793) > +++ gcc/testsuite/ChangeLog (working copy) > @@ -1,3 +1,9 @@ > +2015-11-10 Dominique d'Hu

Re: [PATCH] PR fortran/68283 -- remove a rogue gfc_internal_error()

2015-11-11 Thread Jerry DeLisle
On 11/11/2015 10:34 AM, Steve Kargl wrote: > This probably falls under the "obviously correct" moniker. > It has been built and tested on i386-*-freebsd. OK to commit? > > The patch removes a gfc_internal_error(). I suspect that it > was originally put into gfortran to cover "correctly written >

[PATCH 2/2] [graphite] improve construction of the original schedule

2015-11-11 Thread Sebastian Pop
The patch builds the original schedule based on the now optimized scattering dimension instead of building one based on the loop index only. The implementation is simpler and catches more cases where the original schedule and the transformed schedule are the same, such as the one below: for (i =

[PATCH 1/2] [graphite] add testsuite automatic dg-options and dg-do action for isl-ast-gen-* and fuse-* files

2015-11-11 Thread Sebastian Pop
--- gcc/testsuite/gcc.dg/graphite/fuse-1.c | 10 +++--- gcc/testsuite/gcc.dg/graphite/fuse-2.c | 4 +--- gcc/testsuite/gcc.dg/graphite/graphite.exp | 2 ++ gcc/testsuite/gcc.dg/graphite/isl-ast-gen-blocks-1.c| 3 --- gcc/testsuite/gcc.dg/gr

  1   2   >