Re: Stage3 closing soon, call for patch pings

2015-01-19 Thread Daniel Micay
On 15/01/15 04:26 PM, H.J. Lu wrote: > On Thu, Jan 15, 2015 at 1:04 PM, Jeff Law wrote: >> >> Stage3 is closing rapidly. I've drained my queue of patches I was tracking >> for gcc-5.However, note that I don't track everything. If it's a patch >> for a backend, language other than C or seemin

Re: [PATCH] Workaround -Wmaybe-uninitialized false positives during profiledbootstrap

2015-01-19 Thread Martin Liška
On 01/19/2015 03:48 PM, Jakub Jelinek wrote: On Mon, Jan 19, 2015 at 03:44:20PM +0100, Martin Liška wrote: There's a bunch of another places that emit false positives. With following patch, I am able to run profiledbootstrap on x86_64-linux-pc and ppc64-linux-pc. I don't know what we consider

Re: GCC 5 Status Report (2015-01-19), Trunk in Stage 4

2015-01-19 Thread H.J. Lu
On Mon, Jan 19, 2015 at 1:17 AM, Richard Biener wrote: > > Status > == > > The trunk is now in "Stage 4" which means it is open for regression > and documentation fixes only, like if it were a release branch. > > Please concentrate on getting P1 bugs fixed and provide help in > confirming and

[PATCH] PR 64645: Fix libffi under cygwin-32

2015-01-19 Thread Bernd Edlinger
Hi, this patch fixes libffi build problems under cygwin-32. See PR 64645. Successfully boot-strapped under Cygwin-32/XP with libjava. OK for trunk? Thanks Bernd. 2015-01-19 Bernd Edlinger PR libffi/64645 * src/x86/sysv.S: Exclude t

Re: GCC 5 Status Report (2015-01-19), Trunk in Stage 4

2015-01-19 Thread Joseph Myers
On Mon, 19 Jan 2015, Jakub Jelinek wrote: > On Mon, Jan 19, 2015 at 05:32:39PM +, Jonathan Wakely wrote: > > I would like to commit these two patches which complete the C++11 > > library implementation: > > > > https://gcc.gnu.org/ml/gcc-patches/2015-01/msg01694.html > > https://gcc.gnu.org/m

Re: [PATCH] PR ada/64640: Fix Ada bootstrap under cygwin.

2015-01-19 Thread Arnaud Charlet
> this patch fixes the Ada bootstrap under cygwin-32. See PR ada/64640. > > Boot-strapped successfully under cygwin-32/XP. > OK for trunk? OK

Re: [C++ PATCH, RFC] PR c++/63959, continued

