[Ada] Add assertion for context of subprograms

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

[Ada] Skip derived types treatment for subprogram types

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] Fix interfacing with C++ methods on 32-bit Windows

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] Fix alignment warning on aliased formal parameter

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

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

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

Re: Stepping down as global maintainer

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

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: [PATCH][ARM][PING] __ARM_FP & __ARM_NEON_FP defined when -march=armv7-m

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, fortran] PR64952 - Missing temporary in assignment from elemental function

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, fortran] PR63744 accept duplicate use-rename

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

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][AArch64] Remove crypto extension from default for cortex-a53, cortex-a57

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: [Patch, fortran] PR64952 - Missing temporary in assignment from elemental function

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, fortran] PR64952 - Missing temporary in assignment from elemental function

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] PR rtl-optimization/32219: optimizer causes wrong code in pic/hidden/weak symbol checking

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

Fix PR 63566 part 1

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

Fix PR 63566 part 2

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 3

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

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

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 4

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

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

[Committed/AARCH64] Remove dummy GTY variable

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: [wwwdocs] IPA/LTO/FDO updates for gcc-5/changes.html

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

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

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

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

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

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

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]

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

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

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.

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

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

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

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

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

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