Re: [PATCH] Allow `make tags` to work from top-level directory

2016-10-13 Thread Jeff Law
On 10/06/2016 07:21 AM, Eric Gallager wrote: The libdecnumber, libgcc, and libobjc subdirectories are missing TAGS targets in their Makefiles. The attached patch causes them to be skipped when running `make tags`. ChangeLog entry: 2016-10-06 Eric Gallager * Makefile.def: Mark libdec

Re: RFC: Split into smaller pieces

2016-10-13 Thread Jonathan Wakely
Apparently this got spam-filtered and didn't make it to the lists... On 13/10/16 18:34 +0100, Jonathan Wakely wrote: This splits the large (2200 lines) header into smaller pieces, so there are separate headers for: - std::less, std::equal_to etc. (already in their own header) - std::__invoke (

Re: RFC: Split into smaller pieces

2016-10-13 Thread Jonathan Wakely
On 13/10/16 19:19 +0100, Jonathan Wakely wrote: On 13/10/16 18:34 +0100, Jonathan Wakely wrote: Code which doesn't need the whole of should include the relevant header instead. This means that we don't need to pull the whole of (and and ) into just because shared_ptr wants to use reference

Re: [C++ PATCH] RFC: implement P0386R2 - C++17 inline variables

2016-10-13 Thread Jason Merrill
On Tue, Oct 11, 2016 at 9:39 AM, Jakub Jelinek wrote: > Here is an attempt to implement C++17 inline variables. > Bootstrapped/regtested on x86_64-linux and i686-linux. > > The main question is if the inline variables, which are vague linkage, > should be !DECL_EXTERNAL or DECL_EXTERNAL DECL_NOT_R

[v3] Remove 'test' variables from a few more testsuite dirs

2016-10-13 Thread Paolo Carlini
Hi, nothing especially interesting here... Tested x86_64-linux. Thanks, Paolo. // 2016-10-13 Paolo Carlini * testsuite/24_iterators/container_access.cc: Remove 'test' variables. * testsuite/24_iterators/istream_iterator/2.cc: Likewise. * testsuit

[PATCH] Test cases for PR77937

2016-10-13 Thread Bill Schmidt
Here are torture test cases for https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77937. Markus Trippelsdorf kindly provided the source for the tests and verified the correct dejagnu options on x86_64-pc-linux-gnu. Committed. Thanks, Bill 2016-10-13 Bill Schmidt PR tree-optimization/779

Re: [PATCH] Test cases for PR77937

2016-10-13 Thread Rainer Orth
Hi Bill, > Here are torture test cases for > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77937. Markus Trippelsdorf > kindly provided the source for the tests and verified the correct > dejagnu options on x86_64-pc-linux-gnu. Committed. > > Thanks, > Bill > > > 2016-10-13 Bill Schmidt > >

Re: [PATCH, libfortran] PR 48587 Newunit allocator

2016-10-13 Thread Jerry DeLisle
On 10/13/2016 08:16 AM, Janne Blomqvist wrote: Currently GFortran newer reuses unit numbers allocated with NEWUNIT=, instead having a simple counter that is decremented each time such a unit is opened. For a long running program which repeatedly opens files with NEWUNIT= and closes them, the cou

Re: PR35503 - warn for restrict pointer

2016-10-13 Thread Prathamesh Kulkarni
On 7 October 2016 at 10:33, Prathamesh Kulkarni wrote: > On 22 September 2016 at 23:15, Joseph Myers wrote: >> On Thu, 22 Sep 2016, Prathamesh Kulkarni wrote: >> >>> Would that be acceptable ? I am not sure how to make %Z check if the >>> argument has type vec * >>> since vec is not really a buil

Re: [PATCH v4] PR48344: Fix unrecognizable insn error with -fstack-limit-register=r2

2016-10-13 Thread Andreas Schwab
I've committed this to fix the ICE. Andreas. * config/m68k/m68k.c (m68k_option_override): Check opt_fstack_limit_symbol_arg and opt_fstack_limit_register_no instead of stack_limit_rtx. * gcc.target/m68k/stack-limit-1.c: Expect warning on line 0. diff --git a/gcc/

Re: [PATCH] Allow `make tags` to work from top-level directory

2016-10-13 Thread Eric Gallager
On 10/13/16, Jeff Law wrote: > On 10/06/2016 07:21 AM, Eric Gallager wrote: >> The libdecnumber, libgcc, and libobjc subdirectories are missing TAGS >> targets in their Makefiles. The attached patch causes them to be >> skipped when running `make tags`. >> >> ChangeLog entry: >> >> 2016-10-06 Eri