2015-01-19 Thread Ville Voutilainen
On 19 January 2015 at 18:28, Ville Voutilainen wrote: > When I patched the triviality test for volatile types, I missed two cases: > 1) volatile members in a class should make the class non-trivial. > 2) a volatile class type should itself be non-trivial. > (based on [basic.types]/9, [class]/6, [c

[PATCH, committed] Fix int vs bool issue in jit.dg/test-arith-overflow.c

2015-01-19 Thread David Malcolm
This jit testcase was failing on s390x and ppc64 due to carelessly assuming int==bool. Patch fixes the testcase on s390x and ppc64. Also tested on x86_64 and aarch64. Committed to trunk as r219856. gcc/testsuite/ChangeLog: * jit.dg/test-arith-overflow.c: Include stdbool.h. (crea

Re: Use static chain and libffi for Go closures

2015-01-19 Thread Uros Bizjak
On Mon, Jan 19, 2015 at 7:14 PM, Richard Henderson wrote: >> Attached is a more conservative v2 patch that conditionally enables >> fixup code when FFI_TARGET_HAS_COMPLEX_TYPE is not defined. >> >> Tested on x86_64-linux-gnu {-m32} without regressions. > > I don't think this version should be req

patch to fix PR64671

2015-01-19 Thread Vladimir Makarov
The following patch fixes https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64671 The patch was successfully bootstrapped on x86-64.I don't expect that the patch can result in any trouble. Committed as rev. 219857. 2015-01-19 Vladimir Makarov PR rtl-optimization/64671 *

Speedup WPA on libreoffice

2015-01-19 Thread Jan Hubicka
Hi, libreoffice WPA gets somewhat slower compared to 4.8 because of more work we do analyzing polymorphic call. Many of these calls don't need analyzis because we will not derive anything useful anyway. In this case it is a win to simply drop the info. Bootstrapped/regtested x86_64-linux, comit

Re: [PATCH] Fix PR ipa/64668

2015-01-19 Thread Jan Hubicka
> Hello. > > This is fix for PR64668, where I did a wrong assumption about an arguments > of OBJ_TYPE_REF tree type. > > I've been testing the patch on x86_64-linux-pc, ready after it finishes? > > Thanks, > Martin > >From 54840c592a866eca0a535cdda3714b4251042153 Mon Sep 17 00:00:00 2001 > From

Fix ICE building boost testsuite

2015-01-19 Thread Jan Hubicka
Index: ChangeLog === --- ChangeLog (revision 219858) +++ ChangeLog (working copy) @@ -1,5 +1,11 @@ 2015-01-19 Jan Hubicka + PR ipa/64218 + * ipa-inline.c (want_inline_function_to_all_callers_p): Fix check +

Re: [AArch64] Add a new scheduling description for the ARM Cortex-A57 processor

2015-01-19 Thread Gerald Pfeifer
On Monday 2015-01-19 17:52, James Greenhalgh wrote: > OK after the Cortex-A57 scheduling description goes in to the ARM port? Yes, thanks, except that once will be sufficient. ;-) (The current patch features the same hunk twice?) Gerald

Re: [AArch64] Add a new scheduling description for the ARM Cortex-A57 processor

2015-01-19 Thread James Greenhalgh
On Mon, Jan 19, 2015 at 08:57:31PM +, Gerald Pfeifer wrote: > On Monday 2015-01-19 17:52, James Greenhalgh wrote: > > OK after the Cortex-A57 scheduling description goes in to the ARM port? > > Yes, thanks, except that once will be sufficient. ;-) (The current > patch features the same hunk t

Re: [PATCH] Fix scheduling undeterminism from sorting with DEBUG_INSNs

2015-01-19 Thread Mike Stump
On Jan 19, 2015, at 10:14 AM, Maxim Kuvyrkov wrote: > On Jan 19, 2015, at 6:48 PM, Mike Stump wrote: >> On Jan 19, 2015, at 5:41 AM, Maxim Kuvyrkov >> wrote: >>> In A < B < C < A case all A, B and C are normal instructions. It is a >>> pre-existing condition. When compiling without debug inf

Re: [PATCH] Fix scheduling undeterminism from sorting with DEBUG_INSNs

2015-01-19 Thread Maxim Kuvyrkov
On Jan 20, 2015, at 12:11 AM, Mike Stump wrote: > On Jan 19, 2015, at 10:14 AM, Maxim Kuvyrkov > wrote: >> On Jan 19, 2015, at 6:48 PM, Mike Stump wrote: >>> On Jan 19, 2015, at 5:41 AM, Maxim Kuvyrkov >>> wrote: In A < B < C < A case all A, B and C are normal instructions. It is a >>

RE: [PATCH] Fix for PR64081 in RTL loop unroller

2015-01-19 Thread David Edelsohn
Unfortunately this patch broke bootstrap on AIX (PPC32) PR bootstrap/64676 - David

Re: Use static chain and libffi for Go closures

2015-01-19 Thread Richard Henderson
On 01/19/2015 02:26 AM, Andreas Schwab wrote: > Ian Lance Taylor writes: > >> @@ -83,7 +83,7 @@ >> #else /* !defined(USE_LIBFFI_CLOSURES) */ >> >> void >> -makeFuncFFI(const struct __go_func_type *ftyp, ffi_go_closure *impl) >> +makeFuncFFI(const struct __go_func_type *ftyp, void *impl) >> {

Reduce inline-unit-growth to 15%

2015-01-19 Thread Jan Hubicka
Hi, this patch reduces unit growth to 15%. This is important change for LTO builds. For example Firefox binary reduces by about 10%. I hope this change is now safe with increased early-inlining-insns and improvements in inline analysis. I benchmarked this on Firefox, majority of C++ testsuite and

Re: [SH] Introduce treg_set_expr

2015-01-19 Thread Oleg Endo
On Sun, 2015-01-18 at 13:25 +0100, Oleg Endo wrote: > On Sat, 2015-01-17 at 22:40 +0900, Kaz Kojima wrote: > > Oleg Endo wrote: > > > Kaz, could you please test the patch on your sh4-linux setup and report > > > your findings? Even though it's a bit late, I'd like to get this in for > > > GCC 5,

RE: [PATCH,MIPS] Only pass floating-point options to the assembler then

2015-01-19 Thread Matthew Fortune
Hi Catherine, > The new behaviour of the GCC driver passing floating point options like > -msoft-float to the assembler is essential for the new o32 ABI > extensions but is a change in behaviour. In particular GCC 5 used with > binutils 2.24 would require a user to fix any hand-crafted code that >

RE: [PATCH,MIPS] Only pass floating-point options to the assembler then

2015-01-19 Thread Moore, Catherine
> -Original Message- > From: Matthew Fortune [mailto:matthew.fort...@imgtec.com] > Sent: Monday, January 19, 2015 5:54 PM > To: Moore, Catherine > Cc: 'gcc-patches@gcc.gnu.org' (gcc-patches@gcc.gnu.org) > Subject: RE: [PATCH,MIPS] Only pass floating-point options to the assembler > then >

Re: [SH] Introduce treg_set_expr

2015-01-19 Thread Oleg Endo
On Sun, 2015-01-18 at 13:25 +0100, Oleg Endo wrote: > On Sat, 2015-01-17 at 22:40 +0900, Kaz Kojima wrote: > > Oleg Endo wrote: > > > Kaz, could you please test the patch on your sh4-linux setup and report > > > your findings? Even though it's a bit late, I'd like to get this in for > > > GCC 5,

Re: [C++ PATCH, RFC] PR c++/63959, continued

2015-01-19 Thread Ville Voutilainen
On 19 January 2015 at 21:12, Ville Voutilainen wrote: > On 19 January 2015 at 18:28, Ville Voutilainen > wrote: >> When I patched the triviality test for volatile types, I missed two cases: >> 1) volatile members in a class should make the class non-trivial. >> 2) a volatile class type should its

