Re: [PATCH] Remove UNSPEC_LOADU and UNSPEC_STOREU

2016-01-10 Thread Uros Bizjak
On Sun, Jan 10, 2016 at 11:32 PM, H.J. Lu wrote: > Since *mov_internal and _(load|store)_mask patterns > can handle unaligned load and store, we can remove UNSPEC_LOADU and > UNSPEC_STOREU. We use function prototypes with pointer to scalar for > unaligned load/store builtin

Re: [PATCH 3/3] [RFC] Treat a gimplification failure as an internal error

2016-01-10 Thread Patrick Palka
On Thu, Dec 31, 2015 at 10:40 AM, Patrick Palka wrote: > This patch makes it so that a gimplification failure is considered to be > an internal error under normal circumstances, so that we otherwise avoid > silently generating wrong code if e.g. a buggy frontend fed us a >

Re: [PATCH] OpenACC documentation for libgomp

2016-01-10 Thread Sandra Loosemore
On 01/05/2016 08:47 AM, James Norris wrote: Hi! I've updated the original patch after some very helpful comments from Sandra (thank you, thank you). OK to commit to trunk? I'm assuming this is now waiting for technical review? I can give it another read-through for tech-writing issues but

Re: [PATCH] Fix PR c++/69056 (argument pack deduction failure during overload resolution)

2016-01-10 Thread Patrick Palka
On Sun, Dec 27, 2015 at 12:14 PM, Patrick Palka wrote: > In try_one_overload(), it should not be a deduction failure if we > deduced more arguments of an argument pack than were explicitly > specified (as long as the arguments are otherwise equivalent). > > This patch makes

Re: [PATCH] Fix constexpr evaluation of comparisons involving pointer-to-members

2016-01-10 Thread Patrick Palka
On Mon, Dec 21, 2015 at 6:07 PM, Patrick Palka wrote: > We are currently failing to fold equality comparisons involving > PTRMEM_CSTs since (I think) r229018 thus making this a GCC 6 regression. > This regression shows up in the boost testsuite. > > Fixed in a

Re: [PATCH 1/3] Fix logic bug in Cilk Plus array expansion

2016-01-10 Thread Patrick Palka
On Mon, Jan 4, 2016 at 1:35 PM, Jeff Law wrote: > On 01/02/2016 04:26 PM, Patrick Palka wrote: >> >> On Sat, Jan 2, 2016 at 3:21 AM, Jakub Jelinek wrote: >>> >>> On Fri, Jan 01, 2016 at 10:06:34PM -0700, Jeff Law wrote: > > gcc/cp/ChangeLog: > >

Re: [PATCH] arc/gcc: Fix help text for some options.

2016-01-10 Thread Joern Wolfgang Rennecke
On 08/01/16 14:13, Andrew Burgess wrote: * config/arc/arc.opt (mdiv-rem): Add period to the end. (mcode-density): Likewise. Thanks, I have applied this patch.

Re: [PATCH 2/3] Avoid creating an initializer for a flexible array member

2016-01-10 Thread Patrick Palka
On Sun, Jan 3, 2016 at 3:14 PM, Martin Sebor wrote: > On 12/31/2015 08:40 AM, Patrick Palka wrote: >> >> If we do create such an initializer, we end up with an error_mark_node >> during gimplification, because in cp-gimplify.c we pass this >> VEC_INIT_EXPR of the flexible array

[patch, fortran] Fix PR 69154, inline matmul with WHERE

2016-01-10 Thread Thomas Koenig
Hello world, the attached patch fixes the regression. Before this, front-end optimization would have tried to put a DO loop inside a WHERE construct, leading to an ICE. Regression-tested. OK for trunk? Thomas 2016-01-10 Thomas Koenig PR fortran/69154

Re: [PATCH] Fix PR68707

2016-01-10 Thread Thomas Schwinge
Hi! On Fri, 8 Jan 2016 11:30:16 +, Alan Lawrence wrote: > Here's an alternative patch, [...] > +/* { dg-final { scan-tree-dump "note: Built SLP cancelled: can use > load/store-lanes" { target { vect_perm && vect_load_lanes } } } } */ For all these, you're missing

Re: [Patch, fortran] PR67779 - Strange ordering with strings in extended object

2016-01-10 Thread Paul Richard Thomas
Dear All, The patch applies cleanly to 5-branch and fixes the problem. I will commit it later on today, at the same time as the first batch of deferred character fixes unless there are any objections. Cheers Paul On 10 January 2016 at 14:02, Paul Richard Thomas

[Patch, fortran] PR67779 - Strange ordering with strings in extended object

2016-01-10 Thread Paul Richard Thomas
Dear All, I had already posted a patch for the above, which sets the offset correctly in the caller to gfc_conv_expr_descriptor. However, I was so concerned that the latter was not doing the job properly that I decided to investigate further. The result was that I found a completely trivial

[Committed, PR69039] Only allow single exit phi for reduction in try_create_reduction_list