Re: [PATCH, libfortran] PR 48587 Newunit allocator

2016-10-13 Thread Dominique d'Humières
With the patch, the following code integer :: i, j i = -10 write(unit=i,fmt=*, iostat=j) 10 print *, j end fails at run time with Assertion failed: (ind >= 0 && ind < newunit_size), function newunit_free, file ../../../work/libgfortran/io/unit.c, line 966. Without the patch the output is 5002.

Re: [PATCH] Remove x86 pcommit instruction

2016-10-13 Thread H.J. Lu
On Thu, Oct 13, 2016 at 5:09 AM, Andrew Senkevich wrote: > 2016-10-11 20:09 GMT+03:00 H.J. Lu : >> On Tue, Oct 11, 2016 at 10:04 AM, Andrew Senkevich >> wrote: >>> 2016-10-06 1:07 GMT+03:00 H.J. Lu : On Wed, Oct 5, 2016 at 1:42 PM, Andrew Senkevich wrote: > 2016-10-05 18:06 GMT+03:

Re: Questionable code in gcov-io.c

2016-10-13 Thread Andrew Pinski
On Wed, Oct 12, 2016 at 11:24 AM, Nathan Sidwell wrote: > On 10/12/16 11:04, Andreas Schwab wrote: > >> Do we still need to call fstat? I don't think it can ever fail here. > > > Update removing the fstat. Survived a profiled bootstrap, so I'll commit > tomorrow, unless there are further comment

Re: PING [PATCH] accept flexible arrays in struct in unions (c++/71912 - [6/7 regression])

2016-10-13 Thread Martin Sebor
On 10/12/2016 07:43 AM, Jason Merrill wrote: On Tue, Oct 11, 2016 at 9:45 PM, Martin Sebor wrote: Are there any other changes you want me to make to the patch? I leave this weekend for the WG14 meeting and would like to get this change finalized and hopefully committed before then. https://g

Re: Set nonnull attribute to ptr_info_def based on VRP

2016-10-13 Thread kugan
Hi Richard, On 13/10/16 20:44, Richard Biener wrote: On Thu, Oct 13, 2016 at 6:49 AM, kugan wrote: Hi Richard, what does this try to do? Preserve info VRP computed across PTA? I think we didn't yet sort out the nonlocal/escaped vs. null handling properly (or how PTA should handle get_ptr_

Re: [ipa-vrp] Use get/set_ptr_nonnull in ipa-vrp

2016-10-13 Thread kugan
Hi Honza, On 12/10/16 22:16, Jan Hubicka wrote: Hi, This patch uses the get/set_ptr_nonnull so that ipa-vrp also propagates nonnull ranges for pinter. Bootstrapped and regression tested this with other patched without any new regressions on x86_64-linux-gnu. Is this OK for trunk? Thanks, Kug

[patch, fortran] PR77972 ICE on broken character continuation with -Wall etc.

2016-10-13 Thread Jerry DeLisle
This patch is straight forward. We were sending bogus locus info to the diagnostics machinery and catch an assert in error,c. The patch avoids doing this. Regression tested on x86-64-linux. OK for trunk? Regards, Jerry 2016-10-13 Jerry DeLisle * scanner.c (gfc_next_char_literal)

Re: [patch, fortran] PR77972 ICE on broken character continuation with -Wall etc.

2016-10-13 Thread Steve Kargl
On Thu, Oct 13, 2016 at 07:04:04PM -0700, Jerry DeLisle wrote: > This patch is straight forward. We were sending bogus locus info to the > diagnostics machinery and catch an assert in error,c. > > The patch avoids doing this. > > Regression tested on x86-64-linux. > > OK for trunk? > Yes, but

Re: [ping * 2] remove optab functions for [us]divmod_optab in optabs.def

2016-10-13 Thread Prathamesh Kulkarni
On 13 October 2016 at 22:54, Bernd Schmidt wrote: > On 10/13/2016 07:18 PM, Prathamesh Kulkarni wrote: >> >> On 13 October 2016 at 16:56, Bernd Schmidt wrote: >>> >>> On 10/06/2016 07:43 AM, Prathamesh Kulkarni wrote: Pinging patch: https://gcc.gnu.org/ml/gcc-patches/2016-09/msg010

<    1   2