RE: [PATCH,MIPS] Only pass floating-point options to the assembler then

2015-01-19 Thread Matthew Fortune
Moore, Catherine writes: > > -Original Message- > > From: Matthew Fortune [mailto:matthew.fort...@imgtec.com] > > Sent: Monday, January 19, 2015 5:54 PM > > To: Moore, Catherine > > Cc: 'gcc-patches@gcc.gnu.org' (gcc-patches@gcc.gnu.org) > > Subject: RE: [PATCH,MIPS] Only pass floating-poi

Re: [Patch, libstdc++/64584, libstdc++/64585] Clear basic_regex after imbue and make assign exception tolerant

2015-01-19 Thread Tim Shen
On Mon, Jan 19, 2015 at 5:25 AM, Jonathan Wakely wrote: > OK for trunk and 4.9 - thanks. Committed. -- Regards, Tim Shen

Re: [Patch, libstdc++/64649] Fix regex_traits::lookup_collatename and regex_traits::lookup_classname

2015-01-19 Thread Tim Shen
On Mon, Jan 19, 2015 at 5:23 AM, Jonathan Wakely wrote: > Maybe it's OK, because all the elements of __collatename use ASCII > chars which have the same value as wide chars? (Probably not true for > all locales). I just noticed that if we want to implement it 100% accurately, narrowing down the i

Re: [SH][committed] Fix PR 64652

