Re: [PATCH] Be less conservative in process_{output,input}_constraints (PR target/65689)

2015-04-08 Thread Jakub Jelinek
On Thu, Apr 09, 2015 at 12:20:40AM +0200, Jakub Jelinek wrote: > On Wed, Apr 08, 2015 at 05:16:08PM -0500, Segher Boessenkool wrote: > > On Wed, Apr 08, 2015 at 05:12:07PM -0500, Segher Boessenkool wrote: > > > On Wed, Apr 08, 2015 at 11:00:59PM +0200, Jakub Jelinek wrote: > > > > +case MATCH_C

Re: [PATCH] Fix ICE with x86_64 alloca (PR target/65693)

2015-04-08 Thread Uros Bizjak
On Wed, Apr 8, 2015 at 10:52 PM, Jakub Jelinek wrote: > This patch extends the PR65220 patch: > 1) there is really no reason to limit the divisor to 32 or 64, >we can divide/modulo by pow2 constants up to 2G (0x7fff is >then still representable in 32-bit signed immediate) > 2) on the

breakage with "[PATCH] combine: Disregard clobbers in another test for two SETs (PR65693)"

2015-04-08 Thread Hans-Peter Nilsson
On Wed, 8 Apr 2015, Segher Boessenkool wrote: > 2015-04-08 Segher Boessenkool > > * combine.c (is_parallel_of_n_reg_sets): Change first argument > from an rtx_insn * to an rtx. > (try_combine): Adjust both callers. Use it once more. That "once more" is outside of #ifndef HAVE

Re: [PATCH] combine: Disregard clobbers in another test for two SETs (PR65693)

2015-04-08 Thread Jeff Law
On 04/08/2015 03:33 PM, Segher Boessenkool wrote: PR65693 exposes a case where combine does a worse job after my patches to split parallels before combining. We start with a parallel of an udiv and an umod, and a clobber; the umod is dead. The instruction is combined with one setting the diviso

Re: [PATCH] Be less conservative in process_{output,input}_constraints (PR target/65689)

