Re: [PATCH] RFC: On-demand locations within string-literals

2016-08-22 Thread Martin Sebor
Beyond that, the range normally works fine, except when macros are involved like they are in my tests. You can see the effect in the range.out file. (This works without your patch but it could very well be because I didn't set it up right.) Sadly I can't figure out what's going wrong - but

Re: [PATCH] Automatically generate dependencies for the build-machine object files

2016-08-22 Thread Patrick Palka
On Mon, 22 Aug 2016, Patrick Palka wrote: > Currently the dependencies of each build/*.o object file are manually > maintained within the Makefile and are rather incomplete. For instance > build/vec.o doesn't depend on hash-table.h even though vec.c includes > it. Because of this I ran into a

Re: [RFC][IPA-VRP] Early VRP Implementation

2016-08-22 Thread Kugan Vivekanandarajah
Hi, On 19 August 2016 at 21:41, Richard Biener wrote: > On Tue, Aug 16, 2016 at 9:45 AM, kugan > wrote: >> Hi Richard, >> >> On 12/08/16 20:43, Richard Biener wrote: >>> >>> On Wed, Aug 3, 2016 at 3:17 AM, kugan

[PATCH] Automatically generate dependencies for the build-machine object files

2016-08-22 Thread Patrick Palka
Currently the dependencies of each build/*.o object file are manually maintained within the Makefile and are rather incomplete. For instance build/vec.o doesn't depend on hash-table.h even though vec.c includes it. Because of this I ran into a rebuild failure after changing hash-table.h. Any

Re: [PATCH], Patch #5, Improve vector int initialization on PowerPC

2016-08-22 Thread Segher Boessenkool
On Mon, Aug 22, 2016 at 06:01:22PM -0400, Michael Meissner wrote: > > > +static void > > > +rs6000_split_v4si_init_di_reg (rtx dest, rtx si1, rtx si2, rtx tmp) > > > +{ > > > + const unsigned HOST_WIDE_INT mask_32bit = HOST_WIDE_INT_C (0x); > > > > Does using that macro buy us anything?

Re: [PATCH] Remove whitespace

2016-08-22 Thread Jonathan Wakely
On 22/08/16 15:30 -0700, Mike Stump wrote: On Aug 22, 2016, at 2:45 PM, Aditya Kumar wrote: libstdc++-v3/ChangeLog | 5 + libstdc++-v3/include/bits/algorithmfwd.h| 206 ++-- libstdc++-v3/include/bits/shared_ptr_base.h |

Re: [PATCH] Remove whitespace

2016-08-22 Thread Mike Stump
On Aug 22, 2016, at 2:45 PM, Aditya Kumar wrote: > > libstdc++-v3/ChangeLog | 5 + > libstdc++-v3/include/bits/algorithmfwd.h| 206 ++-- > libstdc++-v3/include/bits/shared_ptr_base.h | 26 ++-- > 3 files changed, 121

Re: [PATCH] newlib-stdint.h: Remove 32 bit longs

2016-08-22 Thread Pavel Pisa
Hello all, I am not expert but I would like bring attention to some facts which come to my mind On Monday 22 of August 2016 18:19:06 Andy Ross wrote: > --- a/gcc/config/newlib-stdint.h > +++ b/gcc/config/newlib-stdint.h > @@ -22,10 +22,9 @@ a copy of the GCC Runtime Library Exception along with

Re: [Patch, fortran] PR48298 - [F03] User-Defined Derived-Type IO (DTIO)

2016-08-22 Thread Dominique d'Humières
Dear Jerry, > Dominiq, please test on your system. It does work on darwin! Thanks for the quick fix. Dominique

Re: C++11? (Re: protected alloca class for malloc fallback)

2016-08-22 Thread Mike Stump
On Aug 22, 2016, at 5:02 AM, Eric Gallager wrote: > > As a rookie programmer considering possibly contributing to GCC in the > future once I'm more confident in my abilities, switching to C++11 > would increase the barrier for me to contribute. I currently really > only

Re: [PATCH], Patch #5, Improve vector int initialization on PowerPC

2016-08-22 Thread Michael Meissner
On Mon, Aug 22, 2016 at 11:37:55AM -0500, Segher Boessenkool wrote: > [ seems this mail never arrived, resending, sorry if it turns out a duplicate > ] > > Hi Mike, > > Okay for trunk. A few comments... > > On Fri, Aug 19, 2016 at 06:17:54PM -0400, Michael Meissner wrote: > > ---

Re: [PATCH] Fix suggestions for non-trivial Wformat type cases (PR c/72858)

2016-08-22 Thread Martin Sebor
On 08/12/2016 01:27 PM, David Malcolm wrote: In r239260 I attempted to add fix-it hints for -Wformat type warnings. Unfortunately, my implementation was too simplistic, and only worked correctly for the most simple format strings: the fix-it hint would suggest replacement of an entire

Re: [patch] [gsoc] [gimplefe] GIMPLE FE Project

2016-08-22 Thread Pedro Alves
On 08/22/2016 10:26 PM, David Malcolm wrote: > it has no problems, whereas with the default for gcc 5 and earlier: > > $ gcc -c test.cc -std=gnu++98 > test.cc:3:15: warning: non-static data member initializers only > available with -std=c++11 or -std=gnu++11 >int field = 42; >

Re: C++ PATCH to fix ICE with memset warning (PR c++/77321)

2016-08-22 Thread Marek Polacek
On Mon, Aug 22, 2016 at 06:38:10PM +0200, Marek Polacek wrote: > Here we were crashing because the code wasn't prepared for NULL_TREE types. > Thus fixed in a rather obvious way. Bernd approved the patch in the BZ, but I'm going to commit this version, with the original unreduced testcase:

[PATCH] Remove whitespace

2016-08-22 Thread Aditya Kumar
--- libstdc++-v3/ChangeLog | 5 + libstdc++-v3/include/bits/algorithmfwd.h| 206 ++-- libstdc++-v3/include/bits/shared_ptr_base.h | 26 ++-- 3 files changed, 121 insertions(+), 116 deletions(-) diff --git a/libstdc++-v3/ChangeLog

[PATCH] remove whitespace

2016-08-22 Thread Aditya Kumar
--- libstdc++-v3/include/bits/shared_ptr_base.h | 26 +- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/libstdc++-v3/include/bits/shared_ptr_base.h b/libstdc++-v3/include/bits/shared_ptr_base.h index 787dc9b..60b825c 100644 ---

Go patch committed: Don't permit P.M for a pointer type

2016-08-22 Thread Ian Lance Taylor
https://golang.org/issue/15722 points out that gccgo was incorrectly permitting P.M to used as a method expression when P is a pointer type. This patch by Than McIntosh fixes the problem. Bootstrapped and ran Go testsuite on x86_64-pc-linux-gnu. Committed to mainline. Ian Index:

Re: [patch] [gsoc] [gimplefe] GIMPLE FE Project

2016-08-22 Thread David Malcolm
On Tue, 2016-08-23 at 00:10 +0530, Prasad Ghangal wrote: > On 22 August 2016 at 16:55, Trevor Saunders > wrote: > > On Sun, Aug 21, 2016 at 10:35:17PM +0530, Prasad Ghangal wrote: [...] > > @@ -228,6 +228,12 @@ struct GTY(()) function { > >/* GIMPLE body for this

Re: [patch] [gsoc] [gimplefe] GIMPLE FE Project

2016-08-22 Thread Trevor Saunders
On Tue, Aug 23, 2016 at 12:10:29AM +0530, Prasad Ghangal wrote: > On 22 August 2016 at 16:55, Trevor Saunders wrote: > > On Sun, Aug 21, 2016 at 10:35:17PM +0530, Prasad Ghangal wrote: > >> Hi all, > >> > >> As a part of my gsoc project. I have completed the following

Re: [Patch] Reduce regex _M_dfs frame size

2016-08-22 Thread Tim Shen
On Mon, Aug 22, 2016 at 10:35 AM, Jonathan Wakely wrote: > On 20/08/16 03:28 -0700, Tim Shen wrote: > OK for trunk with the more detailed changelog entry. Thanks! Done. Tested on x86_64-linux-gnu (if I remember it correctly back to several days ago) and committed as r239673.

[PATCH, i386, AVX-512] Fix runtime test for kunpck intrinsic.

2016-08-22 Thread Kirill Yukhin
Hello, It looks like reference value calculations use operands in wrong order. Patch in the bottom makes test pass on SDE. gcc/testsuite/ * gcc.target/i386/avx512f-klogic-2.c: Fix calculation of reference value. Will check into main trunk tomorrow. -- Thanks, K commit

PING [PATCH] PR middle-end/74113: Don't limit piecewise move to MAX_FIXED_MODE_SIZE

2016-08-22 Thread H.J. Lu
On Thu, Aug 11, 2016 at 1:38 PM, H.J. Lu wrote: > alignment_for_piecewise_move is called only with MOVE_MAX_PIECES or > STORE_MAX_PIECES, which are the number of bytes at a time that we > can move or store efficiently. We should call mode_for_size without > limit to

Re: [PATCH build/doc] Replacing libiberty with gnulib

2016-08-22 Thread Pedro Alves
I noticed that libcpp/include/symtab.h includes obstack.h and puts a struct obstack in an exported structure: /* An identifier hash table for cpplib and the front ends. */ struct ht { /* Identifiers are allocated from here. */ struct obstack stack; [...] typedef struct ht cpp_hash_table;

Re: [patch] [gsoc] [gimplefe] GIMPLE FE Project

2016-08-22 Thread Prasad Ghangal
On 22 August 2016 at 16:55, Trevor Saunders wrote: > On Sun, Aug 21, 2016 at 10:35:17PM +0530, Prasad Ghangal wrote: >> Hi all, >> >> As a part of my gsoc project. I have completed the following tasks: >> >> * Parsed gimple-expression >> * Parsed gimple-labels >> * Parsed

Re: [PATCH] newlib-stdint.h: Remove 32 bit longs

2016-08-22 Thread Richard Earnshaw
On 22/08/16 17:09, Andy Ross wrote: > The reproduction is straightforward. Just build any cross gcc with > --enable-newlib (e.g. the one in the Zephyr SDK) and compile this > (on any 32 or 64 bit 2's complement architecture) with newlib's > headers. > > #include > > extern void

Re: [PATCH, C++] Warn on redefinition of builtin functions (PR c++/71973)

2016-08-22 Thread Joseph Myers
On Sat, 20 Aug 2016, Bernd Edlinger wrote: > Initially I tried to warn unconditionally but that made too many tests > in the C++ testsuite emit that warning :-( I think that's probably a peculiarity of testsuite code trying to avoid standard headers, and does not reflect what normal user C++

Re: [PATCH] newlib-stdint.h: Remove 32 bit longs

2016-08-22 Thread Joseph Myers
On Mon, 22 Aug 2016, Andy Ross wrote: > And gcc, as seen by this patch, sets it to a long because it thinks > that's what newlib *wants*. That would be because a newlib version that included the change commit 843e635aaa02f16f314688ba5dd8a5edc3929095 Author: Jeff Johnston

Re: Implement C _FloatN, _FloatNx types [version 6]

2016-08-22 Thread Joseph Myers
On Mon, 22 Aug 2016, Joseph Myers wrote: > > This change makes gcc.dg/torture/arm-fp16-int-convert-{alt,ieee].c fail > > because they still pass four arguments to the macro, not five. > > I've applied this patch that should fix this. I should point, however, that these tests are currently

Re: Support __builtin_isinf_sign for new floating-point types (PR middle-end/77269)

2016-08-22 Thread Richard Biener
On August 22, 2016 7:17:25 PM GMT+02:00, Joseph Myers wrote: >The __builtin_isinf_sign folding uses a type-specific signbit built-in >function, meaning it only works for the types float, double and long >double, not for types such as _FloatN, _FloatNx, __float128. Since

Re: [PATCH build/doc] Replacing libiberty with gnulib

2016-08-22 Thread Joseph Myers
You appear to be including ../gnulib/config.h for both host and build. That can't be right when host and build are different. In that case you need to include config.h from the correct build directory in each case. You also need to arrange for BUILD_CPPFLAGS to have the right -I options for

Re: [Patch] Reduce regex _M_dfs frame size

2016-08-22 Thread Jonathan Wakely
On 20/08/16 03:28 -0700, Tim Shen wrote: I merely split _M_dfs() into small functions to see how it goes. It turns out to save half of the stack consumption in -O0 without observable performance impact. That seems like a good improvement. Split _M_dfs() into smaller functions.

Re: [PATCHv2] do not throw in std::make_exception_ptr

2016-08-22 Thread Jonathan Wakely
On 21/08/16 15:20 +0300, Gleb Natapov wrote: Jonathan, Is this version OK with you? I've committed the attached version, which just adds some whitespace and fixes the testsuite_abi.cc test. Thanks very much for the improvement to the code. commit f80c0cc401fc34b424662286bedc559d0e2bb243

Support __builtin_isinf_sign for new floating-point types (PR middle-end/77269)

2016-08-22 Thread Joseph Myers
The __builtin_isinf_sign folding uses a type-specific signbit built-in function, meaning it only works for the types float, double and long double, not for types such as _FloatN, _FloatNx, __float128. Since the signbit built-in function is now type-generic, that can be used unconditionally, much

Re: [PATCH] newlib-stdint.h: Remove 32 bit longs

2016-08-22 Thread Andy Ross
Andrew Pinski wrote: > On Mon, Aug 22, 2016 at 9:42 AM, Andy Ross wrote: > > I don't follow. There's no change to the ABI, the generated code is > > identical in all cases. Can you explain what you mean? > > The encoding of int and long are different in C++. So if you

[PATCH] Update links to libstdc++ Doxygen pages

2016-08-22 Thread Jonathan Wakely
There's no reason to have a fixed list of release versions in the libstdc++ docs which needs updating manually. The page at https://gcc.gnu.org/onlinedocs already links to the Doxygen docs for each release, so we can just refer to that. * doc/xml/api.xml: Replace hardcoded links for

[PATCH] Add noexcept to std::function swap

2016-08-22 Thread Jonathan Wakely
We're missing noexcept on std::function::swap, and once LWG 2062 is resolved we'll be missing it on the non-member swap too. This adds it to both. PR libstdc++/77322 * doc/xml/manual/intro.xml: Document DR 2062 change. * include/std/functional (function::swap): Add

Re: [PATCH] newlib-stdint.h: Remove 32 bit longs

2016-08-22 Thread Andrew Pinski
On Mon, Aug 22, 2016 at 9:42 AM, Andy Ross wrote: > Andrew Pinski wrote: >> Why do you think the above code does not have a bug in it? int32_t >> is long and changing it now is changing the ABI (especially for >> C++). > > I don't follow. There's no change to the ABI,

Re: [PATCH], Patch #6, Improve vector short/char splat initialization on PowerPC

2016-08-22 Thread Segher Boessenkool
Hi Mike, Okay for trunk. Comment below... On Fri, Aug 19, 2016 at 07:59:39PM -0400, Michael Meissner wrote: > * config/rs6000/rs6000.c (rs6000_expand_vector_init): Add support > for using VSPLTH/VSPLTB to initialize vector short and vector char > vectors with all of the same

Re: [PATCH/BUILD] [GSOC] [REPLACING LIBIBERTY WITH GNULIB]

2016-08-22 Thread Manuel López-Ibáñez
On 22 August 2016 at 17:34, Manuel López-Ibáñez wrote: > On 22 August 2016 at 17:27, ayush goel wrote: >> Links to the patches containing the work: >> >> https://gcc.gnu.org/ml/gcc-patches/2016-07/msg01302.html > > Actually, the above version is

Re: [PATCH build/doc] Replacing libiberty with gnulib

2016-08-22 Thread Pedro Alves
On 08/22/2016 05:36 PM, ayush goel wrote: > +# The expected version number for the various auto tools we will > +# use after the import. > +AUTOCONF_VERSION="2.69" > +AUTOMAKE_VERSION="1.15" > +ACLOCAL_VERSION="$AUTOMAKE_VERSION" > + Is there a reason these need to be different from what the rest

Re: [PATCH] newlib-stdint.h: Remove 32 bit longs

2016-08-22 Thread Andy Ross
Andrew Pinski wrote: > Why do you think the above code does not have a bug in it? int32_t > is long and changing it now is changing the ABI (especially for > C++). I don't follow. There's no change to the ABI, the generated code is identical in all cases. Can you explain what you mean? The

C++ PATCH to fix ICE with memset warning (PR c++/77321)

2016-08-22 Thread Marek Polacek
Here we were crashing because the code wasn't prepared for NULL_TREE types. Thus fixed in a rather obvious way. Bootstrapped/regtested on x86_64-linux, ok for trunk? 2016-08-22 Marek Polacek PR c++/77321 * c-common.c (warn_for_memset): Check type for null.

Re: [PATCH], Patch #5, Improve vector int initialization on PowerPC

2016-08-22 Thread Segher Boessenkool
[ seems this mail never arrived, resending, sorry if it turns out a duplicate ] Hi Mike, Okay for trunk. A few comments... On Fri, Aug 19, 2016 at 06:17:54PM -0400, Michael Meissner wrote: > --- gcc/config/rs6000/rs6000.c >

Re: [PATCH] newlib-stdint.h: Remove 32 bit longs

2016-08-22 Thread Andrew Pinski
On Mon, Aug 22, 2016 at 9:09 AM, Andy Ross wrote: > The reproduction is straightforward. Just build any cross gcc with > --enable-newlib (e.g. the one in the Zephyr SDK) and compile this > (on any 32 or 64 bit 2's complement architecture) with newlib's > headers. > >

Re: [PATCH build/doc] Replacing libiberty with gnulib

2016-08-22 Thread ayush goel
Updating the patch, contains only changes to the documentation. PFA the patch 2016-08-22 Ayush Goel * Makefile.def: Added gnulib as build & host library and dependency of all-gcc on gnulib * Makefile.in: regenerated * configure.ac: Added gnulib as build and host

Re: [PATCH/BUILD] [GSOC] [REPLACING LIBIBERTY WITH GNULIB]

2016-08-22 Thread Manuel López-Ibáñez
On 22 August 2016 at 17:27, ayush goel wrote: > Links to the patches containing the work: > > https://gcc.gnu.org/ml/gcc-patches/2016-07/msg01302.html Actually, the above version is outdated and it should not be reviewed. The latest version of this patch, is this one:

Re: Implement -Wimplicit-fallthrough (version 4)

2016-08-22 Thread Martin Sebor
Just a few minor nits based on a quick reading of the patch: @@ -24114,6 +24164,16 @@ cp_parser_std_attribute (cp_parser *parser) " use %"); TREE_PURPOSE (TREE_PURPOSE (attribute)) = get_identifier ("gnu"); } + /* C++17 fallthrough

[PATCH/BUILD] [GSOC] [REPLACING LIBIBERTY WITH GNULIB]

2016-08-22 Thread ayush goel
Hi, This mail is only to summarise my patches for the gsoc project. Most of the details of the project can be found at https://gcc.gnu.org/wiki/replacelibibertywithgnulib I’ve essentially tried to do the following things as a part of this project: * Import gnulib as a top level directory inside

Re: [PATCH] newlib-stdint.h: Remove 32 bit longs

2016-08-22 Thread Andy Ross
Same code. Includes an attempt to format a change log entry in a more gcc-friendly way. I'm new here. >From c208b51fb55c6758c1059dfaee398c27da398e9d Mon Sep 17 00:00:00 2001 From: Andy Ross Date: Fri, 19 Aug 2016 09:40:42 -0700 Subject: [PATCH] newlib-stdint.h: Remove

Re: [PATCH] newlib-stdint.h: Remove 32 bit longs

2016-08-22 Thread Andy Ross
The reproduction is straightforward. Just build any cross gcc with --enable-newlib (e.g. the one in the Zephyr SDK) and compile this (on any 32 or 64 bit 2's complement architecture) with newlib's headers. #include extern void takes_fmt(const char *fmt, ...) __attribute__

Re: [Patch, fortran] PR48298 - [F03] User-Defined Derived-Type IO (DTIO)

2016-08-22 Thread Paul Richard Thomas
Hi Dominique, Thanks for trying this out so quickly and finding this rather fundamental problem. In pr30617, DTIO is obliquely described as recursive IO. It isn't really, is it? I'll contact Jerry on #gfortran to see what we can do about this. Cheers Paul On 22 August 2016 at 17:09,

Re: [PATCH] Add a TARGET_GEN_MEMSET_VALUE hook

2016-08-22 Thread H.J. Lu
On Mon, Aug 22, 2016 at 4:31 AM, Richard Biener wrote: > On Fri, Aug 19, 2016 at 4:45 PM, H.J. Lu wrote: >> On Fri, Aug 19, 2016 at 2:21 AM, Richard Biener >> wrote: >>> On Thu, Aug 18, 2016 at 5:16 PM, H.J. Lu

Re: [wwwdocs] superh.com has gone poker(?)

2016-08-22 Thread Oleg Endo
On Mon, 2016-08-22 at 14:27 +0200, Gerald Pfeifer wrote: > It appears www.superh.com no longer has anything to do with > processors, > but poker. Better remove this (old) link. Yeah, I think I've removed one of those links from some other page a while ago. Also, note that SH5 support has been

Re: [RFT PATCH, i386]: Optimize zero-extensions from mask registers

2016-08-22 Thread Kirill Yukhin
Hello Uroš, On 05 Aug 14:22, Uros Bizjak wrote: > Hello! > > Attached patch was inspired by assembly from PR 72805 testcase. > Currently, the compiler generates: > > test: > vpternlogd $0xFF, %zmm0, %zmm0, %zmm0 > vpxord %zmm1, %zmm1, %zmm1 > vpcmpd $1, %zmm1,

Re: [PATCH] Fix PR77286

2016-08-22 Thread Dominique d'Humières
This patch causes: FAIL: gfortran.dg/vect/pr69466.f90 -O (internal compiler error) FAIL: gfortran.dg/vect/pr69466.f90 -O (test for excess errors) with -m32. [Book15] f90/bug% gfc /opt/gcc/_clean/gcc/testsuite/gfortran.dg/vect/pr69466.f90 -O2 -ftree-vectorize -m32 -march=core-avx2

Re: [Patch, fortran] PR48298 - [F03] User-Defined Derived-Type IO (DTIO)

2016-08-22 Thread Dominique d'Humières
The executable hangs with Call graph: 2699 Thread_2235 DispatchQueue_1: com.apple.main-thread (serial) 2699 get_external_unit (in libgfortran.3.dylib) + 254 [0x10de297be] gthr-default.h:748 2699 _pthread_mutex_lock_wait (in libsystem_pthread.dylib) + 89

Re: [Patch, fortran] PR48298 - [F03] User-Defined Derived-Type IO (DTIO)

2016-08-22 Thread Paul Richard Thomas
Dear Dominique, It is related to pr30617. When I compile the dtio testcases with -lpthread, they hang as you say (as does the testcase of pr30617). Blast! Both Jerry and I are using Fedora. Do I gather that the use of threads is not the default there? Cheers Paul On 22 August 2016 at 15:57,

Re: [PATCH] Tree-level fix for PR 69526

2016-08-22 Thread Robin Dapp
> if !inner_ovf (just set that to false if !check_inner_ovf to simplify > checks please). > you know it's valid to transform the op to (T)@0 innerop (T)@1 outerop @2 > (if T is wider than the inner type which I think you should check and > which should > simplify things). I simplified the control

Re: [wwwdocs] superh.com has gone poker(?)

2016-08-22 Thread Jeff Law
On 08/22/2016 06:27 AM, Gerald Pfeifer wrote: It appears www.superh.com no longer has anything to do with processors, but poker. Better remove this (old) link. How amusing... jeff

Implement -Wimplicit-fallthrough (version 4)

2016-08-22 Thread Marek Polacek
This version of -Wimplicit-fallthrough incorporates changes I made to address Jakub's and David's comments. I fixed up the fixit hints handling (the output looks much better now) and added tests for that. I also added tests for ObjC and Obj-C++ and the warning now works for these languages too.

Re: [wwwdocs,libstdc++] Remove libstdc++/

2016-08-22 Thread Gerald Pfeifer
On Mon, 22 Aug 2016, Jonathan Wakely wrote: >> So, after https://gcc.gnu.org/ml/gcc-patches/2016-08/msg01467.html I >> suggest to streamline things, remove this page, and apply the patch >> below. > Would this remove the link to the libstdc++ FAQ? I had actually watched out for that. ;-)

Implement -Wimplicit-fallthrough (version 4): add gcc_fallthrough

2016-08-22 Thread Marek Polacek
Accompanying changes to the core patch. 2016-08-22 Marek Polacek PR c/7652 * Makefile.in (insn-attrtab.o-warn, insn-dfatab.o-warn, insn-latencytab.o-warn, insn-output.o-warn, insn-emit.o-warn): Add -Wno-switch-fallthrough. *

Re: [PATCH] remove HARD_FRAME_POINTER_IS_ARG_POINTER macro

2016-08-22 Thread Marek Polacek
On Sat, Aug 20, 2016 at 01:07:32PM -0400, tbsaunde+...@tbsaunde.org wrote: > @@ -859,7 +859,8 @@ expand_builtin_setjmp_receiver (rtx receiver_label) >emit_clobber (hard_frame_pointer_rtx); > } > > - if (!HARD_FRAME_POINTER_IS_ARG_POINTER && fixed_regs[ARG_POINTER_REGNUM]) > + if

Re: [Patch, fortran] PR48298 - [F03] User-Defined Derived-Type IO (DTIO)

2016-08-22 Thread Dominique d'Humières
Dear Paul, I have tried the patch, but all the tests hang at run time (expect dtio_10.f90). This is on x86_64-apple-darwin15. Could it be related to pr30617? Cheers, Dominique

Re: [RFC] ipa bitwise constant propagation

2016-08-22 Thread Prathamesh Kulkarni
On 22 August 2016 at 19:03, Martin Jambor wrote: > Hi, > > On Tue, Aug 16, 2016 at 06:34:48PM +0530, Prathamesh Kulkarni wrote: >> Thanks, I updated the patch to address these issues (attached). >> However the patch caused ICE during testing >> objc.dg/torture/forward-1.m (and

Re: [ARM][PATCH] Add support for overflow add, sub, and neg operations

2016-08-22 Thread Kyrill Tkachov
On 21/08/16 19:11, Christophe Lyon wrote: On 13 August 2016 at 00:00, Michael Collison wrote: Christophe, I have resolved the issue you identified. I had not update my upstream patch with a previous fix I had made to the usubv pattern. Tested and bootstrapped on

Re: [RFC] ipa bitwise constant propagation

2016-08-22 Thread Martin Jambor
Hi, On Tue, Aug 16, 2016 at 06:34:48PM +0530, Prathamesh Kulkarni wrote: > Thanks, I updated the patch to address these issues (attached). > However the patch caused ICE during testing > objc.dg/torture/forward-1.m (and few others but with same ICE): > > Command line options: >

Re: C++11? (Re: protected alloca class for malloc fallback)

2016-08-22 Thread Manuel López-Ibáñez
On 22/08/16 13:02, Eric Gallager wrote: As a rookie programmer considering possibly contributing to GCC in the future once I'm more confident in my abilities, switching to C++11 would increase the barrier for me to contribute. I currently really only know C, and I still have to learn C++ in

Re: [PATCH] - improve sprintf buffer overflow detection (middle-end/49905)

2016-08-22 Thread Florian Weimer
On 08/20/2016 05:18 AM, Trevor Saunders wrote: Patch #5 and beyond: Further optimization work. As one of the next steps I'd like to make this feature available to user-defined sprintf-like functions decorated with attribute format. To do that, I'm thinking of adding either a fourth (optional)

[wwwdocs] superh.com has gone poker(?)

2016-08-22 Thread Gerald Pfeifer
It appears www.superh.com no longer has anything to do with processors, but poker. Better remove this (old) link. Applied. Gerald Index: gcc-3.1/changes.html === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-3.1/changes.html,v retrieving

Re: [PATCH] Improve readability of debug_tree() dumps for SSA_NAME and VECTOR_CST

2016-08-22 Thread Richard Biener
On Mon, Aug 22, 2016 at 1:39 PM, Patrick Palka wrote: > On Mon, 22 Aug 2016, Richard Biener wrote: > >> On Sat, Aug 20, 2016 at 1:27 AM, Patrick Palka wrote: >> > * For SSA_NAME: Print the ssa name's def stmt on its own line. >> > >> > Before: >> > >

Re: C++11? (Re: protected alloca class for malloc fallback)

2016-08-22 Thread Eric Gallager
On 8/21/16, Pedro Alves wrote: > On 08/20/2016 03:29 AM, Mike Stump wrote: >> On Aug 10, 2016, at 10:03 AM, Oleg Endo wrote: >>> >>> Or just wait until people have agreed to switch to C++11 or C++14. I >>> don't think in practice anybody uses an

[wwwdocs] benchmarks/index.html -- vmakarov.fedorapeople.org now defaults to https

2016-08-22 Thread Gerald Pfeifer
Applied. Gerald Index: index.html === RCS file: /cvs/gcc/wwwdocs/htdocs/benchmarks/index.html,v retrieving revision 1.35 diff -u -r1.35 index.html --- index.html 27 May 2016 19:35:42 - 1.35 +++ index.html 22 Aug 2016

Re: Implement C _FloatN, _FloatNx types [version 6]

2016-08-22 Thread Joseph Myers
On Mon, 22 Aug 2016, Matthew Wahab wrote: > > + TEST_I_F(TItype, UTItype, _Float32, FLT32X_MANT_DIG, FLT32X_MAX_EXP); > > + exit (0); > > + > > This test fails with an abort at runtime. The other float32x tests pass. That should be fixed by the commit I made to fix the issue pointed out by

[doc] doc/install.texi: www.opencsw.org now uses https.

2016-08-22 Thread Gerald Pfeifer
2016-08-22 Gerald Pfeifer * doc/install.texi (Binaries): www.opencsw.org now uses https. Applied. I'll backport to GCC 6 as well, once Richi has unfrozen the branch. Gerald Index: gcc/doc/install.texi

Re: [PATCH] remove HARD_FRAME_POINTER_IS_ARG_POINTER macro

2016-08-22 Thread Trevor Saunders
On Mon, Aug 22, 2016 at 11:25:01AM +0200, Bernd Schmidt wrote: > On 08/20/2016 07:07 PM, tbsaunde+...@tbsaunde.org wrote: > > From: Trevor Saunders > > > > Hi, > > > > its kind of silly, and this allows us to remove a few more #ifdefs. > > > #if FRAME_POINTER_REGNUM

Re: [PATCH] Improve readability of debug_tree() dumps for SSA_NAME and VECTOR_CST

2016-08-22 Thread Patrick Palka
On Mon, 22 Aug 2016, Richard Biener wrote: > On Sat, Aug 20, 2016 at 1:27 AM, Patrick Palka wrote: > > * For SSA_NAME: Print the ssa name's def stmt on its own line. > > > > Before: > > > type > type > public unsigned SI > > size > >

Re: [PR59319] output friends in debug info

2016-08-22 Thread Richard Biener
On Fri, Aug 19, 2016 at 8:46 PM, Alexandre Oliva wrote: > This is not a finished patch. There are two issues I'd like feedback > on before a final submission. See them below. First, a general > description. > > Handling non-template friends is kind of easy, but it required a

Re: [PATCH] Add a TARGET_GEN_MEMSET_VALUE hook

2016-08-22 Thread Richard Biener
On Fri, Aug 19, 2016 at 4:45 PM, H.J. Lu wrote: > On Fri, Aug 19, 2016 at 2:21 AM, Richard Biener > wrote: >> On Thu, Aug 18, 2016 at 5:16 PM, H.J. Lu wrote: >>> On Thu, Aug 18, 2016 at 1:18 AM, Richard Biener >>>

Re: [BUILDROBOT] x86_64: Segmentation fault during -fself-test (was: Implement C _FloatN, _FloatNx types [version 6]

2016-08-22 Thread Joseph Myers
On Mon, 22 Aug 2016, Jan-Benedict Glaw wrote: > Between dee8cef0c1c1ebf85fceb5c37996ed12a2bec352 (Fri Aug 19 15:42:11 > 2016 +) and 82c85aba845985e55c27a7a9c448810d433adb17 (Fri Aug 19 > 17:43:26 2016 +), a new build regression for > x86_64-{linux,rtems,elf} showed up and I think this

Re: Add minimal _FloatN, _FloatNx built-in functions [version 2]

2016-08-22 Thread Richard Biener
On Fri, Aug 19, 2016 at 10:57 PM, Joseph Myers wrote: > [Version 2 of this patch updates the testcases to use dg-add-options as in > the final version of the _FloatN patch that went in; there are no changes > of substance outside the testsuite. Version 1 was >

Re: [patch] [gsoc] [gimplefe] GIMPLE FE Project

2016-08-22 Thread Trevor Saunders
On Sun, Aug 21, 2016 at 10:35:17PM +0530, Prasad Ghangal wrote: > Hi all, > > As a part of my gsoc project. I have completed the following tasks: > > * Parsed gimple-expression > * Parsed gimple-labels > * Parsed local declaration > * Parsed gimple-goto statement > * Parsed gimple-if-else

Re: Implement C _FloatN, _FloatNx types [version 6]

2016-08-22 Thread Matthew Wahab
Hello, On 17/08/16 21:17, Joseph Myers wrote: [Version 6 changes the testsuite to use dg-add-options systematically to add any options that may be needed for the types to be supported; this should allow the _Float128 and _Float64x tests to run for powerpc64le, but I have not tested that; it

Re: Implement C _FloatN, _FloatNx types [version 6]

2016-08-22 Thread Joseph Myers
On Sun, 21 Aug 2016, Andreas Schwab wrote: > > + TEST_I_F(TItype, UTItype, _Float128, FLT128X_MANT_DIG, FLT128X_MAX_EXP); > > s/_Float128/_Float128x/ Thanks, fixed. gcc/testsuite: 2016-08-22 Joseph Myers * gcc.dg/torture/fp-int-convert-float128x-timode.c,

GCC 6.3 Status Report (2016-08-22)

2016-08-22 Thread Richard Biener
Status == GCC 6.2 has been released, the GCC 6 branch is now open for regression and documentation fixes again. Quality Data Priority # Change from last report --- --- P10 P2 124 - 3 P3

[PATCH][v2] GIMPLE store merging pass

2016-08-22 Thread Kyrill Tkachov
Hi all, This is a v2 of the patch at [1] addressing feedback from Richi. The most major changes are in the first phase of the pass that detects store chains. This now uses a hash_map to track multiple chains to different bases, eliminating the need to iterate over a basic block multiple times,

[PATCH] Fix PR77305

2016-08-22 Thread Richard Biener
The following robustifies statsistics accounting against current_pass being NULL. Bootstrap running on x86_64-unknown-linux-gnu. Richard. 2016-08-22 Richard Biener PR middle-end/77305 * statistics.c (statistics_counter_event): Robustify against

[PATCH] Improve VRP for stmts that can throw (PR27336)

2016-08-22 Thread Richard Biener
We currently fail to assert that this != NULL at the caller side because we're chickening out on stmts that could throw. While it would be trivial to fix the testcase with only bailing out for internally throwing stmts the following properly handles those as well by making sure to infer stuff

Re: [wwwdocs,libstdc++] Remove libstdc++/

2016-08-22 Thread Jonathan Wakely
On 22/08/16 10:27 +0100, Jonathan Wakely wrote: On 20/08/16 20:36 +0200, Gerald Pfeifer wrote: Our libstdc++/ page dates back to when libstdc++ was merged into GCC in 2000, 0x10 years ago. It has been a mostly empty shell for most of that time period and hardly seen any updates in the last

Re: [wwwdocs,libstdc++] Remove libstdc++/

2016-08-22 Thread Jonathan Wakely
On 20/08/16 20:36 +0200, Gerald Pfeifer wrote: Our libstdc++/ page dates back to when libstdc++ was merged into GCC in 2000, 0x10 years ago. It has been a mostly empty shell for most of that time period and hardly seen any updates in the last decade. So, after

Re: [PATCH] remove HARD_FRAME_POINTER_IS_ARG_POINTER macro

2016-08-22 Thread Bernd Schmidt
On 08/20/2016 07:07 PM, tbsaunde+...@tbsaunde.org wrote: From: Trevor Saunders Hi, its kind of silly, and this allows us to remove a few more #ifdefs. #if FRAME_POINTER_REGNUM != ARG_POINTER_REGNUM -#if HARD_FRAME_POINTER_IS_ARG_POINTER +#if

[PATCH] Use RPO order for fwprop iteration

2016-08-22 Thread Richard Biener
This changes it from PRE on the inverted graph to RPO order which works better for loops and blocks with no path to exit. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2016-08-22 Richard Biener * tree-ssa-forwprop.c

[BUILDROBOT] x86_64: Segmentation fault during -fself-test (was: Implement C _FloatN, _FloatNx types [version 6]

2016-08-22 Thread Jan-Benedict Glaw
Hi Joseph! On Wed, 2016-08-17 20:17:07 +, Joseph Myers wrote: [...] > ISO/IEC TS 18661-3:2015 defines C bindings to IEEE interchange and > extended types, in the form of _FloatN and _FloatNx type names with > corresponding fN/FN and fNx/FNx constant suffixes and

Re: C++11? (Re: protected alloca class for malloc fallback)

2016-08-22 Thread Richard Biener
On Mon, Aug 22, 2016 at 9:18 AM, Trevor Saunders wrote: > On Sun, Aug 21, 2016 at 09:00:26PM +0100, Pedro Alves wrote: >> On 08/20/2016 03:29 AM, Mike Stump wrote: >> > On Aug 10, 2016, at 10:03 AM, Oleg Endo wrote: >> >> >> >> Or just wait until

Re: [PATCH] Restrict jump threading statement simplifier to scalar types (PR71077)

2016-08-22 Thread Richard Biener
On Sat, Aug 20, 2016 at 1:25 AM, Patrick Palka wrote: > On Fri, 19 Aug 2016, Yuri Rumyantsev wrote: > >> Hi, >> >> Here is a simple test-case to reproduce 176.gcc failure (I run it on >> Haswell machine). >> Using 20160819 compiler build we get: >> gcc -O3 -m32 -mavx2 test.c

Re: [PATCH] Fix PR77290

2016-08-22 Thread Richard Biener
On Sun, 21 Aug 2016, Andreas Schwab wrote: > $ gcc/gfortran -B gcc/ -B aarch64-suse-linux/libgfortran/ -std=legacy > -floop-nest-optimize -O2 -S ../gcc/testsuite/gfortran.dg/graphite/pr68279.f90 > isl_aff.c:1001: position out of bounds > ../gcc/testsuite/gfortran.dg/graphite/pr68279.f90:8:0: >

Re: [PATCH] Fix sporadic failure of gcc.dg/cpp/pr66415-1.c

2016-08-22 Thread Richard Biener
On Sat, Aug 20, 2016 at 7:53 AM, Bernd Edlinger wrote: > Hi, > > it turns out that in this test case the expected output depends on the COLUMNS > setting of the terminal where the test suite is started. With less than 82 > columns > the multiline output is not as

Re: [PATCH] Handle VECTOR_CST in integer_nonzerop()

2016-08-22 Thread Richard Biener
On Mon, Aug 22, 2016 at 9:10 AM, Richard Biener wrote: > On Sat, Aug 20, 2016 at 1:38 AM, Patrick Palka wrote: >> On Fri, Aug 19, 2016 at 7:30 PM, Patrick Palka wrote: >>> integer_nonzerop() currently unconditionally

Re: C++11? (Re: protected alloca class for malloc fallback)

2016-08-22 Thread Trevor Saunders
On Sun, Aug 21, 2016 at 09:00:26PM +0100, Pedro Alves wrote: > On 08/20/2016 03:29 AM, Mike Stump wrote: > > On Aug 10, 2016, at 10:03 AM, Oleg Endo wrote: > >> > >> Or just wait until people have agreed to switch to C++11 or C++14. I > >> don't think in practice anybody

Re: [PATCH] Handle VECTOR_CST in integer_nonzerop()

2016-08-22 Thread Richard Biener
On Sat, Aug 20, 2016 at 1:38 AM, Patrick Palka wrote: > On Fri, Aug 19, 2016 at 7:30 PM, Patrick Palka wrote: >> integer_nonzerop() currently unconditionally returns false for a >> VECTOR_CST argument. This is confusing because one would expect that

  1   2   >