2016-01-10 Thread Tom de Vries
Hi, Consider this test-case (reduced from ira-color.c): ... unsigned int b; unsigned int fn1 (unsigned int d) { int i; for (i = 0; i < 1000; i++) b |= d; return b; } ... When compiling with -O2 -ftree-parallelize-loops=2, we run into a segfault. Before parloops, we have reduction

Re: [patch, fortran] Fix PR 69154, inline matmul with WHERE

2016-01-10 Thread Thomas Koenig
Am 10.01.2016 um 12:55 schrieb Thomas Koenig: Hello world, the attached patch fixes the regression. Before this, front-end optimization would have tried to put a DO loop inside a WHERE construct, leading to an ICE. The updated test case demonstrates that matmul inlining is done even after

Re: [patch, fortran] Fix PR 69154, inline matmul with WHERE

2016-01-10 Thread Paul Richard Thomas
Hi Thomas, Thanks for changing the testcase to demonstrate that in_where does not get left as true. OK for trunk. That's one regression down! Thanks for the patch Paul On 10 January 2016 at 13:38, Thomas Koenig wrote: > Am 10.01.2016 um 12:55 schrieb Thomas Koenig: >>

[gomp4] Merge trunk r232189 (2016-01-09) into gomp-4_0-branch

2016-01-10 Thread Thomas Schwinge
Hi! Committed to gomp-4_0-branch in r232198: commit bec521e294ddf37d52860bfa0724a5e6a2bc2225 Merge: 1d481a9 0f8f48d Author: tschwinge Date: Sun Jan 10 12:27:25 2016 + svn merge -r 231690:232189 svn+ssh://gcc.gnu.org/svn/gcc/trunk

Correct GCC Internals doc for loop_depth

2016-01-10 Thread Nicklas Bo Jensen
Hi, In the GCC Internals documentation the loop depth is documented incorrectly. From git commit 9e3536f4f it is accessed through a function, not a field. Please install this patch to reflect this. Best, Nicklas 2017-01-10 Nicklas Bo Jensen * doc/loop.texi

[PATCH configure,config.gcc] Fix for PR69153 to allow config.gcc to set target_header_dir

2016-01-10 Thread Peter Bergner
While testing a configure fragment change for an upcoming patch, I noticed that --with-advance-toolchain=... wasn't correctly setting $target_header_dir to point to the Advance Toolchain's headers and instead was pointing at the system headers. This patch arranges for configure.ac to allow

[Committed, PR69062] Don't parallelize loops containing phis with addr_exprs

2016-01-10 Thread Tom de Vries
Hi, when compiling the testcase in the patch (reduced from ira-color.c) with -O2 -ftree-parallelize-loops=2, we run into an ICE: ... prephitmp_39 = PHI <_40(16), _regs(7)> ira-color.c:69:8: internal compiler error: verify_gimple failed ... The variable conflicting_regs is a local array in the

Re: [PATCH configure,config.gcc] Fix for PR69153 to allow config.gcc to set target_header_dir

2016-01-10 Thread Bernd Edlinger
Hi Peter, > @@ -4167,6 +4167,7 @@ > -d "/opt/$with_advance_toolchain/bin/." -a \ > -d "/opt/$with_advance_toolchain/include/."; then > >+ target_header_dir=`find /opt/$with_advance_toolchain/ >-path '*/include/features.h' | sed -e

New Chinese (simplified) PO file for 'gcc' (version 5.2.0)

2016-01-10 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 Chinese (simplified) team of translators. The file is available at: http://translationproject.org/latest/gcc/zh_CN.po (This file,

[PATCH] Remove snprintf from _(load|store)_mask

2016-01-10 Thread H.J. Lu
This patch removes snprintf from _(load|store)_mask patterns. Tested on x86-64. OK for trunk? H.J. --- * config/i386/sse.md (_load_mask): Remove snprintf. (_store_mask): Likewise. --- gcc/config/i386/sse.md | 52 -- 1

[doc, 0/n] improve organization of invoke.texi

2016-01-10 Thread Sandra Loosemore
I am about to start a project to incrementally move chunks of invoke.texi around to better organize this very long chapter of the manual. For reference, the current top-level structure looks like this: @chapter GCC Command Options @section Option Summary @section Options Controlling the Kind

[PATCH] Remove UNSPEC_LOADU and UNSPEC_STOREU

2016-01-10 Thread H.J. Lu
Since *mov_internal and _(load|store)_mask patterns can handle unaligned load and store, we can remove UNSPEC_LOADU and UNSPEC_STOREU. We use function prototypes with pointer to scalar for unaligned load/store builtin functions so that memory passed to *mov_internal is unaligned. Tested on

Re: [PATCH configure,config.gcc] Fix for PR69153 to allow config.gcc to set target_header_dir

2016-01-10 Thread Peter Bergner
On Sun, 2016-01-10 at 19:28 +, Bernd Edlinger wrote: > Hi Peter, > > > @@ -4167,6 +4167,7 @@ > > -d "/opt/$with_advance_toolchain/bin/." -a \ > > -d "/opt/$with_advance_toolchain/include/."; > > then > > > > + target_header_dir=`find