2015-01-19 Thread Oleg Endo
On Sun, 2015-01-18 at 19:14 +0100, Oleg Endo wrote: > Hi, > > The attached patch fixes PR 64652. Tested with > make -k check-gcc RUNTESTFLAGS="sh-torture.exp --target_board=sh-sim > \{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb}" > Committed as r219824. > > Cheers, > Oleg > > gcc

Re: [Patch, libstdc++/64649] Fix regex_traits::lookup_collatename and regex_traits::lookup_classname

2015-01-19 Thread Tim Shen
On Mon, Jan 19, 2015 at 3:28 PM, Tim Shen wrote: > I just noticed that if we want to implement it 100% accurately, > narrowing down the input string to char string is clearly wrong. Let me take that back, since [22.4.1.1.2].12 requires that do_widen(do_narrow(c,0)) == c, which gives us a chance t

Re: [PATCH] Re: Stage 3 RFC: using "jit" for ahead-of-time compilation

2015-01-19 Thread David Malcolm
On Mon, 2015-01-19 at 10:51 +0100, Richard Biener wrote: > On Fri, Jan 16, 2015 at 7:47 PM, David Malcolm wrote: > > On Thu, 2015-01-15 at 22:50 +0100, Richard Biener wrote: > >> On January 15, 2015 9:05:59 PM CET, David Malcolm > >> wrote: > >> >Release managers: given that this only touches th

Re: [Patch, libstdc++/64649] Fix regex_traits::lookup_collatename and regex_traits::lookup_classname

2015-01-19 Thread Tim Shen
On Mon, Jan 19, 2015 at 3:52 PM, Tim Shen wrote: > I think it'll be a quick fix. Here it is :) -- Regards, Tim Shen commit a2058d87f3b9e2f00a11ae55c2598d00829022cd Author: timshen Date: Mon Jan 19 16:21:10 2015 -0800 PR libstdc++/64649 * include/bits/regex.tcc (regex_traits

Re: [PATCH] Fix PR64313

2015-01-19 Thread Joseph Myers
On Mon, 19 Jan 2015, Richard Biener wrote: > c/ > * c-decl.c (merge_decls): Call set_builtin_decl_declared_p > for builtins the user declared correctly. The C front-end change is OK. -- Joseph S. Myers jos...@codesourcery.com

Re: [PATCH] toplev.c: Process the failure when read fails for random_seed

2015-01-19 Thread Joseph Myers
On Mon, 19 Jan 2015, Chen Gang S wrote: > On 12/31/2014 06:26 AM, Joseph Myers wrote: > > On Mon, 29 Dec 2014, Chen Gang S wrote: > > > >> And in honest, this year what I have done is really not quite well, next > >> year I should be improved: should scanning Bugzilla and try to fix the > >> exis

[PATCH] pr 64076 - tolerate different definitions of symbols in lto

