[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 ebotca...@adacore.com * gcc-interface/utils.c

[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 ebotca...@adacore.com * gcc-interface/decl.c (gnat_to_gnu_entity): Do not bother about

[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 ebotca...@adacore.com

[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 ebotca...@adacore.com * gcc-interface/decl.c (is_cplusplus_method):

[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

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 hjl.to...@gmail.com wrote: On

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 ramana@googlemail.com wrote: On Mon, Nov 17, 2014 at 2:48 PM, Kyrill Tkachov kyrylo.tkac...@arm.com wrote: Hi all, Some configurations of Cortex-A53 and Cortex-A57 don't ship with

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: Stepping down as global maintainer

2015-02-08 Thread Diego Novillo
On Sun, Feb 8, 2015 at 6:58 AM, Gerald Pfeifer ger...@pfeifer.com 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!

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

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 access),

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 rearn...@arm.com 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

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 paul.richard.tho...@gmail.com wrote: Dear All, This came up at

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 have to defer it

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

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 pins...@gmail.com 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: *

[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: *

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

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

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

[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

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

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

[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

[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 bin.ch...@arm.com PR tree-optimization/43378 * gcc.dg/tree-ssa/pr43378.c: New

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 Hi, I

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 Segher,

[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

[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

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] New