[PATCH] gcc/ira-color.c: save a init statement

2015-02-08 Thread Zhouyi Zhou
In function setup_left_conflict_sizes_p, init assignment conflict_size = 0 could be eliminated for the tidiness. I have no write access to gcc repository and I can't provide a testcase because the improvement has effective compiler no output. Bootstraped and regtested in x86_64 Linux

Re: [RFC][PR target/39726 P4 regression] match.pd pattern to do type narrowing

2015-02-08 Thread Jeff Law
On 02/03/15 04:39, Richard Biener wrote: I found that explicit types were ignored in some cases. It was frustrating to say the least. Huh, that would be a bug. Do you have a pattern where that happens? I'll have to recreate them. In the mean time consider something else I'm playing with tha

RE: [PATCH PR43378]Add test case for the issue

2015-02-08 Thread Bin Cheng
And the missed patch. Thanks, bin > -Original Message- > From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- > ow...@gcc.gnu.org] On Behalf Of Bin Cheng > Sent: Monday, February 09, 2015 2:07 PM > To: gcc-patches@gcc.gnu.org > Subject: [PATCH PR43378]Add test case for the issue > >

[PATCH PR43378]Add test case for the issue

2015-02-08 Thread Bin Cheng
Hi, I crossed to PR43378 and found it had already been fixed on trunk long before. I am adding a test case and going to close it after this patch. The case is tested, Is it OK? 2015-02-09 Bin Cheng PR tree-optimization/43378 * gcc.dg/tree-ssa/pr43378.c: New test.

[PATCH/AARCH64] Fix gcc.c-torture/compile/pr37433.c for AARCH64:ILP32.

2015-02-08 Thread Andrew Pinski
The problem here is that we get a symbol_ref which is SImode but for the sibcall patterns we only match symbol_refs which use DImode. I added a new testcase that tests the non-value sibcall pattern too. OK? Bootstrapped and tested on aarch64-linux-gnu with no regressions. Thanks, Andrew Pinski

RE: [Ping^2] [PATCH, ARM, libgcc] New aeabi_idiv function for armv6-m

2015-02-08 Thread Hale Wang
Ping https://gcc.gnu.org/ml/gcc-patches/2014-12/msg01059.html. > -Original Message- > From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- > ow...@gcc.gnu.org] On Behalf Of Hale Wang > Sent: Friday, December 12, 2014 9:36 AM > To: gcc-patches > Subject: RE: [Ping] [PATCH, ARM, libgcc]

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

2015-02-08 Thread Hale Wang
Ping? > -Original Message- > From: Hale Wang [mailto:hale.w...@arm.com] > Sent: Thursday, January 29, 2015 9:58 AM > To: Hale Wang; 'Segher Boessenkool' > Cc: GCC Patches > Subject: RE: [PATCH] [gcc, combine] PR46164: Don't combine the insns if a > volatile register is contained. > > Hi S

[patch, libgfortran] Bug 57822 - I/O: "(g0)" wrongly prints "E+0000"

2015-02-08 Thread Jerry DeLisle
The attached patch fixes this by checking for the case when we are doing g0 editing and the exponent is 0. Regression tested on X86-64. For the larger kinds, we are on a different code path out of necessity, so we need to address this corner case. I will commit in a day or two as simple/obvi

Re: [wwwdocs] IPA/LTO/FDO updates for gcc-5/changes.html

2015-02-08 Thread Gerald Pfeifer
On Wednesday 2014-09-24 17:25, Jan Hubicka wrote: > this patch adds list of changes to IPA/LTO/FDO before I forget about > them ;) Good work, lots of! :-) In preparation of the GCC 5.0 release I did go through this (and other changes) and made a number of editorial changes which you can find bel

[Committed/AARCH64] Remove dummy GTY variable

2015-02-08 Thread Andrew Pinski
The aarch64 back-end has a dummy gty variable which is no longer needed as there are other GTY marked variables now in aarch64.c This should speed up (very slightly) PCH generate and reading in. Committed as obvious after a build and test for aarch64-elf. Thanks, Andrew Pinski ChangeLog: * conf

Re: [PATCH/AARCH64] Xfail ssa-dom-cse-2.c

