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

On Fri, Feb 13, 2015 at 5:06 PM, Richard Sandiford rdsandif...@googlemail.com wrote: Segher Boessenkool seg...@kernel.crashing.org writes: On Thu, Feb 12, 2015 at 03:54:21PM +, Richard Sandiford wrote: Hale Wang hale.w...@arm.com writes: Ping? It's not a regression (or is it?), so it is

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

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

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

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

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

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)

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

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

[Patch, fortran] PR60898 premature release of entry symbols

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

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

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

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

=== 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 (experimental) [trunk revision 220715p2a] (GCC

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

On Feb 13, 2015, at 11:25 AM, Jakub Jelinek ja...@redhat.com wrote: 2015-02-12 Ilya Enkovich ilya.enkov...@intel.com 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

Re: Chromium: LTO

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 to look at

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

On Feb 13, 2015, at 2:24 PM, Jack Howarth howarth.at@gmail.com 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=219035root=gccview=rev Log: * lib/ubsan-dg.exp: Add library

Re: Chromium: LTO

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

[PATCH, fixincludes] Fix PR 48009 53348

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

Re: [PATCH, fixincludes] Fix PR 48009 53348

Looks good to me. On Sun, Feb 15, 2015 at 12:49 PM, David Edelsohn dje@gmail.com 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

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

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 @@

[doc, committed] use Title Case for section titles

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

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

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 m.ostape...@partner.samsung.com To: GCC Patches gcc-patches@gcc.gnu.org CC: Yury Gribov

Re: [doc, committed] small grammar error fix

On Tuesday 2015-02-10 19:15, Sandra Loosemore wrote: I've checked it in as obvious. -Sandra 2015-02-10 David Wohlferd d...@limegreensocks.com Sandra Loosemore san...@codesourcery.com gcc/ * doc/extend.texi (Loop-Specific Pragmas): Fix grammar error. That's