2015-01-19 Thread tbsaunde+gcc
From: Trevor Saunders Hi, when doing an lto link we can have some symbols be ir only and others be machine code, which trips the assert here. Just adjust the assert to handle that. bootstrapped + regtested x86_64-linux-gnu, ok? Trev gcc/ * ipa-visibility.c (update_visibility_by_reso

Re: [patch, Fortran] PR61933 Inquire on Internal Units

2015-01-19 Thread Jerry DeLisle
ping This PR has been marked as a regression, so I suppose we ought to get it fixed. On 01/18/2015 09:10 PM, Jerry DeLisle wrote: I reopened this PR to do some cleanup and to address a use case presented by Joost in comment #7 of the subject PR. The fundamental problem: if the variable contain

[PATCH] pr 60211 - error out if ivdep pragma is outside a function

2015-01-19 Thread tbsaunde+gcc
From: Trevor Saunders Hi, not doing $subject caused ICES when we tried to parse statements outside of functions. Just copy the similar code for other pragmas that effect statements. bootstrapped + regtested x86_64-linux-gnu, ok? Trev diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog index f6

Re: [PING] [PATCH] Fix parameters of __tsan_vptr_update

2015-01-19 Thread Konstantin Serebryany
[text-only] On Mon, Jan 19, 2015 at 7:42 AM, Mike Stump wrote: > On Jan 19, 2015, at 12:43 AM, Dmitry Vyukov wrote: >> I can't really make my mind on this. I would mildly prefer sleep's (if >> they work reliably!). > > Let me state it more forcefully. You don't have to convince us here. I'd love

Re: [PATCH] Fix scheduling undeterminism from sorting with DEBUG_INSNs

2015-01-19 Thread Mike Stump
On Jan 19, 2015, at 1:28 PM, Maxim Kuvyrkov wrote: > > Yes, the ordering relation is screwed, as you put it. With the number of > independent heuristics that rank_for_schedule has to consider there is no > other way then to have a "screwed" ordering. I have not seen a proof that an working co

Re: Fix 59828 - Broken assembly on ppc* with two -mcpu= options

2015-01-19 Thread David Edelsohn
On Fri, Jan 17, 2014 at 10:58 PM, Alan Modra wrote: > This patch cures PR59828 by translating all the -mcpu options at once, > in order, to their equivalent assembler -m options by using a new spec > function. In the process this removes some duplication. > > All the rhs of -mcpu= options from th

Re: Use static chain and libffi for Go closures

2015-01-19 Thread Ian Lance Taylor
On Mon, Jan 19, 2015 at 2:09 PM, Richard Henderson wrote: > > Fixed thus; applying to mainline after build testing a cross tomips64el-linux. > > Ian, I assume you'll push this upstream? Committed. (It's a little easier for me if I submit patches to the libgo and gcc/go/gofrontend directories.)

Re: Use static chain and libffi for Go closures

2015-01-19 Thread Ian Lance Taylor
On Sat, Jan 17, 2015 at 10:34 AM, Uros Bizjak wrote: > On Sat, Jan 17, 2015 at 12:19 AM, Ian Lance Taylor wrote: > >>> You should also revert alpha specific change to >>> libgo/go/testing/quick/quick_test.go, please see [1] and [2]. >>> >>> [1] https://gcc.gnu.org/ml/gcc-patches/2013-03/msg00038.

Make branch_cost indpenent of optimize_size

2015-01-19 Thread Jan Hubicka
Hi, this patch makes branch_cost to be stable over optimize_size and optimize to allow inlining. This is safe because BRANCH_COST macro will use constant of 2 for all branches optimized for size. Bootstrapped/regtested x86_64-linux, comitted. PR lto/45375 * i386.c (ix86_o

Re: libgo patch committed: Update to Go 1.4

2015-01-19 Thread Ian Lance Taylor
On Mon, Jan 19, 2015 at 4:17 AM, Rainer Orth wrote: > Ian Lance Taylor writes: > >> On Thu, Jan 15, 2015 at 8:30 AM, Rainer Orth >> wrote: >>> >>> Apart from that, bootstrap fails in gotools: due to the use of >>> -static-libgo, all commands there fail to link since the socket >>> functions are

[PATCH, ARM] Backport fix for PR59593 (minipool of small values on big endian targets)

2015-01-19 Thread Thomas Preud'homme
Currently on GCC 4.8 and 4.9, constant pool entries for QImode, HImode and SImode values are filled as 32-bit quantities. This works fine for little endian system but gives some incorrect results for big endian system when the value is accessed with a mode smaller than 32-bit in size. Suppose th

Re: Fix 59828 - Broken assembly on ppc* with two -mcpu= options

2015-01-19 Thread Alan Modra
On Mon, Jan 19, 2015 at 10:43:29PM -0500, David Edelsohn wrote: > On Fri, Jan 17, 2014 at 10:58 PM, Alan Modra wrote: > > This patch cures PR59828 by translating all the -mcpu options at once, > > in order, to their equivalent assembler -m options by using a new spec > > function. In the process

Re: [PATCH] Fix scheduling undeterminism from sorting with DEBUG_INSNs

2015-01-19 Thread Maxim Kuvyrkov
On Jan 20, 2015, at 6:32 AM, Mike Stump wrote: > On Jan 19, 2015, at 1:28 PM, Maxim Kuvyrkov wrote: >> >> Yes, the ordering relation is screwed, as you put it. With the number of >> independent heuristics that rank_for_schedule has to consider there is no >> other way then to have a "screwed

[PATCH] add includes in config/tilepro/mul-tables.c and config/tilegx/mul-tables.c

2015-01-19 Thread Prathamesh Kulkarni
Hi, When I committed r219655, I didn't check in tilepro/mul-tables.c since it was auto-generated from config/tilepro/gen-mul-tables.cc. Andrew pointed out to me that this causes the files confi/tilepro/mul-tables.c and config/tilegx/mul-tables.c to show up in svn diff. The attached patch fixes that

RE: [PING] [PATCH] Fix parameters of __tsan_vptr_update

2015-01-19 Thread Bernd Edlinger
Hi, On Mon, 19 Jan 2015 18:49:21, Konstantin Serebryany wrote: > > [text-only] > > On Mon, Jan 19, 2015 at 7:42 AM, Mike Stump wrote: >> On Jan 19, 2015, at 12:43 AM, Dmitry Vyukov wrote: >>> I can't really make my mind on this. I would mildly prefer sleep's (if >>> they work reliably!). >> >>

PATCH, nds32] Committed: Fix wrong data flow when building nds32 target with --enable-checking=yes.

