[patch, committed] Bug 83613 - [8 Regression] Executing gfortran.dg/inquire_internal.f90 hangs on darwin

2017-12-29 Thread Jerry DeLisle
Committed as obvious after regression testing on x85_64-pc-linux-gnu. Dominiq: Please confirm is fixed. Regards, Jerry Author: jvdelisle Date: Fri Dec 29 22:36:25 2017 New Revision: 256035 URL: https://gcc.gnu.org/viewcvs?rev=256035=gcc=rev Log: 2017-12-29 Jerry DeLisle

Re: [PATCH] Use pointer sized array indices.

2017-12-29 Thread Janne Blomqvist
On Fri, Dec 29, 2017 at 7:17 PM, Thomas Koenig wrote: > Hi Janne, > >> Using pointer sized variables (e.g. size_t / ptrdiff_t) when the >> variables are used as array indices allows accessing larger arrays, >> and can be a slight performance improvement due to no need for

Re: [patch, lingfortran] Bug 83560 - list-directed formatting of INTEGER is missing plus on output

2017-12-29 Thread Thomas Koenig
Hi Jerry, OK for trunk? Looks good, thanks for the patch! Looking at the number of test cases, namelist has to be one of the most complicated things in all of Fortran... Regards Thomas

Re: [PATCH] Use pointer sized array indices.

2017-12-29 Thread Thomas Koenig
Hi Janne, Using pointer sized variables (e.g. size_t / ptrdiff_t) when the variables are used as array indices allows accessing larger arrays, and can be a slight performance improvement due to no need for sign or zero extending, or masking. Unless I have missed something, all the examples

Re: [PATCH] Add selftest for vec::reverse

2017-12-29 Thread David Malcolm
On Fri, 2017-12-29 at 15:39 +0100, Martin Liška wrote: > On 10/06/2017 06:20 PM, David Malcolm wrote: > > Martin: I noticed that your switch expansion patch added a > > vec::reverse () > > method. Here's a proposed selftest for it, mostly to verify > > that it handles even vs odd lengths (which

[v2 of PATCH 03/14] C++: add location_t wrapper nodes during parsing (minimal impl)

2017-12-29 Thread David Malcolm
On Fri, 2017-12-15 at 13:58 -0500, Jason Merrill wrote: > On Fri, Dec 15, 2017 at 11:35 AM, David Malcolm > wrote: > > On Fri, 2017-12-15 at 10:01 -0500, Jason Merrill wrote: > > > On Thu, Dec 14, 2017 at 2:25 PM, David Malcolm > > om> > > > wrote: > > >

[PATCH] Use pointer sized array indices.

2017-12-29 Thread Janne Blomqvist
Using pointer sized variables (e.g. size_t / ptrdiff_t) when the variables are used as array indices allows accessing larger arrays, and can be a slight performance improvement due to no need for sign or zero extending, or masking. Regtested on x86_64-pc-linux-gnu, Ok for trunk?

Re: debug container mutex association

2017-12-29 Thread Andreas Schwab
On Sep 19 2016, François Dumont wrote: > diff --git > a/libstdc++-v3/testsuite/23_containers/vector/debug/mutex_association.cc > b/libstdc++-v3/testsuite/23_containers/vector/debug/mutex_association.cc > new file mode 100644 > index 000..a3c56e2 > --- /dev/null > +++

Re: [PATCH] Implement smart multiple switch expansion algorithms.

2017-12-29 Thread Martin Liška
On 10/06/2017 03:46 PM, Wilco Dijkstra wrote: > Martin Liska wrote: > >> There are some numbers for cc1plus: >> >> $ bloaty ./objdir2/gcc/cc1plus -- ./objdir/gcc/cc1plus >> VM SIZE FILE SIZE >> +3.8% +1.11Mi TOTAL +1.03Mi +0.5% > >> insn-attrtab.o: >> VM

Re: [PATCH] Add selftest for vec::reverse

2017-12-29 Thread Martin Liška
On 10/06/2017 06:20 PM, David Malcolm wrote: > Martin: I noticed that your switch expansion patch added a > vec::reverse () > method. Here's a proposed selftest for it, mostly to verify > that it handles even vs odd lengths (which it does). > > Only lightly tested; hope this is useful. > Dave

Re: [PATCH] Implement smart multiple switch expansion algorithms.

2017-12-29 Thread Martin Liška
On 10/06/2017 07:24 PM, David Malcolm wrote: > On Fri, 2017-10-06 at 14:25 +0200, Martin Liška wrote: >> Hello. >> >> As presented at this year's Cauldron, I rewrote current switch >> expansion to support >> multiple algorithms (jump table and bit-test) that can be used just >> for a fraction >>

Re: a new libgcov interface: __gcov_dump_all

2017-12-29 Thread Martin Liška
On 10/26/2017 10:47 AM, Martin Liška wrote: > On 07/22/2014 06:04 PM, Xinliang David Li wrote: >> Please take a look the updated patch. It addresses the issue of using >> dlclose before dump, and potential races (between a thread closing a >> library and the dumper call). >> >> David >> >> On Sun,

[RFC] Add vec::ordered_remove_if

2017-12-29 Thread Tom de Vries
[ was: Re: [libgomp, openacc, openmp, PR83046] Prune removed funcs from offload table ] On 12/28/2017 05:06 PM, Jakub Jelinek wrote: On Thu, Dec 28, 2017 at 04:53:29PM +0100, Tom de Vries wrote: --- a/gcc/lto-cgraph.c +++ b/gcc/lto-cgraph.c @@ -,6 +,16 @@ output_offload_tables (void)

[PATCH] PR 78534 Change character length from int to size_t

2017-12-29 Thread Janne Blomqvist
In order to handle large character lengths on (L)LP64 targets, switch the GFortran character length from an int to a size_t. This is an ABI change, as procedures with character arguments take hidden arguments with the character length. I also changed the _size member in vtables from int to

[RFC] Statistics of new ASAN tests: -fsanitize=pointer-{compare,subtract}

2017-12-29 Thread Martin Liška
Hello. AS you've probably already noted, me and Jakub installed patches for the new ASAN feature. It does Address sanitization of pointer comparison (or subtraction) and finds situations where there pointers point to a different object. And as memory layout is a randomness, that can lead to a

Re: [libgomp, openacc, openmp, PR83046] Prune removed funcs from offload table

2017-12-29 Thread Tom de Vries
On 12/28/2017 05:14 PM, Jakub Jelinek wrote: On Thu, Dec 28, 2017 at 05:06:57PM +0100, Jakub Jelinek wrote: This has O(n^2) complexity for n == vec_safe_length (offload_funcs). Can't you instead just have 2 IVs, one for where we read the vector elt and one for where we write it if the 2 are

Re: [nvptx, committed] Disable -gstatement-frontiers for nvptx

2017-12-29 Thread Tom de Vries
On 12/29/2017 05:12 AM, Alexandre Oliva wrote: On Dec 27, 2017, Tom de Vries wrote: .loc file_index line_number column_position so this causes ptxas errors when compiling something for nvptx with -g, which breaks the nvptx build. What do the errors look like?

[PATCH] Remove out-of-date comment

2017-12-29 Thread A. Skrobov
As of currently, `-fmessy-debugging` is not a valid option, nor is `REG_USERVAR` looked at by the splitting logic. 2017-12-29 Artyom Skrobov tyomi...@gmail.com * web.c: Remove out-of-date comment --- gcc/web.c | 4 1 file changed, 4 deletions(-) diff --git a/gcc/web.c b/gcc/web.c index