Re: [PATCH] libiberty: support demangling of rvalue reference typenames

2016-01-04 Thread Artemiy Volkov
On Mon, Jan 04, 2016 at 10:06:44PM -0800, Ian Lance Taylor wrote: > Artemiy Volkov writes: > > > This patch adds handling of 'O' (rvalue ref) type codes in the C++ > > demangling > > code which is done similarly to the 'R' (regular references) case. It also > > adds > > a few testcases for vari

[PATCH, testsuite] Fix g++.dg/pr67989.C test failure when running with -march or -mcpu

2016-01-04 Thread Thomas Preud'homme
Hi, g++.dg/pr67989.C passes -march=armv4t to gcc when compiling which fails if RUNTESTFLAGS passes -mcpu or -march with a different value. This patch adds a dg-skip-if directive to skip the test when such a thing happens. ChangeLog entry is as follows: *** gcc/testsuite/ChangeLog *** 2015-12

Re: [PATCH, GCC] Fix PR67781: wrong code generation for partial load on big endian targets

2016-01-04 Thread Thomas Preud'homme
On Tuesday, January 05, 2016 01:53:37 PM you wrote: > > Regression testsuite was run on a bootstrapped native x86_64-linux-gnu GCC > and on an arm-none-eabi GCC cross-compiler without any regression. I'm > waiting for a slot on gcc110 to do a big endian bootstrap but at least the > testcase works

Re: [PATCH 2/4] Equate MEM_REFs and ARRAY_REFs in tree-ssa-scopedtables.c

2016-01-04 Thread Richard Biener
On January 4, 2016 8:08:17 PM GMT+01:00, Jeff Law wrote: >On 12/21/2015 06:13 AM, Alan Lawrence wrote: >> This is a respin of patches >> https://gcc.gnu.org/ml/gcc-patches/2015-10/msg03266.html and >> https://gcc.gnu.org/ml/gcc-patches/2015-10/msg03267.html, which were >> "too quickly" approved be

Re: [PATCH] libiberty: support demangling of rvalue reference typenames

2016-01-04 Thread Ian Lance Taylor
Artemiy Volkov writes: > This patch adds handling of 'O' (rvalue ref) type codes in the C++ demangling > code which is done similarly to the 'R' (regular references) case. It also > adds > a few testcases for various demangling styles which are just mirrored versions > of the corresponding regul

[PATCH, GCC] Fix PR67781: wrong code generation for partial load on big endian targets

2016-01-04 Thread Thomas Preud'homme
Hi, bswap optimization pass generate wrong code on big endian targets when the result of a bit operation it analyzed is a partial load of the range of memory accessed by the original expression (when one or more bytes at lowest address were lost in the computation). This is due to the way cmpxc

Re: [PATCH] c++/58109 - alignas() fails to compile with constant expression

2016-01-04 Thread Martin Sebor
Ping: looking for review/approval of the patch below: https://gcc.gnu.org/ml/gcc-patches/2015-12/msg02074.html Thanks Martin On 12/22/2015 07:32 PM, Martin Sebor wrote: The attached patch adds handling of dependent arguments to attribute aligned and attribute vector_size, fixing c++/58109 and

Re: [PATCH] c/68966 - atomic_fetch_* on atomic_bool not diagnosed

2016-01-04 Thread Martin Sebor
On 01/04/2016 08:22 AM, Marek Polacek wrote: Hi Martin, ... Thanks for the careful review! I've fixed the problems you pointed out in the attached patch. The typos are my bad. As for the whitespace, I have to confess I'm finding all the rules tedious to follow without some sort of automation

Re: [PATCH] document -Winvalid-memory-model

2016-01-04 Thread Sandra Loosemore
On 01/04/2016 05:15 PM, Martin Sebor wrote: s/built-ins/builtins/ (like in the @refs you used previously) I thought the @refs were an inconsistency and built-in was the preferred spelling. That's what someone else pointed out to me sometime ago and what I see documented in the GCC Coding Conv

Re: [PATCH] document -Winvalid-memory-model

