[Ping] [PATCH PR64820] Fix ASan UAR detection fails on 32-bit targets if SSP is enabled.

2015-02-15 Thread Maxim Ostapenko
Ping. Original Message Subject: [PATCH PR64820] Fix ASan UAR detection fails on 32-bit targets if SSP is enabled. Date: Mon, 09 Feb 2015 14:03:54 +0400 From: Maxim Ostapenko To: GCC Patches CC: Yury Gribov , Slava Garbuzov Hi, when testing I noticed, that i

Re: [doc, committed] small grammar error fix

2015-02-15 Thread Gerald Pfeifer
On Tuesday 2015-02-10 19:15, Sandra Loosemore wrote: I've checked it in as obvious. -Sandra 2015-02-10 David Wohlferd Sandra Loosemore gcc/ * doc/extend.texi (Loop-Specific Pragmas): Fix grammar error. That's fine, just the attachment was the ChangeLog entry

[doc, committed] use "Title Case" for section titles

2015-02-15 Thread Sandra Loosemore
I noticed a while back when searching for something in the table of contents for the GCC manual that most section names used "Title Case", but some only capitalized the first word, or not even that. This patch is the result of me going through the whole ToC and trying to fix everything to use

Re: Chromium: LTO

2015-02-15 Thread Jan Hubicka
> Hi, > > > +symtab_node::iterate_direct_aliases (unsigned i, ipa_ref *&ref) > > +{ > > + ref_list.referring.iterate (i, &ref); > > + > > + if (ref && ref->use != IPA_REF_ALIAS) > > +return NULL; > > + > > + return ref; > > +} > > it seems a little weird the out arg can return a non alias,

Re: [PATCH, fixincludes] Fix PR 48009 53348

2015-02-15 Thread Bruce Korb
Looks good to me. On Sun, Feb 15, 2015 at 12:49 PM, David Edelsohn wrote: > The stdlib.h header in AIX 4.3 does not correctly declare strtof with > a const char* argument. Users are building the latest releases of GCC > on AIX 4.3 The appended patch from Richard G Daniel uses fixincludes > to c

[4.8 branch] PATCH: PR middle-end/53623: [4.7/4.8 Regression] sign extension is effectively split into two x86-64 instructions

2015-02-15 Thread H.J. Lu
Hi, This is a backport of the patch for PR middle-end/53623 plus all bug fixes caused by it. Tested on Linux/x86-32, Linux/x86-64 and x32. OK for 4.8 branch? Thanks. H.J. --- diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 469ee31..44bf322 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1

[PATCH, fixincludes] Fix PR 48009 53348

2015-02-15 Thread David Edelsohn
The stdlib.h header in AIX 4.3 does not correctly declare strtof with a const char* argument. Users are building the latest releases of GCC on AIX 4.3 The appended patch from Richard G Daniel uses fixincludes to correct the declaration. Okay? Thanks, David PR bootstrap/48009 PR

Re: Chromium: LTO

2015-02-15 Thread Trevor Saunders
Hi, > +symtab_node::iterate_direct_aliases (unsigned i, ipa_ref *&ref) > +{ > + ref_list.referring.iterate (i, &ref); > + > + if (ref && ref->use != IPA_REF_ALIAS) > +return NULL; > + > + return ref; > +} it seems a little weird the out arg can return a non alias, and so if you only want t

Re: [patch, testsuite] Fix ubsan for testing when libstdc++ isn't installed

2015-02-15 Thread Mike Stump
On Feb 13, 2015, at 2:24 PM, Jack Howarth wrote: > Mike and FX, > Shouldn't we also apply… Ok. > Author: fxcoudert > Date: Mon Dec 22 21:57:45 2014 > New Revision: 219035 > > URL: https://gcc.gnu.org/viewcvs?rev=219035&root=gcc&view=rev > Log: > * lib/ubsan-dg.exp: Add library path for libs