2015-02-08 Thread Richard Biener
On February 8, 2015 9:39:20 PM CET, Andrew Pinski wrote: >Like https://gcc.gnu.org/ml/gcc-patches/2015-01/msg02646.html, we >should xfail this testcase for aarch64 too. > >OK? Bootstrapped and tested on aarch64-linux-gnu. OK. Thanks, Richard. >Thanks, >Andrew Pinski > >ChangeLog: >* gcc.dg/tre

Fix PR 63566 part 4

2015-02-08 Thread Jan Hubicka
Hi, this patch finally enables local functions for functions with aliases and thunks. Honza PR ipa/63566 * ipa-visibility.c (cgraph_node::non_local_p): Accept aliases. (cgraph_node::local_p): Remove thunk related FIXME. Index: ipa-visibility.c ===

[PATCH/AARCH64] Xfail ssa-dom-cse-2.c

2015-02-08 Thread Andrew Pinski
Like https://gcc.gnu.org/ml/gcc-patches/2015-01/msg02646.html, we should xfail this testcase for aarch64 too. OK? Bootstrapped and tested on aarch64-linux-gnu. Thanks, Andrew Pinski ChangeLog: * gcc.dg/tree-ssa/ssa-dom-cse-2.c: xfail for AARCH64 also. commit 5ddfd811b7b7a0f6bcbaf78b597ae473bb36

Fix PR 63566 part 3

2015-02-08 Thread Jan Hubicka
Hi, this patch makes i386.c ready for presence of aliases of local functions, but also fixes a wrong code issue where ix86_function_sseregparm does use caller TARET_SSE_MATH flags instead of callees. If callee disagree on this flag, it may get called with wrong calling convention. I added error m

Fix PR 63566 part 2