2016-01-04 Thread Martin Sebor
We just had a patch a month or so ago (r231022) to sort this table into something approaching alphabetical order, module no- prefixes, I guess. Can you please insert the new entry into a less random place? Sure. It was meant to be inserted in the right place, my brain just filtered out the "i

Re: [PATCH] PR target/68991: Add vector_memory_operand and "Bm" constraint

2016-01-04 Thread H.J. Lu
On Mon, Jan 4, 2016 at 1:11 PM, H.J. Lu wrote: > On Mon, Jan 4, 2016 at 12:19 PM, Uros Bizjak wrote: >> On Mon, Jan 4, 2016 at 4:11 AM, H.J. Lu wrote: >>> On Sat, Jan 2, 2016 at 10:26 AM, H.J. Lu wrote: On Sat, Jan 2, 2016 at 3:58 AM, Richard Biener wrote: > On January 2, 2016 11

Re: [PATCH] document -Winvalid-memory-model

2016-01-04 Thread Sandra Loosemore
On 01/04/2016 03:17 PM, Martin Sebor wrote: As discussed in c/69104, the -Winvalid-memory-model option is not documented in the manual. The attached patch rectifies that. Thanks for tackling this. Index: doc/invoke.texi === ---

[PATCH] libiberty: support demangling of rvalue reference typenames

2016-01-04 Thread Artemiy Volkov
This patch adds handling of 'O' (rvalue ref) type codes in the C++ demangling code which is done similarly to the 'R' (regular references) case. It also adds a few testcases for various demangling styles which are just mirrored versions of the corresponding regular references demangling tests. lib

[PATCH] document -Winvalid-memory-model

2016-01-04 Thread Martin Sebor
As discussed in c/69104, the -Winvalid-memory-model option is not documented in the manual. The attached patch rectifies that. Martin gcc/ChangeLog: 2016-01-04 Martin Sebor * doc/invoke.texi (Warning Options): Document -Winvalid-memory-model. Index: doc/invoke.texi

Re: [PATCH], PowerPC IEEE 128-bit fp, #11 (enable libgcc conversions)

2016-01-04 Thread Michael Meissner
On Thu, Dec 31, 2015 at 08:29:58PM +, Joseph Myers wrote: > On Tue, 29 Dec 2015, Michael Meissner wrote: > > > +/* __eqkf2 returns 0 if equal, or 1 if not equal or NaN. */ > > +CMPtype > > +__eqkf2_hw (TFtype a, TFtype b) > > +{ > > + return (__builtin_isunordered (a, b) || (a != b)) ? 1 : 0

Re: [patch] ARM FreeBSD fix bootstrap

2016-01-04 Thread Andreas Tobler
Ping :) TIA, Andreas On 23.12.15 20:28, Andreas Tobler wrote: On 23.12.15 11:22, Richard Earnshaw (lists) wrote: On 22/12/15 19:53, Andreas Tobler wrote: Hi all, the commit for PR68617 broke boostrap on armv6*-*-freebsd*. We still have unaligned_access = 0 on armv6 here on FreeBSD. The com

Re: [PATCH] PR target/68991: Add vector_memory_operand and "Bm" constraint

2016-01-04 Thread H.J. Lu
On Mon, Jan 4, 2016 at 12:19 PM, Uros Bizjak wrote: > On Mon, Jan 4, 2016 at 4:11 AM, H.J. Lu wrote: >> On Sat, Jan 2, 2016 at 10:26 AM, H.J. Lu wrote: >>> On Sat, Jan 2, 2016 at 3:58 AM, Richard Biener >>> wrote: On January 2, 2016 11:32:33 AM GMT+01:00, Uros Bizjak wrote: >On

Re: [PATCH] PR target/68991: Add vector_memory_operand and "Bm" constraint

2016-01-04 Thread Uros Bizjak
On Mon, Jan 4, 2016 at 4:11 AM, H.J. Lu wrote: > On Sat, Jan 2, 2016 at 10:26 AM, H.J. Lu wrote: >> On Sat, Jan 2, 2016 at 3:58 AM, Richard Biener >> wrote: >>> On January 2, 2016 11:32:33 AM GMT+01:00, Uros Bizjak >>> wrote: On Thu, Dec 31, 2015 at 4:29 PM, H.J. Lu wrote: > On Thu, D

