Re: [patch] Restore cross-language inlining into Ada

Hi Eric, On 23/01/16 10:25, Eric Botcazou wrote: I think we was inlining them with LTO until I installed the patch. Most of time DECL_STRUCT_FUNCTION == NULL for WPA and thus the original check testing the flags was disabled. We did not update the EH coddegen during inlining, so probably we

[PATCH, testsuite]: Define _GNU_SOURCE for gcc.target/i386/chkp-strlen-2.c

Hello! gcc.target/i386/chkp-strlen-2.c testcase uses stpcpy, which with older glibc requires _GNU_SOURCE. The patch also moves some definitions from compile flags to the source. 2016-01-23 Uros Bizjak * gcc.target/i386/chkp-strlen-2.c: Define _GNU_SOURCE. *

Re: [PATCH 4/4][AArch64] Cost CCMP instruction sequences to choose better expand order

"Wilco Dijkstra" writes: > diff --git a/gcc/testsuite/gcc.target/aarch64/ccmp_1.c > b/gcc/testsuite/gcc.target/aarch64/ccmp_1.c > index ef077e0..7c39b61 100644 > --- a/gcc/testsuite/gcc.target/aarch64/ccmp_1.c > +++ b/gcc/testsuite/gcc.target/aarch64/ccmp_1.c > @@ -80,5

Re: [patch] Restore cross-language inlining into Ada

> I think we was inlining them with LTO until I installed the patch. Most of > time DECL_STRUCT_FUNCTION == NULL for WPA and thus the original check > testing the flags was disabled. We did not update the EH coddegen during > inlining, so probably we just did not produce non-call EH for these.

Re: [PATCH, PR69426] Fix clobber removal in parloops

On January 23, 2016 10:34:46 AM GMT+01:00, Tom de Vries wrote: >Hi, > >this patches fixes a 5/6 regression PR69426. > >When compiling the test-case in the patch, the verify_ssa todo check >fails after removing a clobber in eliminate_local_variables_stmt. > >The problem

[PATCH, 69217]: [6 Regression] ICE at var-tracking.c:5038 Segmentation fault

Hi, This fixes the regression seen from a change at revision 231897. Unfortunately I don't have any tests as it was noticed in my frontend (D). And equivalent C/C++ doesn't trigger because: - C set the mode of empty struct types as BLKmode. - C++ lowered passing around empty struct types as

[PATCH] Skip re-computing the mips frame info after reload completed

Hi, this patch skips the redundant re-computing of the frame info after reload completed. I looked at all available targets initial_elimination_offset functions: All of them currently use either a trivial function of crtl->outgoing_args_size, get_frame_size () and df_regs_ever_live_p (x),

Re: [PATCH] Fix up "the the" in the comments