2015-04-08 Thread Jakub Jelinek
On Wed, Apr 08, 2015 at 05:16:08PM -0500, Segher Boessenkool wrote: > On Wed, Apr 08, 2015 at 05:12:07PM -0500, Segher Boessenkool wrote: > > On Wed, Apr 08, 2015 at 11:00:59PM +0200, Jakub Jelinek wrote: > > > +case MATCH_CODE: > > > + if (*XSTR (exp, 1) == '\0') > > > + { > > > + const

Re: [PATCH] Be less conservative in process_{output,input}_constraints (PR target/65689)

2015-04-08 Thread Segher Boessenkool
On Wed, Apr 08, 2015 at 05:12:07PM -0500, Segher Boessenkool wrote: > On Wed, Apr 08, 2015 at 11:00:59PM +0200, Jakub Jelinek wrote: > > +case MATCH_CODE: > > + if (*XSTR (exp, 1) == '\0') > > + { > > + const char *code, *codes = XSTR (exp, 0); > > + int ret = 0; > > + while

Re: [PATCH] Be less conservative in process_{output,input}_constraints (PR target/65689)

2015-04-08 Thread Segher Boessenkool
On Wed, Apr 08, 2015 at 11:00:59PM +0200, Jakub Jelinek wrote: > +case MATCH_CODE: > + if (*XSTR (exp, 1) == '\0') > + { > + const char *code, *codes = XSTR (exp, 0); > + int ret = 0; > + while ((code = scan_comma_elt (&codes)) != 0) > + if (strncmp (code, "re

[PATCH] combine: Disregard clobbers in another test for two SETs (PR65693)

2015-04-08 Thread Segher Boessenkool
PR65693 exposes a case where combine does a worse job after my patches to split parallels before combining. We start with a parallel of an udiv and an umod, and a clobber; the umod is dead. The instruction is combined with one setting the divisor pseudo to a power-of-two constant, so we end up wi

[PATCH] Be less conservative in process_{output,input}_constraints (PR target/65689)

2015-04-08 Thread Jakub Jelinek
Hi! Right now, stmt.c on constraints not hardcoded in it, and not define_{register,address,memory}_constraint just assumes the constraint might allow both reg and mem. Unfortunately, on some constraints which clearly can't allow either of those leads to errors at -O0, because the expander doesn't

[PATCH] Fix ICE with x86_64 alloca (PR target/65693)

2015-04-08 Thread Jakub Jelinek
Hi! This patch extends the PR65220 patch: 1) there is really no reason to limit the divisor to 32 or 64, we can divide/modulo by pow2 constants up to 2G (0x7fff is then still representable in 32-bit signed immediate) 2) on the testcase RTL cprop unfortunately isn't performed, because

Re: [PATCH, bfin] handle BFIN_CPU_UNKNOWN in TARGET_CPU_CPP_BUILTINS

2015-04-08 Thread Bernhard Reutner-Fischer
ping On 1 April 2015 at 23:34, Bernhard Reutner-Fischer wrote: > Hi, > > gcc/c-family/c-cppbuiltin.c > In file included from ./tm.h:21:0, > from > ../../../../../../home/me/src/gcc-5.0.mine/gcc/c-family/c-cppbuiltin.c:23: > ../../../../../../home/me/src/gcc-5.0.mine/gcc/c-family/

Re: [PATCH, c6x] handle unk_isa in TARGET_CPU_CPP_BUILTINS

2015-04-08 Thread Bernhard Reutner-Fischer
ping On 2 April 2015 at 22:49, Bernhard Reutner-Fischer wrote: > On Wed, Apr 01, 2015 at 11:37:41PM +0200, Bernhard Reutner-Fischer wrote: >>Bernd, >> >>same for c6x for unk_isa, fwiw. > > Attached. > > Ok for trunk for the c6x bits? > Ok for trunk for the bfin bits? > > thanks,

Re: [PATCH] mklog: Fallback to env author name and addr

2015-04-08 Thread Bernhard Reutner-Fischer
On 8 April 2015 at 21:45, Diego Novillo wrote: > On Wed, Apr 8, 2015 at 3:36 PM, Bernhard Reutner-Fischer > wrote: >> contrib/ChangeLog: >> >> 2015-04-08 Bernhard Reutner-Fischer >> >> * mklog ($name, $addr): Fallback to env author settings. > > This looks fine, but note that I no long

Re: [PATCH] mklog: Fallback to env author name and addr

2015-04-08 Thread Diego Novillo
On Wed, Apr 8, 2015 at 3:36 PM, Bernhard Reutner-Fischer wrote: > contrib/ChangeLog: > > 2015-04-08 Bernhard Reutner-Fischer > > * mklog ($name, $addr): Fallback to env author settings. This looks fine, but note that I no longer have approval rights for patches. Code in contrib/ has lo

Re: [PATCH, CHKP] Fix static const bounds creation in LTO

2015-04-08 Thread Ilya Enkovich
On 07 Apr 22:43, Jan Hubicka wrote: > > > > 2015-04-07 Ilya Enkovich > > > > * tree-chkp.c (chkp_find_const_bounds_var): Remove. > > (chkp_make_static_const_bounds): Search existing > > symbol by assembler name. Use make_decl_one_only. > > > > gcc/testsuite/ > > > > 2015-04-07

Re: [patch] Fix shared_timed_mutex::try_lock_until() et al

2015-04-08 Thread Jonathan Wakely
On 08/04/15 20:11 +0100, Jonathan Wakely wrote: We can get rid of the _Mutex type then, and just use std::mutex, and that also means we can provide the timed locking functions even when !defined(_GTHREAD_USE_MUTEX_TIMEDLOCK). And so maybe we should use this fallback implementation instead of the

[PATCH] mklog: Fallback to env author name and addr

2015-04-08 Thread Bernhard Reutner-Fischer
contrib/ChangeLog: 2015-04-08 Bernhard Reutner-Fischer * mklog ($name, $addr): Fallback to env author settings. --- contrib/mklog | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/contrib/mklog b/contrib/mklog index f7974a7..4cad351 100755 --- a/contrib/mklog +++

Re: [patch] Fix shared_timed_mutex::try_lock_until() et al

2015-04-08 Thread Jonathan Wakely
On 08/04/15 18:59 +0200, Torvald Riegel wrote: +// set or the maximum number of reader locks is held, then increment the +// reader lock count. +// To release decrement the count, then if the write-entered flag is set +// and the count is zero then signal gate2 to wake a queued wr

Re: [PATCH 1/4] Docs: extend.texi: Add missing semicolon for consistency

2015-04-08 Thread Gerald Pfeifer
Hi Michael, and big apologies for this falling through a lot of cracks apparently. I just committed your patch with the ChangeLog below. If there are any other patches that have not been committed (nor NACKed yet, I know there were some as well), please let us know and I will look into getting a

Re: MAINTAINERS: resign as testsuite maintainer, update address

2015-04-08 Thread Gerald Pfeifer
On Mon, 2 Feb 2015, Janis Johnson wrote: > I retired from Mentor Graphics 3 weeks ago and have no immediate plans > to be active in GCC, so I'm resigning as a testsuite maintainer. I'm > leaving myself under Write After Approval with my personal email address > so people can find me. Thanks for y

Re: [PATCH 1/2] tree.h: Commentary typo fix

2015-04-08 Thread Bernhard Reutner-Fischer
On 8 April 2015 at 20:34, Gerald Pfeifer wrote: > On Wed, 8 Apr 2015, Bernhard Reutner-Fischer wrote: >> 2015-04-01 Bernhard Reutner-Fischer >> >> * tree.h (CONVERT_EXPR_P): Commentary typo fix. > > Go ahead. Pure comment fixes still should be fine at this point > in time. Ok, thanks. r

[PATCH] PR target/55144: bfin: fix opening glibc-c.o: No such file or directory

2015-04-08 Thread Bernhard Reutner-Fischer
building all-gcc for bfin-linux-uclibc results in build/genchecksum cp/cp-lang.o c-family/stub-objc.o ... glibc-c.o \ libbackend.a .. > cc1plus-checksum.c.tmp opening glibc-c.o: No such file or directory make[2]: *** [cc1-checksum.c] Error 1 Fix this by prepending tmake_file which nowadays consis

Re: [PATCH 1/2] tree.h: Commentary typo fix

2015-04-08 Thread Gerald Pfeifer
On Wed, 8 Apr 2015, Bernhard Reutner-Fischer wrote: > 2015-04-01 Bernhard Reutner-Fischer > > * tree.h (CONVERT_EXPR_P): Commentary typo fix. Go ahead. Pure comment fixes still should be fine at this point in time. Gerald

[patch] Update my email address

2015-04-08 Thread Cary Coutant
I'm retiring, and my last day at google is this Friday, April 10. I plan to continue to contribute to GCC and binutils in my retirement. I've updated the MAINTAINERS file to use my personal address, ccout...@gmail.com. -cary 2015-04-08 Cary Coutant * MAINTAINERS: Update my email add

[PATCH 2/2] tree-tailcall: Commentary typo fix, remove fwd declaration

2015-04-08 Thread Bernhard Reutner-Fischer
gcc/ChangeLog: 2015-04-01 Bernhard Reutner-Fischer * tree-tailcall.c (suitable_for_tail_opt_p, find_tail_calls): Remove unneeded forward declarations. (suitable_for_tail_call_opt_p): Commentary typo fix. Signed-off-by: Bernhard Reutner-Fischer --- gcc/tree-tailcall.c

[PATCH 1/2] tree.h: Commentary typo fix

2015-04-08 Thread Bernhard Reutner-Fischer
gcc/ChangeLog: 2015-04-01 Bernhard Reutner-Fischer * tree.h (CONVERT_EXPR_P): Commentary typo fix. Signed-off-by: Bernhard Reutner-Fischer --- gcc/tree.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gcc/tree.h b/gcc/tree.h index 4fcc272..bedf103 100644 --- a/g

[PATCH 0/2] Commentary typo fixes

2015-04-08 Thread Bernhard Reutner-Fischer
Two typos and remove two unneeded forward declarations for static functions in tree-tailcall.c. These patches were applied when doing a config-list.mk build and showed no negative effect. I'd usually consider them obvious but given current stage.. Ok for trunk now? Bernhard Reutner-Fischer (2):

[doc] extend.texi grammar fix

2015-04-08 Thread Gerald Pfeifer
I ran into this while trying to help a user. Committed. Gerald 2015-04-08 Gerald Pfeifer * doc/extend.texi (__sync Builtins): Fix grammar. Index: doc/extend.texi === --- doc/extend.texi (revision 221926) +++ doc/ext

Re: [PATCH] Update {x86_64,i[34]86,aarch64,s390{,x},powerpc64}-linux baseline_symbols.txt

2015-04-08 Thread Jonathan Wakely
On 08/04/15 19:40 +0200, Jakub Jelinek wrote: Hi! Attached patch updates baseline_symbols.txt for a couple of architectures. Don't have 32-bit powerpc-linux around though this time. Ok for trunk? OK, thanks very much for doing this.

[PATCH] emit_bss(): Remove redundant guard

2015-04-08 Thread Bernhard Reutner-Fischer
gcc/ChangeLog: 2015-04-01 Bernhard Reutner-Fischer * varasm.c (emit_bss): Remove redundant guard. Signed-off-by: Bernhard Reutner-Fischer --- gcc/varasm.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/gcc/varasm.c b/gcc/varasm.c index 537a64d..2bb5f27 100644 --- a/gcc/varasm.

[PATCH, doc, committed] cfg.texi (GIMPLE statement iterators): Fix typo

2015-04-08 Thread Bernhard Reutner-Fischer
Hi, Committed as r221926. 2015-04-08 Bernhard Reutner-Fischer * doc/cfg.texi (GIMPLE statement iterators): Fix typo. Index: gcc/doc/cfg.texi === --- gcc/doc/cfg.texi (revision 221925) +++ gcc/doc/cfg.texi (revision 221926) @@ -54

Re: Fix increase_alignment

2015-04-08 Thread Bernhard Reutner-Fischer
On 2 April 2015 at 01:20, Jan Hubicka wrote: >> >> Your follow-up patch 88ada5e935d58223ae2d9ce6d0c1c71c372680a8 a.k.a r221269 >> added this to emit_local(): >> >> static bool >> -emit_local (tree decl ATTRIBUTE_UNUSED, >> +emit_local (tree decl, >> const char *name ATTRIBUTE_UNUSED,

Re: [PATCH] add self-tuning to x86 hardware fast path in libitm

2015-04-08 Thread Andi Kleen
> On the STAMP suite of benchmarks for transactional memory (described here > [1]). > I have ran an unmodified GCC 5.0.0 against the patched GCC with these > modifications and obtain the following speedups in STAMP with 4 > threads (on a Haswell with 4 cores, average 10 runs): I expect you'll nee

[PATCH] Update {x86_64,i[34]86,aarch64,s390{,x},powerpc64}-linux baseline_symbols.txt

2015-04-08 Thread Jakub Jelinek
Hi! Attached patch updates baseline_symbols.txt for a couple of architectures. Don't have 32-bit powerpc-linux around though this time. Ok for trunk? 2015-04-07 Jakub Jelinek * config/abi/post/x86_64-linux-gnu/baseline_symbols.txt: Update. * config/abi/post/x86_64-linux-gnu/3

Re: [C++ PATCH] Fix alignment handling in build_cplus_array_type/cp_build_qualified_type_real (PR c++/65690)

2015-04-08 Thread Jakub Jelinek
On Wed, Apr 08, 2015 at 06:32:29PM +0200, Jan Hubicka wrote: > > On Wed, Apr 08, 2015 at 06:22:10PM +0200, Jan Hubicka wrote: > > > > On 04/08/2015 06:02 AM, Jakub Jelinek wrote: > > > > > (cp_build_qualified_type_real): Use check_base_type. Build a > > > > > variant and copy over even

Re: [patch] Fix shared_timed_mutex::try_lock_until() et al

2015-04-08 Thread Torvald Riegel
There is an correctness issue related to mutex destruction. The added documentation is a good start, but I'd still add some more for the complicated pieces of reasoning. Details inline below. On Tue, 2015-04-07 at 15:28 +0100, Jonathan Wakely wrote: > diff --git a/libstdc++-v3/include/std/shared

Re: [WEB][PATCH] Describe -pg and LTO changes

2015-04-08 Thread Gerald Pfeifer
On Sun, 16 Nov 2014, Andi Kleen wrote: > This patch describes some user visible changes that were > added to gcc 5. Thanks, Andi! I added some environments, broke up a long one, changed "a LTO build" to "an LTO build", Index: changes.html ===

Re: [C++ PATCH] Fix alignment handling in build_cplus_array_type/cp_build_qualified_type_real (PR c++/65690)

2015-04-08 Thread Jan Hubicka
> On Wed, Apr 08, 2015 at 06:22:10PM +0200, Jan Hubicka wrote: > > > On 04/08/2015 06:02 AM, Jakub Jelinek wrote: > > > > (cp_build_qualified_type_real): Use check_base_type. Build a > > > > variant and copy over even TYPE_CONTEXT and > > > > TYPE_ALIGN/TYPE_USER_ALIGN if any of those are differen

Re: [C++ PATCH] Fix alignment handling in build_cplus_array_type/cp_build_qualified_type_real (PR c++/65690)

2015-04-08 Thread Jakub Jelinek
On Wed, Apr 08, 2015 at 06:22:10PM +0200, Jan Hubicka wrote: > > On 04/08/2015 06:02 AM, Jakub Jelinek wrote: > > > (cp_build_qualified_type_real): Use check_base_type. Build a > > > variant and copy over even TYPE_CONTEXT and > > > TYPE_ALIGN/TYPE_USER_ALIGN if any of those are different. >

Re: [C++ PATCH] Fix alignment handling in build_cplus_array_type/cp_build_qualified_type_real (PR c++/65690)

2015-04-08 Thread Jan Hubicka
> On 04/08/2015 06:02 AM, Jakub Jelinek wrote: > > (cp_build_qualified_type_real): Use check_base_type. Build a > > variant and copy over even TYPE_CONTEXT and > > TYPE_ALIGN/TYPE_USER_ALIGN if any of those are different. > > This seems wrong. If there is an array with the same name,

Re: [v3] Update Solaris baselines

2015-04-08 Thread Jonathan Wakely
On 08/04/15 15:41 +0200, Rainer Orth wrote: With the GCC 5 release approaching, it's time to update the Solaris baselines again. This patch does just that and is pretty much straightforward: * With one exception, all new symbols are in the GLIBCXX_3.4.21 and CXXABI_1.3.9 versions. * On Solari

Re: libgomp nvptx plugin: rework initialisation and support the proposed load/unload hooks (was: Merge current set of OpenACC changes from gomp-4_0-branch)

2015-04-08 Thread Julian Brown
On Wed, 8 Apr 2015 17:58:56 +0300 Ilya Verbin wrote: > Have you tested it with disabled offloading? > > I see several regressions: > FAIL: libgomp.oacc-c/../libgomp.oacc-c-c++-common/acc_on_device-1.c > -DACC_DEVICE_TYPE_host_nonshm=1 -DACC_MEM_SHARED=0 execution test > FAIL: libgomp.oacc-c/../l

Re: [PATCH] add self-tuning to x86 hardware fast path in libitm

2015-04-08 Thread Nuno Diegues
Thank you for the feedback. Comments inline. On Wed, Apr 8, 2015 at 3:05 PM, Andi Kleen wrote: > > Nuno Diegues writes: > > What workloads did you test this on? On the STAMP suite of benchmarks for transactional memory (described here [1]). I have ran an unmodified GCC 5.0.0 against the patch

Re: [C++ PATCH] Fix alignment handling in build_cplus_array_type/cp_build_qualified_type_real (PR c++/65690)

2015-04-08 Thread Jakub Jelinek
On Wed, Apr 08, 2015 at 10:47:15AM -0400, Jason Merrill wrote: > On 04/08/2015 06:02 AM, Jakub Jelinek wrote: > > (cp_build_qualified_type_real): Use check_base_type. Build a > > variant and copy over even TYPE_CONTEXT and > > TYPE_ALIGN/TYPE_USER_ALIGN if any of those are different. >

Re: libgomp nvptx plugin: rework initialisation and support the proposed load/unload hooks (was: Merge current set of OpenACC changes from gomp-4_0-branch)

2015-04-08 Thread Ilya Verbin
On Wed, Apr 08, 2015 at 15:31:42 +0100, Julian Brown wrote: > This version is mostly the same as the last posted version but has a > tweak in GOACC_parallel to account for the new splay tree arrangement > for target functions: > > - tgt_fn = (void (*)) tgt_fn_key->tgt->tgt_start; > + tgt

Re: [C++ PATCH] Fix alignment handling in build_cplus_array_type/cp_build_qualified_type_real (PR c++/65690)

2015-04-08 Thread Jason Merrill
On 04/08/2015 06:02 AM, Jakub Jelinek wrote: (cp_build_qualified_type_real): Use check_base_type. Build a variant and copy over even TYPE_CONTEXT and TYPE_ALIGN/TYPE_USER_ALIGN if any of those are different. This seems wrong. If there is an array with the same name, at

Re: libgomp nvptx plugin: rework initialisation and support the proposed load/unload hooks (was: Merge current set of OpenACC changes from gomp-4_0-branch)

2015-04-08 Thread Jakub Jelinek
On Wed, Apr 08, 2015 at 03:31:42PM +0100, Julian Brown wrote: > It passes libgomp testing on NVPTX. OK? Please write a proper ChangeLog entry for it. Ok with that. Jakub

Re: libgomp nvptx plugin: rework initialisation and support the proposed load/unload hooks (was: Merge current set of OpenACC changes from gomp-4_0-branch)

2015-04-08 Thread Julian Brown
On Tue, 7 Apr 2015 17:26:45 +0200 Jakub Jelinek wrote: > On Mon, Apr 06, 2015 at 03:45:57PM +0300, Ilya Verbin wrote: > > On Wed, Apr 01, 2015 at 15:20:25 +0200, Jakub Jelinek wrote: > > > LGTM with proper ChangeLog entry. > > > > I've commited this patch into trunk. > > > > Julian, you probabl

Re: [libgo] Remove Solaris 11.1+ zone_net_addr_t treatment

2015-04-08 Thread Ian Lance Taylor
On Wed, Apr 8, 2015 at 7:10 AM, Rainer Orth wrote: > Ian Lance Taylor writes: > >> On Wed, Apr 8, 2015 at 6:48 AM, Rainer Orth >> wrote: >>> Ian Lance Taylor writes: >>> On Mon, Nov 3, 2014 at 8:59 AM, Rainer Orth wrote: > > The recent godump changes broke Solaris 11.1+ boo

Re: [libgo] Remove Solaris 11.1+ zone_net_addr_t treatment

2015-04-08 Thread Rainer Orth
Ian Lance Taylor writes: > On Wed, Apr 8, 2015 at 6:48 AM, Rainer Orth > wrote: >> Ian Lance Taylor writes: >> >>> On Mon, Nov 3, 2014 at 8:59 AM, Rainer Orth >>> wrote: The recent godump changes broke Solaris 11.1+ bootstrap in libgo: before, gen-sysinfo.so had type

Re: [libgo] Remove Solaris 11.1+ zone_net_addr_t treatment

2015-04-08 Thread Ian Lance Taylor
On Wed, Apr 8, 2015 at 6:48 AM, Rainer Orth wrote: > Ian Lance Taylor writes: > >> On Mon, Nov 3, 2014 at 8:59 AM, Rainer Orth >> wrote: >>> >>> The recent godump changes broke Solaris 11.1+ bootstrap in libgo: >>> before, gen-sysinfo.so had >>> >>> type _zone_net_addr_t struct { zna_family ui

Re: [PATCH] fix building for alpha-dec-vms

2015-04-08 Thread Jakub Jelinek
On Wed, Apr 08, 2015 at 03:57:09PM +0200, Bernhard Reutner-Fischer wrote: > [CC ing maintainers] > > Ping. This is ok. > > --- a/gcc/ChangeLog > > +++ b/gcc/ChangeLog > > @@ -1,3 +1,9 @@ > > +2015-03-27 Trevor Saunders > > + > > + * config/alpha/alpha.c (alpha_use_linkage): Change type

Re: [PATCH] add self-tuning to x86 hardware fast path in libitm

2015-04-08 Thread Andi Kleen
Nuno Diegues writes: What workloads did you test this on? > +static inline float fastLog(float x) > +{ > + union { float f; uint32_t i; } vx = { x }; > + float y = vx.i; > + y *= 8.2629582881927490e-8f; > + return y - 87.989971088f; > +} > + > +static inline float fastSqrt(float x) > +{ > +

Re: [PATCH] fix building for alpha-dec-vms

2015-04-08 Thread Richard Biener
On Fri, Mar 27, 2015 at 11:24 AM, wrote: > From: Trevor Saunders > > Hi, > > Unfortunately when testing r217869 I didn't realize the modified code in > alpha.c was only used for some alpha targets. So testing alpha-linux wasn't > enough or even really useful :( > > I tested cc1 for alpha-dec-vm

Re: [PATCH] fix building for alpha-dec-vms

2015-04-08 Thread Bernhard Reutner-Fischer
[CC ing maintainers] Ping. On 27 March 2015 at 11:24, wrote: > From: Trevor Saunders > > Hi, > > Unfortunately when testing r217869 I didn't realize the modified code in > alpha.c was only used for some alpha targets. So testing alpha-linux wasn't > enough or even really useful :( > > I teste

Re: [PATCH] Walk through thunks when propagating comdat group

2015-04-08 Thread Ilya Enkovich
On 07 Apr 22:37, Jan Hubicka wrote: > > OK, thanks! > Please put the comment about inline clones just before the global.inlined_to > test > and make the comment about thunks separate. > Probably "Thunks can not call across section boundary" > > Honza Here is a committed variant. Thanks, Ilya -

[v3] Update Solaris baselines

2015-04-08 Thread Rainer Orth
With the GCC 5 release approaching, it's time to update the Solaris baselines again. This patch does just that and is pretty much straightforward: * With one exception, all new symbols are in the GLIBCXX_3.4.21 and CXXABI_1.3.9 versions. * On Solaris/x86 only (obviously), we have +OBJECT:0:CX

Re: [libgo] Remove Solaris 11.1+ zone_net_addr_t treatment

2015-04-08 Thread Rainer Orth
Ian Lance Taylor writes: > On Mon, Nov 3, 2014 at 8:59 AM, Rainer Orth > wrote: >> >> The recent godump changes broke Solaris 11.1+ bootstrap in libgo: >> before, gen-sysinfo.so had >> >> type _zone_net_addr_t struct { zna_family uint16; zna_plen uint16; >> zna_addru struct { znau_addr6 _in6_ad

[testsuite, i386] Avoid finite in gcc.target/i386/avx512dq-vfpclasspd-2.c etc.

2015-04-08 Thread Rainer Orth
Some AVX512DQ tests are failing on Solaris/x86 with gas: FAIL: gcc.target/i386/avx512dq-vfpclasspd-2.c (test for excess errors) FAIL: gcc.target/i386/avx512dq-vfpclassps-2.c (test for excess errors) FAIL: gcc.target/i386/avx512vl-vfpclasspd-2.c (test for excess errors) FAIL: gcc.target/i386/avx512

Re: libgo patch committed: Build libnetgo.a

2015-04-08 Thread Lynn A. Boger
I see there is some documentation about build constraints in go/go/build/doc.go. There is no mention of netgo on this page now but this seems like an appropriate spot to mention the use of the netgo tag for gc and then how to achieve the equivalent effect with gccgo. On 04/07/2015 01:09 PM, Ian

Re: [PATCH, bootstrap]: Add bootstrap-lto-noplugin build configuration (PR65537)

2015-04-08 Thread Gerald Pfeifer
On Mon, 6 Apr 2015, Sandra Loosemore wrote: > s/frontend/front end/ ?? > > (Since we have "middle end" and "back end" on the previous line.) Absolutely -- I should have caught this. Is also is in line with https://gcc.gnu.org/codingconventions.html. Here is the updated patch I just committed.

Re: [ping] Re: proper name of i386/x86-64/etc targets

2015-04-08 Thread Gerald Pfeifer
Hi Sandra, On Mon, 26 Jan 2015, Sandra Loosemore wrote: OK, here is a patch that attempts to implement that convention. I'd appreciate review from a target maintainer to check that I've correctly disambiguated places where "i386" was referring to both 32- and 64-bit variants vs 32-bit only. I'

[PATCH, i386] Fix PR target/65676.

2015-04-08 Thread Kirill Yukhin
Hello, Patch in the bottom fixes PR65676. Bootstrapped, reg-testing is in progress. I am going to commit if testing will pass. I am also going back port the patch to 4.9.x. gcc/ PR target/65676 * config/i386/i386.c (fixup_modeless_constant): New. (ix86_expand_args_builtin

Re: pr59016

2015-04-08 Thread Mikael Morin
Le 08/04/2015 12:29, Mikael Morin a écrit : > Except that the following ... ahem ... doesn't work. > And it doesn't work because gfc_get_ha_symbol doesn't version host-associated symbols. So one has to call symbol.c's save_symbol_data by hand. And then, we can as well keep the original gfc_find_s

Re: [patch,avr]: Part2: Fix various problems with specs and specs file generation.

2015-04-08 Thread Georg-Johann Lay
Am 04/08/2015 um 10:28 AM schrieb Denis Chertykov: 2015-04-07 15:34 GMT+03:00 Georg-Johann Lay : Am 04/06/2015 um 11:54 AM schrieb Sivanupandi, Pitchumani: Hi Johann, Did you try running g++ tests? It seems xgcc is invoked to get multilibs (from gcc/testsuite/lib/g++.exp) which failed to find

Re: [wwwdocs] Update changes.html with libstdc++ changes

2015-04-08 Thread Jonathan Wakely
On 08/04/15 13:06 +0200, Gerald Pfeifer wrote: On Sat, 6 Dec 2014, Jonathan Wakely wrote: I'm also noting one old change in the GCC 4.5 page, and removing/changing some links to the C++0x status table. The list of features supported on trunk is fairly irrelevant to someone looking at the 4.4 rel

Re: Revert "PowerPC shrink-wrap support 3 of 3"

2015-04-08 Thread Gerald Pfeifer
On Thu, 10 Nov 2011, Hans-Peter Nilsson wrote: > I think I need someone with appropriate write privileges to > agree with that, and to also give 48h for someone to fix the > problem. Sorry for not forthcoming on the second point. > > brgds, H-P > PS. where is the policy written down, besides the

Re: [wwwdocs] Update changes.html with libstdc++ changes

2015-04-08 Thread Gerald Pfeifer
On Sat, 6 Dec 2014, Jonathan Wakely wrote: > I'm also noting one old change in the GCC 4.5 page, and > removing/changing some links to the C++0x status table. The list of > features supported on trunk is fairly irrelevant to someone looking at > the 4.4 release notes, so I've linked to the docs for

Re: pr59016

2015-04-08 Thread Mikael Morin
Le 07/04/2015 14:25, Mikael Morin a écrit : > Le 06/04/2015 20:26, Mikael Morin a écrit : >> Regarding the patch, I don't understand why the existing symbol >> restoration code doesn't work here (see >> gfc_restore_last_undo_checkpoint, restore_old_symbol). I have to >> investigate more. > > I th

[C++ PATCH] Fix alignment handling in build_cplus_array_type/cp_build_qualified_type_real (PR c++/65690)

2015-04-08 Thread Jakub Jelinek
Hi! The following patches (included or attached) fix a regression on WebKit compilation. The first hunk basically reverts Honza's patch from December/January, because layout_type when the variants are already linked in doesn't layout just the current type, but also forcefully overwrites all the ot

Re: [libstdc++/65033] Give alignment info to libatomic

2015-04-08 Thread Jonathan Wakely
On 07/04/15 23:58 -0400, Hans-Peter Nilsson wrote: I'd expect alignof(ai): 4 .is_lock_free(): 1 No... wait, that's because atomic_base.h doesn't have the natural-alignment fix, so it's still broken for less-than-natural-alignment targets. But will be fixed? Yes, with my uncommitted patch to a

[Ada] Fix incorrect call to Pure function returning discriminated type

2015-04-08 Thread Eric Botcazou
This disables incorrect optimization (mainly CSE) of calls to Pure functions returning a discriminated record type. These functions allocate their return value on the secondary stack and thus calls to them cannot be CSE'ed because the stack can be reclaimed in between. Tested on x86_64-suse-li

Re: [patch,avr]: Part2: Fix various problems with specs and specs file generation.

2015-04-08 Thread Denis Chertykov
2015-04-07 15:34 GMT+03:00 Georg-Johann Lay : > Am 04/06/2015 um 11:54 AM schrieb Sivanupandi, Pitchumani: >> >> Hi Johann, >> >> Did you try running g++ tests? >> It seems xgcc is invoked to get multilibs (from gcc/testsuite/lib/g++.exp) >> which failed to find specs file. > > > > This is because

RE: [PATCH] Fix bdverN vector cost of cond_[not_]taken_branch_cost

2015-04-08 Thread Richard Biener
On Wed, 8 Apr 2015, Gopalasubramanian, Ganesh wrote: > > I have added a person from AMD to comment on the decision. > > Otherwise, the patch looks OK, but please wait a couple of days for > > possible comments. > > Thank you Uros! > I am checking the changes with few tests and benchmarking them.

Re: [PATCH, wwwdoc] Describe the changes of NDS32 port in GCC-5.0.

2015-04-08 Thread Gerald Pfeifer
On Wed, 25 Feb 2015, Chung-Ju Wu wrote: > Committed as revision 1.82 of htdocs/gcc-5/changes.html with minor > adjustment. Thanks for adding those release notes! I just applied a number of editorial changes on top of the original patch. Let me know if there are further changes you'd like to se