[PATCH] Fix SLP ICE (PR tree-optimization/69083)

2016-01-04 Thread Jakub Jelinek
Hi! The vec-cmp SLP patch added + if (VECTOR_BOOLEAN_TYPE_P (vector_type)) + { + /* Can't use VIEW_CONVERT_EXPR for booleans because +of possibly different sizes of scalar value and +vector elemen

Re: cilkplus fails without pthreads for me

2016-01-04 Thread Nathan Sidwell
On 01/04/16 14:17, Mike Stump wrote: The next proposed patch is: Index: target-supports.exp === --- target-supports.exp (revision 232062) +++ target-supports.exp (working copy) @@ -1442,11 +1442,6 @@ proc check_effective_target_cil

Re: cilkplus fails without pthreads for me

2016-01-04 Thread Nathan Sidwell
On 01/04/16 14:19, Mike Stump wrote: I believe that patch has: +/* { dg-do compile { target cilkplus } } */ in it, and this I believe is required for the test to be skipped on my target? that bit is still necessary. It's the bit in the .exp file testing nvptx-*-* that's no longer needed.

Re: cilkplus fails without pthreads for me

2016-01-04 Thread Mike Stump
On Jan 4, 2016, at 9:09 AM, Nathan Sidwell wrote: > On 01/04/16 10:06, Bernd Schmidt wrote: >> On 01/01/2016 07:13 PM, Mike Stump wrote: >>> cilkplus fails without pthreads for me: >>> >>> xg++: error: unrecognized command line option '-pthread' compiler >>> exited with status 1 output is: xg++:

Re: cilkplus fails without pthreads for me

2016-01-04 Thread Mike Stump
On Jan 4, 2016, at 7:22 AM, Nathan Sidwell wrote: > On 01/01/16 13:13, Mike Stump wrote: >> cilkplus fails without pthreads for me: >> >> xg++: error: unrecognized command line option '-pthread' >> compiler exited with status 1 >> output is: >> xg++: error: unrecognized command line option '-pthr

Re: [PATCH 2/4] Equate MEM_REFs and ARRAY_REFs in tree-ssa-scopedtables.c

2016-01-04 Thread Jeff Law
On 12/21/2015 06:13 AM, Alan Lawrence wrote: This is a respin of patches https://gcc.gnu.org/ml/gcc-patches/2015-10/msg03266.html and https://gcc.gnu.org/ml/gcc-patches/2015-10/msg03267.html, which were "too quickly" approved before concerns with efficiency were pointed out. I tried to change th

Re: [PATCH 2/4] Equate MEM_REFs and ARRAY_REFs in tree-ssa-scopedtables.c

2016-01-04 Thread Jeff Law
On 12/24/2015 04:55 AM, Alan Lawrence wrote: This version changes the test cases to fix failures on some platforms, by rewriting the initializers so that they aren't pushed out to the constant pool. gcc/ChangeLog: * tree-ssa-scopedtables.c (avail_expr_hash): Hash MEM_REF and ARRAY_REF

Re: varpool/constpool bug

2016-01-04 Thread Jeff Law
On 01/04/2016 08:57 AM, Nathan Sidwell wrote: My patch to stop constant pool objects accidentally ending up in the varpool caused problems with (at least) powerpc. (https://gcc.gnu.org/ml/gcc-patches/2015-12/msg02100.html) Hence reverted. This patch changes compare_base_decls to simply use the v

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

2016-01-04 Thread Jeff Law
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: * cp-array-notation.c (cp_expand_cond_array_notations): Return error_mark_node only if find_rank failed, not

[gomp4] Fix acc_on_device for C++

2016-01-04 Thread Nathan Sidwell
This patch fixes acc_on_device's C++ wrapper when compiling at -O0. The wrapper isn't inlined, and we need to mark the function as needing emission by the device compiler too. nathan 2016-01-04 Nathan Sidwell * openacc.c (acc_on_device): Add routine pragma for C++ wrapper. * testsuite/li

guilty test suite fix

2016-01-04 Thread Mike Stump
So, I’d like for the guality people to chime in. I only kick in, if they fail to do so for any reason. :-) Either, the stuff downstream _must_ arrange for newline ended content, or this code has to do it, if they don’t. My take, I think they are signing up for newline terminated content: bi

Re: cilkplus fails without pthreads for me

2016-01-04 Thread Nathan Sidwell
On 01/04/16 10:06, Bernd Schmidt wrote: On 01/01/2016 07:13 PM, Mike Stump wrote: cilkplus fails without pthreads for me: xg++: error: unrecognized command line option '-pthread' compiler exited with status 1 output is: xg++: error: unrecognized command line option '-pthread' > @@ -1450,6 +1

Re: [PATCH] PR/68089: C++-11: Ingore "alignas(0)".

2016-01-04 Thread Martin Sebor
On 01/04/2016 04:33 AM, Dominik Vogt wrote: On Fri, Jan 01, 2016 at 05:53:08PM -0700, Martin Sebor wrote: On 12/31/2015 04:50 AM, Dominik Vogt wrote: The attached patch fixes C++-11 handling of "alignas(0)" which should be ignored but currently generates an error message. A test case is includ

Re: varpool/constpool bug

2016-01-04 Thread Nathan Sidwell
My patch to stop constant pool objects accidentally ending up in the varpool caused problems with (at least) powerpc. (https://gcc.gnu.org/ml/gcc-patches/2015-12/msg02100.html) Hence reverted. This patch changes compare_base_decls to simply use the varpool getter, rather than get_create. We s

Mark oacc kernels fns

2016-01-04 Thread Nathan Sidwell
There's currently no robust predicate to determine whether an oacc offload function is for a kernels region (as opposed to a parallel region). The test in tree-ssa-loop.c uses the heuristic of seeing if all the dimensions are defaulted (which can easily be true for parallel offloads at that po

Re: [Patch ifcvt] Add a new parameter to limit if-conversion

2016-01-04 Thread Bernd Schmidt
On 12/31/2015 10:21 AM, Yuri Rumyantsev wrote: Here is slightly modified patch which limits a number of conditional moves instead of changing conditional branch cost. This is in fact a work-around for very poor cost model which needs to be enhanced to evaluate cost of conditional move that could

Re: [PATCH] c/68966 - atomic_fetch_* on atomic_bool not diagnosed

2016-01-04 Thread Marek Polacek
Hi Martin, On Sun, Jan 03, 2016 at 08:03:20PM -0700, Martin Sebor wrote: > Index: gcc/doc/extend.texi > === > --- gcc/doc/extend.texi (revision 232047) > +++ gcc/doc/extend.texi (working copy) > @@ -9238,6 +9238,8 @@ > @{

Re: cilkplus fails without pthreads for me

2016-01-04 Thread Nathan Sidwell
On 01/01/16 13:13, Mike Stump wrote: cilkplus fails without pthreads for me: xg++: error: unrecognized command line option '-pthread' compiler exited with status 1 output is: xg++: error: unrecognized command line option '-pthread' FAIL: c-c++-common/attr-simd-3.c -std=gnu++14 PR68158 (test fo

Re: [PATCH], PowerPC, Add -mpower9-dform to switches turned on with -mcpu=power9

2016-01-04 Thread David Edelsohn
On Thu, Dec 31, 2015 at 3:41 PM, Michael Meissner wrote: > When I did the inital d-form support for ISA 3.0 (power9) for loading scalar > SF/DF values into Altivec registers, I did not enable -mpower9-dform with the > other ISA 3.0 switches when you used -mcpu=power9. This was during the > initi

Re: [PATCH], PowerPC, add ISA 3.0 xxperm (power9 patch #12)

2016-01-04 Thread David Edelsohn
On Thu, Dec 31, 2015 at 1:30 PM, Michael Meissner wrote: > This patch adds support for the ISA 3.0 XXPERM instruction, which is like > VPERM, except it can operate on any VSX register. Since the instruction is a > 3 > operand instruction (RT and RA must be the same), I made it so VPERM was > pre

Re: cilkplus fails without pthreads for me

2016-01-04 Thread Bernd Schmidt
On 01/01/2016 07:13 PM, Mike Stump wrote: cilkplus fails without pthreads for me: xg++: error: unrecognized command line option '-pthread' compiler exited with status 1 output is: xg++: error: unrecognized command line option '-pthread' > @@ -1450,6 +1450,10 @@ proc check_effective_target_cilk

Re: [PATCH, PR69043, fortran] Trying to include a directory causes an infinite loop

2016-01-04 Thread Jim MacArthur
On 24/12/15 16:38, Jim MacArthur wrote: Botstrapped and tested for regressions on x86_64-pc-linux-gnu. There is a test case for the bug included. I missed out the test case when creating the first patch. This one should have it. PR fortran/69043 * scanner.c (load_file): Abort and show an

[PATCH] Adjust contrib/update-copyright.py

2016-01-04 Thread Jakub Jelinek
Hi! One of the gfortran.dg/ tests has NVidia copyright, which made update-copyright.py stop changing anything further. Committed to trunk as obvious. 2016-01-04 Jakub Jelinek * update-copyright.py (GCCCopyright): Add NVIDIA Corporation as external author. --- contrib/update-

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

2016-01-04 Thread Ajit Kumar Agarwal
-Original Message- From: Jeff Law [mailto:l...@redhat.com] Sent: Wednesday, December 23, 2015 12:06 PM To: Ajit Kumar Agarwal; Richard Biener Cc: GCC Patches; Vinod Kathail; Shail Aditya Gupta; Vidhumouli Hunsigida; Nagaraju Mekala Subject: Re: [Patch,tree-optimization]: Add new path Sp

Re: [PATCH] shrink-wrap: Once more PRs 67778, 68634, and now 68909

2016-01-04 Thread Bernd Schmidt
On 12/20/2015 05:27 PM, Segher Boessenkool wrote: On Fri, Dec 18, 2015 at 02:19:37AM +0100, Bernd Schmidt wrote: On 12/17/2015 10:07 PM, Segher Boessenkool wrote: It turns out v4 wasn't quite complete anyway; so here "v5". If a candidate PRE cannot get the prologue because a block BB is reacha

Re: [testsuite][ARM target attributes] Fix effective_target tests

2016-01-04 Thread Christophe Lyon
On 4 January 2016 at 15:20, Christophe Lyon wrote: > On 18 December 2015 at 15:16, Kyrill Tkachov > wrote: >> Hi Christophe, >> >> >> On 17/12/15 22:17, Christophe Lyon wrote: >>> >>> Hi, >>> >>> Here is an updated version of this patch. >>> I did test it with >>> -mthumb/-march=armv8-a/-mfpu=cry

Re: [testsuite][ARM target attributes] Fix effective_target tests

2016-01-04 Thread Christophe Lyon
On 18 December 2015 at 15:16, Kyrill Tkachov wrote: > Hi Christophe, > > > On 17/12/15 22:17, Christophe Lyon wrote: >> >> Hi, >> >> Here is an updated version of this patch. >> I did test it with >> -mthumb/-march=armv8-a/-mfpu=crypto-neon-fp-armv8/-mfloat-abi=hard in >> addition to my usual set

Re: [PATCH] PR target/68991: Add vector_memory_operand and "Bm" constraint

2016-01-04 Thread H.J. Lu
On Mon, Jan 4, 2016 at 12:21 AM, Jakub Jelinek wrote: > On Sun, Jan 03, 2016 at 07:11:58PM -0800, H.J. Lu wrote: >> --- a/gcc/config/i386/predicates.md >> +++ b/gcc/config/i386/predicates.md >> @@ -951,6 +951,13 @@ >> (match_test "INTEGRAL_MODE_P (GET_MODE (op))") >> (match_test "o

[committed] Update copyright years, part 1

2016-01-04 Thread Jakub Jelinek
Hi! I've committed following patch to update the user visible copyright years (and rolled new year of gcc/fortran and libjava ChangeLogs, plus added Copyright boilerplate at the end of libitm, libgomp and libquadmath ChangeLog files). 2016-01-04 Jakub Jelinek gcc/ * gcc.c (process_com

Re: [PATCH][Testsuite]Cleanup logs from gdb tests by adding newlines

2016-01-04 Thread Alan Lawrence
Ping. --Alan On 10/12/15 10:31, Alan Lawrence wrote: Runs of the guality testsuite can sometimes end up with gcc.log containing malformed lines like: A debugging session is active.PASS: gcc.dg/guality/pr36728-1.c -O2 line 18 arg4 == 4 A debugging session is active.PASS: gcc.dg/guality/rest

Re: [PATCH 1/4] Make SRA scalarize constant-pool loads

2016-01-04 Thread Alan Lawrence
On 24/12/15 11:53, Alan Lawrence wrote: Here's a new version that fixes the gcc.dg/guality/pr54970.c failures seen on aarch64 and powerpc64. [snip] This also fixes a bunch of other guality tests on AArch64 that were failing prior to the patch series, and another bunch on PowerPC64 (bigendian -m

Re: [PATCH 4/4] Un-XFAIL ssa-dom-cse-2.c for most platforms

2016-01-04 Thread Alan Lawrence
On 24/12/15 19:59, Mike Stump wrote: On Dec 22, 2015, at 8:00 AM, Alan Lawrence wrote: On 21/12/15 15:33, Bill Schmidt wrote: Not on a stage1 compiler - check_p8vector_hw_available itself requires being able to run executables - I'll check on gcc112. However, both look like they're really abo

Re: [PATCH] PR/68089: C++-11: Ingore "alignas(0)".

2016-01-04 Thread Dominik Vogt
On Fri, Jan 01, 2016 at 05:53:08PM -0700, Martin Sebor wrote: > On 12/31/2015 04:50 AM, Dominik Vogt wrote: > >The attached patch fixes C++-11 handling of "alignas(0)" which > >should be ignored but currently generates an error message. A > >test case is included; the patch has been tested on S390

Re: [ping] Enable -mstackrealign with SSE on 32-bit Windows

2016-01-04 Thread Eric Botcazou
> This is really Windows specific setting, so Windows maintainer should > OK the patch. Makes sense, both maintainers now CCed. https://gcc.gnu.org/ml/gcc-patches/2015-12/msg01458.html -- Eric Botcazou

Re: [ping] Enable -mstackrealign with SSE on 32-bit Windows

2016-01-04 Thread Uros Bizjak
On Mon, Jan 4, 2016 at 9:26 AM, Eric Botcazou wrote: > https://gcc.gnu.org/ml/gcc-patches/2015-12/msg01458.html > > Thanks in advance. This is really Windows specific setting, so Windows maintainer should OK the patch. Uros.

[ping] Enable -mstackrealign with SSE on 32-bit Windows

2016-01-04 Thread Eric Botcazou
https://gcc.gnu.org/ml/gcc-patches/2015-12/msg01458.html Thanks in advance. -- Eric Botcazou

Re: [PATCH] PR target/68991: Add vector_memory_operand and "Bm" constraint

2016-01-04 Thread Jakub Jelinek
On Sun, Jan 03, 2016 at 07:11:58PM -0800, H.J. Lu wrote: > --- a/gcc/config/i386/predicates.md > +++ b/gcc/config/i386/predicates.md > @@ -951,6 +951,13 @@ > (match_test "INTEGRAL_MODE_P (GET_MODE (op))") > (match_test "op == CONSTM1_RTX (GET_MODE (op))"))) > > +; Return true when

[SPARC] Fix PR target/69100

2016-01-04 Thread Eric Botcazou
(FUNCTION_ARG_REGNO_P): Return true in 64-bit mode for %f0-%f31 only if TARGET_FPU. 2016-01-04 Eric Botcazou * gcc.target/sparc/20160104-2.c: New test. -- Eric Botcazou/* PR target/69100 */ /* Reported by Zdenek Sojka */ /* { dg-do compile } */ /* { dg-options "-mno-fpu" } */

[SPARC] Fix PR target/69072

2016-01-04 Thread Eric Botcazou
): Minor tweaks. 2016-01-04 Eric Botcazou * gcc.target/sparc/20160104-1.c: New test. -- Eric BotcazouIndex: config/sparc/sparc.c === --- config/sparc/sparc.c (revision 231971) +++ config/sparc/sparc.c (working copy