2015-02-08 Thread Jan Hubicka
Hi, this patch fixes heurstics in ipa-split that disables splitting for functions called once (which may need revisiting anyway). When function has alias, the other uses may come through that alias. Bootstrapped/regtested x86_64. PR ipa/63566 * ipa-split.c (execute_split_functio

Fix PR 63566 part 1

2015-02-08 Thread Jan Hubicka
Hi, PR63566 after fixing the underlying wrong code issue is about functions with aliases not market local. This patch prepares cgraphunit for some changes needed to make i386.c aliases safe. Bootstrapped/regtested x86_64-linux, comitted. Honza PR ipa/63566 * cgraphunit.c (cgra

Re: [PATCH] PR rtl-optimization/32219: optimizer causes wrong code in pic/hidden/weak symbol checking

2015-02-08 Thread Jack Howarth
H.J., This last version of the patch bootstraps and passes the test suite without regressions on x86_64-apple-darwin14. https://gcc.gnu.org/ml/gcc-testresults/2015-02/msg00912.html Thanks for fixing this. Jack On Sat, Feb 7, 2015 at 11:45 AM, H.J. Lu wrote: > On Sat, Feb 07, 2015

Re: [Patch, fortran] PR64952 - Missing temporary in assignment from elemental function

2015-02-08 Thread Paul Richard Thomas
Dear Mikael, Thank you very much for the review. You raise some points that I had thought about and others that I hadn't. I also realised that such things as blocks, within the elemental function would through the fix as well. I'll defer doing anything with it until tomorrow night. I reason that

Re: [Patch, fortran] PR64952 - Missing temporary in assignment from elemental function

2015-02-08 Thread Mikael Morin
Hello Paul, comments below Le 08/02/2015 16:24, Paul Richard Thomas a écrit : > > Index: gcc/fortran/gfortran.h > === > *** gcc/fortran/gfortran.h(revision 220482) > --- gcc/fortran/gfortran.h(working copy) > ***

Re: [PATCH][AArch64] Remove crypto extension from default for cortex-a53, cortex-a57

2015-02-08 Thread Richard Earnshaw
On 17/11/14 17:03, pins...@gmail.com wrote: > > > > >> On Nov 17, 2014, at 8:59 AM, Ramana Radhakrishnan >> wrote: >> >>> On Mon, Nov 17, 2014 at 2:48 PM, Kyrill Tkachov >>> wrote: >>> Hi all, >>> >>> Some configurations of Cortex-A53 and Cortex-A57 don't ship with crypto, >>> so enabling i

Re: Stepping down as global maintainer

2015-02-08 Thread Diego Novillo
On Sun, Feb 8, 2015 at 6:58 AM, Gerald Pfeifer wrote: > On Friday 2015-02-06 16:42, Diego Novillo wrote: >> As such, I propose to become a write-after-approval maintainer >> and relinquish all the other maintainer roles I had. > > Thanks for your contributions over the years, Diego! Thanks! > I

Re: [Patch, fortran] PR63744 accept duplicate use-rename

2015-02-08 Thread Mikael Morin
Le 07/02/2015 14:40, Dominique Dhumieres a écrit : > Dear Mikael, > > Even if > >>> use m, only: A => X >>> use m, only: A => X > > is valid, it does not make sense to use it and it is probably a typo. > Should not gfortran emit a warning, at least with -Wall? > Yeah, why not? I think we ha

Re: [Patch, fortran] PR64952 - Missing temporary in assignment from elemental function

2015-02-08 Thread Paul Richard Thomas
Dear All, Dominique has just flagged up a slight technical problem with the patch... it's not for this PR :-( Please find the correct patch attached. Paul On 8 February 2015 at 12:42, Paul Richard Thomas wrote: > Dear All, > > This came up at > https://groups.google.com/forum/#!topic/comp.lang.

Re: [PATCH][ARM][PING] __ARM_FP & __ARM_NEON_FP defined when -march=armv7-m

2015-02-08 Thread Christophe Lyon
On 3 February 2015 at 17:29, Richard Earnshaw wrote: > On 06/01/15 09:40, Mantas Mikaitis wrote: >> >> Ping and changelog spaces removed. >> >> Thank you, >> Mantas M. >> >> On 18/11/14 11:58, Richard Earnshaw wrote: >>> On 18/11/14 11:30, Mantas Mikaitis wrote: Incorrect predefinitions for c

Re: [PATCH] check_GNU_style.sh "80 characters exceeded" error fix

2015-02-08 Thread Gerald Pfeifer
On Monday 2014-12-08 15:15, Jeff Law wrote: >> * contrib/check_GNU_style.sh (col): Got rid of cut operation >> from the pipe chain and instead added cut inside awk command. > Yes. Please install on the trunk. I was going to apply this for Mantas (who does not have write acce

Re: Stepping down as global maintainer

2015-02-08 Thread Gerald Pfeifer
On Friday 2015-02-06 16:42, Diego Novillo wrote: > As such, I propose to become a write-after-approval maintainer > and relinquish all the other maintainer roles I had. Thanks for your contributions over the years, Diego! I had a look at gcc/doc/contrib.texi and am not sure this properly reflects

[Patch, fortran] PR64952 - Missing temporary in assignment from elemental function

2015-02-08 Thread Paul Richard Thomas
Dear All, This came up at https://groups.google.com/forum/#!topic/comp.lang.fortran/TvVY5j3GPmg gfortran produces wrong result from: PROGRAM Main INTEGER :: i, index(5) = (/ (i, i = 1,5) /) REAL :: array(5) = (/ (i+0.0, i = 1,5) /) array = Fred(index,array) PRINT *, array CONTAIN

[Ada] Fix alignment warning on aliased formal parameter

2015-02-08 Thread Eric Botcazou
This eliminates the alignment warning wrongly issued on an address clause applied to a formal parameter, even though the type of the parameter has the appropriate alignment clause. Tested on x86_64-suse-linux, applied on the mainline. 2015-02-08 Eric Botcazou * gcc-interface/decl.c

[Ada] Fix interfacing with C++ methods on 32-bit Windows

2015-02-08 Thread Eric Botcazou
Gigi fails to set the "thiscall" calling convention on a method imported from C++ if the class has no virtual member functions on the C++ side. Tested on x86-windows, applied on the mainline. 2015-02-08 Eric Botcazou * gcc-interface/decl.c (is_cplusplus_method): Use Is_Primitive fla

[Ada] Skip derived types treatment for subprogram types

2015-02-08 Thread Eric Botcazou
This makes it so that the special code dealing with alias sets for derived types is skipped for subprogram types Tested on x86_64-suse-linux, applied on the mainline. 2015-02-08 Eric Botcazou * gcc-interface/decl.c (gnat_to_gnu_entity): Do not bother about alias sets in pres

[Ada] Add assertion for context of subprograms

2015-02-08 Thread Eric Botcazou
This adds an early assertion that the context of subprograms is in keeping with their scope, to avoid obscure segfaults later if this isn't the case. Tested on x86_64-suse-linux, applied on the mainline. 2015-02-08 Eric Botcazou * gcc-interface/utils.c (begin_subprog_body): Assert t