Re: [Patch, fortran] PR60898 premature release of entry symbols

2015-02-15 Thread Dominique Dhumieres
of unsupported tests 85 === gfortran Summary === # of expected passes104465 # of unexpected failures2 # of expected failures 162 # of unsupported tests 326 /opt/gcc/p_build/gcc/testsuite/gfortran/../../gfortran version 5.0.0 20150215 (experim

Re: [Patch, fortran] PR60898 premature release of entry symbols

2015-02-15 Thread Jerry DeLisle
On 02/15/2015 09:48 AM, Mikael Morin wrote: [*] I have a few failing testcases (also without the patch), namely the following; does this ring a bell ? FAIL: gfortran.dg/erf_3.F90 FAIL: gfortran.dg/fmt_g0_7.f08 FAIL: gfortran.dg/fmt_en.f90 FAIL: gfortran.dg/nan_7.f90 FAIL: gfortran.dg/quad_2.f90

[Patch, fortran] PR60898 premature release of entry symbols

2015-02-15 Thread Mikael Morin
Hello, I propose a fix for PR60898, where a symbol is freed despite remaining reachable in the symbol tree. The problem comes from this code in resolve_symbol: > > /* If we find that a flavorless symbol is an interface in one of the >parent namespaces, find its symtree in this namespa

Re: [PATCH, PR tree-optimization/65002] Disable SRA for functions wrongly marked as read-only

2015-02-15 Thread Mike Stump
On Feb 13, 2015, at 11:25 AM, Jakub Jelinek wrote: >>> 2015-02-12 Ilya Enkovich >>> >>> PR tree-optimization/65002 >>> * gcc.dg/pr65002.C: New. >> >> This test should have gone into g++.dg. > > Into g++.dg/opt or g++.dg/ipa in particular. Pre-approved if someone wants to svn mv it.

Re: patch to fix rtl documentation for new floating point comparisons

2015-02-15 Thread Kenneth Zadeck
On 02/14/2015 03:26 PM, Paolo Bonzini wrote: On 10/02/2015 22:46, Joseph Myers wrote: It may make sense to define LTGT as exactly !UNEQ, and so quiet, but the choice of definition is a matter of what's convenient for the implementation (and which choice you make determines which existing code

[committed] Improve handling of reloads for floating point loads and stores on PA

2015-02-15 Thread John David Anglin
The attached patch fixes an ICE building the code-saturne package and generally improves the code generated for floating point loads and stores. With the previous implementation, it was not possible to load a LO_SUM DLT address to a floating point register (e.g., for an integer multiplication)

[PATCH] PR target/65064: Return false for COMMON symbols

2015-02-15 Thread H.J. Lu
Hi, r220674 exposed a bug in ia64_in_small_data_p. After r220674, COMMON symbols binds locally for executables. But ia64_in_small_data_p returns true for COMMON symbols which are never in small data section. This patch fixes it. OK for trunk? H.J. Since COMMON symbols are never in small

Re: Ping : [PATCH] [gcc, combine] PR46164: Don't combine the insns if a volatile register is contained.

2015-02-15 Thread Segher Boessenkool
Hi Terry, I still think this is stage1 material. > + /* Don't combine if dest contains a user specified register and i3 contains > + ASM_OPERANDS, because the user specified register (same with dest) in i3 > + would be replaced by the src of insn which might be different with > + the

Re: Ping : [PATCH] [gcc, combine] PR46164: Don't combine the insns if a volatile register is contained.

2015-02-15 Thread Terry Guo
On Fri, Feb 13, 2015 at 5:06 PM, Richard Sandiford wrote: > Segher Boessenkool writes: >> On Thu, Feb 12, 2015 at 03:54:21PM +, Richard Sandiford wrote: >>> "Hale Wang" writes: >>> > Ping? >> >> It's not a regression (or is it?), so it is not appropriate for stage4. >> >> >>> >> diff --git a