2015-01-19 Thread Chung-Ju Wu
Hi, all, The nds32 target has two kinds of instructions for returning to the caller. One is normal 'ret' instruction which takes $lp as return address; and the other is 'pop25' instruction which perform registers pop and use $lp as return address in a single step. However, the current design of '

[PATCH, nds32] Committed: Have shrink-wrapping optimization to be performed on nds32 target.

2015-01-19 Thread Chung-Ju Wu
Hi, all, It is reported that the nds32 port on trunk cannot perform shrink-wrapping optimization because there is missing named pattern "simple_return". So we define that named pattern to fix the issue. This patch was committed last week before stage 4: https://gcc.gnu.org/r219711 gcc/ChangeL

[PATCH, nds32] Committed: Have sibling calls optmization to be performed on nds32 target.

2015-01-19 Thread Chung-Ju Wu
Hi, all, It is reported that the nds32 port on trunk cannot perform sibling call optimization because there are missing "sibcall" and "sibcall_value" named patterns. So we define those named patterns to fix the issue. This patch was committed last week before stage 4: https://gcc.gnu.org/r2197

Re: [PATCH, doc] NDS32: Describe -mcmodel= option instead of -mgp-direct in the documentation.

2015-01-19 Thread Chung-Ju Wu
2015-01-18 16:36 GMT+08:00 Gerald Pfeifer : > On Wednesday 2015-01-14 17:56, Chung-Ju Wu wrote: >> 2015-01-14 Chung-Ju Wu >> >> * doc/invoke.texi (NDS32 Options): Add -mcmodel= option and >> remove -mgp-direct option. >> >> Although these changes are target-specific part, I think

Re: [patch, Fortran] PR61933 Inquire on Internal Units

2015-01-19 Thread Tobias Burnus
Hi Jerry, hi all, sorry for the slow patch review. I also still want to review your other inquire patch. Jerry DeLisle wrote: The fundamental problem: if the variable containing the unit number in an INQUIRE statement is of type KIND greater than 4 and the value is outside the range of a KIN

Re: [PATCH, doc] NDS32: Remove -mforce-fp-as-gp, -mforbid-fp-as-gp, and -mex9 options from documentation.

2015-01-19 Thread Chung-Ju Wu
2015-01-18 16:38 GMT+08:00 Gerald Pfeifer : > On Wednesday 2015-01-14 17:58, Chung-Ju Wu wrote: >> 2015-01-14 Chung-Ju Wu >> >> * doc/invoke.texi (NDS32 Options): Remove -mforce-fp-as-gp, >> -mforbid-fp-as-gp, and -mex9 options. > > Can you cover these, and similar changes, in th

<    1   2