On January 23, 2016 11:25:48 AM GMT+01:00, Jakub Jelinek wrote: >On Fri, Jan 22, 2016 at 11:09:59AM +0100, Jakub Jelinek wrote: >> > /* The inverse of wi_unpack. IN_LEN is the the number of input >> >> I know you haven't touched this line and it is preexisting, but when >>

[patch,ira]: Improve on updated memory cost in coloring pass of integrated register allocator.

This patch improves the updated memory cost in coloring pass of integrated register allocator. Only enter_freq of the loop is considered in updated memory cost in the coloring pass. Consideration of only enter_freq is based on the concept that live Out of the entry or header of the Loop is

Re: [patch] Restore cross-language inlining into Ada

> OK, we may have inlined them after all... My understanding of the new code is > that we will still inline them if the Ada callee doesn't use EH, which is good > enough in my opinion. Agreed.

[PATCH] Fix up "the the" in the comments

On Fri, Jan 22, 2016 at 11:09:59AM +0100, Jakub Jelinek wrote: > > /* The inverse of wi_unpack. IN_LEN is the the number of input > > I know you haven't touched this line and it is preexisting, but when > touching this, please also fix the "the the". I'll leave this one to Richard, but found

Re: [patch, fortran] Inline MATMUL(A,TRANSPOSE(B)), PR 66094

Hi Toon, However, today I *did* run the test harness with your modification: ... Thanks for the testing! So, what do people think? Is the patch OK for trunk? Regards Thomas

[PATCH, PR69426] Fix clobber removal in parloops

Hi, this patches fixes a 5/6 regression PR69426. When compiling the test-case in the patch, the verify_ssa todo check fails after removing a clobber in eliminate_local_variables_stmt. The problem is that the clobber is removed, but the uses of the corresponding vdef are not changed, and the

Re: [PATCH, PR69110] Don't return NULL access_fns in dr_analyze_indices

On 23/01/16 18:39, Sebastian Pop wrote: diff --git a/gcc/tree-data-ref.c b/gcc/tree-data-ref.c index a40f40d..7ff5db7 100644 --- a/gcc/tree-data-ref.c +++ b/gcc/tree-data-ref.c @@ -1023,6 +1023,10 @@ dr_analyze_indices (struct data_reference *dr, loop_p nest, loop_p loop) build_int_cst

Re: [PATCH, PR69110] Don't return NULL access_fns in dr_analyze_indices

On Sat, Jan 23, 2016 at 12:28 PM, Tom de Vries wrote: > That was my original patch, and Richard commented: 'I think avoiding a NULL > access_fns is ok but it should be done unconditionally, not only for the > DECL_P case'. In order words, he asked me to do the exact

[PATCH] jit: Fix missing references to pthread in jit-playback.c

Hi, I noticed when building from 2016-01-17 snapshot that the JIT frontend failed to build. --- jit-playback.c:2075:36: error: ‘PTHREAD_MUTEX_INITIALIZER’ was not declared in this scope jit-playback.c: In member function ‘void gcc::jit::playback::context::acquire_mutex()’:

Re: [PATCH] Fix failure of gfortran.dg/backtrace_1.f90 on hppa*-*-hpux*

On 01/23/2016 02:40 PM, John David Anglin wrote: > Ping. > > On 2015-12-28, at 2:58 PM, John David Anglin wrote: > >> The hppa*-*-hpux* target does not support __sync builtins. As a result, >> libbacktrace does not >> support backtraces when threads are active. >> >> Instead of always assuming

NetBSD has SSP in libc

NetBSD has SSP included in libc: http://cvsweb.netbsd.org/bsdweb.cgi/src/lib/libc/ssp/?only_with_tag=MAIN gcc/configure does not know about this. The attached patch (against SVN checkout from today) fixes this. I've previously reported this as https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68380

Re: [PATCH] Fix failure of gfortran.dg/backtrace_1.f90 on hppa*-*-hpux*

Ping. On 2015-12-28, at 2:58 PM, John David Anglin wrote: > The hppa*-*-hpux* target does not support __sync builtins. As a result, > libbacktrace does not > support backtraces when threads are active. > > Instead of always assuming threads are active in > libgfortran/runtime/backtrace.c,

Re: [patch, fortran] Inline MATMUL(A,TRANSPOSE(B)), PR 66094

On 01/23/2016 04:26 AM, Thomas Koenig wrote: > Hi Toon, > >> However, today I *did* run the test harness with your modification: > > ... > > Thanks for the testing! > > So, what do people think? Is the patch OK for trunk? > > Regards > > Thomas > Yes, OK. Jerry

Re: [PATCH, rs6000] Fix PR63354

On Thu, 2016-01-21 23:42:40 -0600, Bill Schmidt wrote: > On Thu, 2016-01-21 at 21:21 -0600, Bill Schmidt wrote: > > The testcase will need a slight adjustment, as currently it fails on > > powerpc64 with -m32 testing. Working on a fix. > > This patch adjusts the

[committed] Fix libstdc++/69446

_GLIBCXX_USE_WEAK_REF needs to be defined to 0 to fix build error. Tested on hppa2.0w-hp-hpux11.11. Dave -- John David Anglin dave.ang...@bell.net 2016-01-23 John David Anglin PR libstdc++/69446 * config/os/hpux/os_defines.h

Re: Patch RFA: Add option -fcollectible-pointers, use it in ivopts

On 01/22/2016 12:03 PM, Ian Lance Taylor wrote: Index: common.opt === --- common.opt (revision 232580) +++ common.opt (working copy) @@ -1380,6 +1380,10 @@ Enable hoisting adjacent loads to encourage generating conditional move