[arm-embedded] Backport mainline r179607, r179979, r179980, r181416, r182014

2012-06-08 Thread Joey Ye
Committed to ARM/embedded-4_6-branch 2012-06-08 Joey Ye joey...@arm.com Backport r182014 from mainline. 2011-12-05 Kazu Hirata k...@codesourcery.com PR target/51408 * config/arm/arm.md (*minmax_arithsi): Always require the else clause in the MINUS

Re: [PATCH][AARCH64] Remove t-aarch64 from libgcc

2012-06-08 Thread Marcus Shawcroft
On 29/05/12 09:31, Jim MacArthur wrote: 2012-05-28 Jim MacArthurjim.macart...@arm.com * config/aarch64/t-aarch64: Delete. * config.host (aarch64*-*-elf): Remove reference to t-aarch64. I've committed this patch to the aarch64 branch. /Marcus

Re: [AARCH64][libgcc] Add __aarch64_sync_cache_range as lib2func

2012-06-08 Thread Marcus Shawcroft
On 31/05/12 16:33, Jim MacArthur wrote: * config/aarch64/sync-cache.S: New file. Corrected to sync-cache.c and committed to aarch64 branch. /Marcus

Merge from gcc 4.7 branch to gccgo branch

2012-06-08 Thread Ian Lance Taylor
I've merged revision 188326 from the gcc 4.7 branch to the gccgo branch. I also included the two patches I sent out and have committed to mainline but have not yet committed to 4.7 branch. Ian

Re: [PATCH][AARCH64][libgcc] Remove t-softfp-sfdf and t-softfp-excl from aarch64 configuration

2012-06-08 Thread Marcus Shawcroft
On 01/06/12 14:29, Jim MacArthur wrote: In response to a comment from http://gcc.gnu.org/ml/gcc-patches/2012-05/msg01721.html, this patch removes t-softfp-sfdf and t-softfp-excl from the aarch64 entries in libgcc/config.host. Every setting in these files is overridden by t-softfp. Addition

Re: [PATCH, GCC][AArch64] Use Enums for TLS dialect option selection

2012-06-08 Thread Marcus Shawcroft
On 01/06/12 09:56, Sofiane Naci wrote: Hi, This patch re-factors TLS dialect option selection in the AArch64 port to use the generic support for enumerated option arguments. Thanks Sofiane - 2012-06-01 Sofiane Nacisofiane.n...@arm.com [AArch64] Use Enums for TLS option

Re: [AArch64] Use snprintf instead sprintf

2012-06-08 Thread Marcus Shawcroft
On 01/06/12 09:53, Sofiane Naci wrote: Hi, This patch replaces instances of sprintf with snprintf with sizeof(..) in the AArch64 port. It also fixes layout issues in the code it touches. Thanks Sofiane - ChangeLog 2012-06-01 Sofiane Nacisofiane.n...@arm.com [AArch64] Replace

[arm-embedded] Backport mainline r182621, r182628, r183095, r183349, r183756, r184442

2012-06-08 Thread Joey Ye
Committed to ARM/embedded-4_6-branch 2012-06-08 Joey Ye joey...@arm.com Backport r184442 from mainline 2012-02-21 Richard Earnshaw rearn...@arm.com PR target/52294 * thumb2.md (thumb2_shiftsi3_short): Split register and immediate shifts. For

Re: [ARM Patch 1/n] PR53447: optimizations of 64bit ALU operation with constant

2012-06-08 Thread Carrot Wei
Hi In rtl expression, substract a constant c is expressed as add a value -c, so it is alse processed by adddi3, and I extend it more to handle a subtraction of 64bit constant. I created an insn pattern arm_subdi3_immediate to specifically represent substraction with 64bit constant while continue

Re: [PATCH] libatomic, v2

2012-06-08 Thread Andreas Schwab
Richard Henderson r...@redhat.com writes: diff --git a/gcc/testsuite/lib/target-supports.exp b/gcc/testsuite/lib/target-supports.exp index 1a25324..b93dc5c 100644 --- a/gcc/testsuite/lib/target-supports.exp +++ b/gcc/testsuite/lib/target-supports.exp @@ -3795,6 +3795,8 @@ proc

[Patch, Fortran, OOP] PR 52552: ICE when trying to allocate non-allocatable object giving a dynamic type

2012-06-08 Thread Janus Weil
Hi all, here is a patch for an ICE-on-invalid bug, which concerns the allocation of CLASS variables. The ICE is fixed by changing the order of the checks which are done in gfc_match_allocate, so that an error is triggered before the ICE can occur. Regtested on x86_64-unknown-linux-gnu. Ok for

Re: [PATCH, WWWDOCS] Document AArch64 branch.

2012-06-08 Thread Marcus Shawcroft
On 06/06/12 21:45, Gerald Pfeifer wrote: Looks good to me. (Note, that as someone with write access to the GCC repository you don't need approval for changes like this, though I am always happy to look at changes.) Ok, thank you, I hadn't realized approval was not required. Change

Re: [Patch, Fortran, OOP] PR 52552: ICE when trying to allocate non-allocatable object giving a dynamic type

2012-06-08 Thread Janus Weil
[Side note: The piece of code which I'm moving contains a FIXME comment, which I don't quite understand, so I'm not sure whether it is still valid. It was added by Steve in http://gcc.gnu.org/viewcvs?view=revisionrevision=145331. Does anyone have an opinion on this?] Possibly the comment

Re: [Patch, Fortran, OOP] PR 52552: ICE when trying to allocate non-allocatable object giving a dynamic type

2012-06-08 Thread Tobias Burnus
Hi, Janus Weil wrote: here is a patch for an ICE-on-invalid bug, which concerns the allocation of CLASS variables. The ICE is fixed by changing the order of the checks which are done in gfc_match_allocate, so that an error is triggered before the ICE can occur. Regtested on

Re: [Patch, Fortran, OOP] PR 52552: ICE when trying to allocate non-allocatable object giving a dynamic type

2012-06-08 Thread Janus Weil
Hi, here is a patch for an ICE-on-invalid bug, which concerns the allocation of CLASS variables. The ICE is fixed by changing the order of the checks which are done in gfc_match_allocate, so that an error is triggered before the ICE can occur. Regtested on x86_64-unknown-linux-gnu. Ok for

Re: [Patch, Fortran, OOP] PR 52552: ICE when trying to allocate non-allocatable object giving a dynamic type

2012-06-08 Thread Tobias Burnus
Janus Weil wrote: In any case, for those two, it does not trigger but one gets later (resolve?) the error: Error: Allocate-object at (1) must be ALLOCATABLE or a POINTER Then I guess one can just remove the FIXME. That's fine with me. Regarding the error: Error: Allocate-object at (1) is

Re: libgo patch committed: Fix testsuite on PPC

2012-06-08 Thread Andreas Schwab
Ian Lance Taylor i...@google.com writes: @@ -326,13 +336,18 @@ } { + text=T + case $GOARCH in + ppc*) text=D ;; This is wrong for ppc. Andreas. diff --git a/libgo/testsuite/gotest b/libgo/testsuite/gotest index da1162e..208cbaf 100755 --- a/libgo/testsuite/gotest +++

[PATCH] Fix libgo testsuite

2012-06-08 Thread Andreas Schwab
This fixes most of the libgo testsuite failures. Andreas. diff --git a/libgo/Makefile.am b/libgo/Makefile.am index 1d761bc..7923567 100644 --- a/libgo/Makefile.am +++ b/libgo/Makefile.am @@ -1806,6 +1806,7 @@ CHECK = \

Re: [Patch, Fortran, OOP] PR 52552: ICE when trying to allocate non-allocatable object giving a dynamic type

2012-06-08 Thread Steve Kargl
On Fri, Jun 08, 2012 at 12:40:10PM +0200, Janus Weil wrote: [Side note: The piece of code which I'm moving contains a FIXME comment, which I don't quite understand, so I'm not sure whether it is still valid. It was added by Steve in http://gcc.gnu.org/viewcvs?view=revisionrevision=145331.

Re: [Patch, Fortran, OOP] PR 52552: ICE when trying to allocate non-allocatable object giving a dynamic type

2012-06-08 Thread Janus Weil
[Side note: The piece of code which I'm moving contains a FIXME comment, which I don't quite understand, so I'm not sure whether it is still valid. It was added by Steve in http://gcc.gnu.org/viewcvs?view=revisionrevision=145331. Does anyone have an opinion on this?] It's been too long!  I

Re: [Patch, Fortran, OOP] PR 52552: ICE when trying to allocate non-allocatable object giving a dynamic type

2012-06-08 Thread Janus Weil
Attached is a third version of the patch, which I will commit soon. It adds at least the relevant standard reference to the resolution-stage check, too, and fixes the regressions resulting from the changed error message. Committed as r188335. Cheers, Janus

Re: [Patch, Fortran, OOP] PR 52552: ICE when trying to allocate non-allocatable object giving a dynamic type

2012-06-08 Thread Steve Kargl
On Fri, Jun 08, 2012 at 07:10:02PM +0200, Janus Weil wrote: [Side note: The piece of code which I'm moving contains a FIXME comment, which I don't quite understand, so I'm not sure whether it is still valid. It was added by Steve in http://gcc.gnu.org/viewcvs?view=revisionrevision=145331.

Re: divide 64-bit by constant for 32-bit target machines

2012-06-08 Thread Dinar Temirbulatov
Hi, Paolo. Here is the new version of patch. I have tested this version with gcc testsuite only on i686 without new regressions, for now. Mips and arm tests are in progress. One strange thing I noticed: No need for this gen_reg_rtx, either, by passing a NULL_RTX target below. +      

[PATCH] Add DWZ DWARF multifile extensions to dwarf2.{def,h}

2012-06-08 Thread Jakub Jelinek
Hi! This patch adds dwz -m DWARF extensions. See http://www.dwarfstd.org/ShowIssue.php?issue=120604.1type=open for more details and http://sourceware.org/git/?p=dwz.git;a=shortlog;h=refs/heads/multifile for the tool that creates it. Compared to the DWARF5 proposal, this is so far implemented as

[AVR,committed]: Set case-values-threshold to 7

2012-06-08 Thread Georg-Johann Lay
http://gcc.gnu.org/viewcvs?view=revisionrevision=188336 Revision 179760 changed casesi so that the libgcc code to read from the lookup table by means of __tablejump2__ is used unconditionally, i.e. does no more depend on -mcall-prologues. Consequently, casesi costs no more depend on

Re: Updated to respond to various email comments from Jason, Diego and Cary (issue6197069)

2012-06-08 Thread Jason Merrill
On 06/01/2012 01:58 PM, Sterling Augustine wrote: It also adds and documents a new option -g[no-]pubtypes which allows users to generate pubtypes even if the target disables them by default. Hmm, I thought the convention for this sort of flag was to start with -f, that -g flags were only for

Re: [PATCH] Add DWZ DWARF multifile extensions to dwarf2.{def,h}

2012-06-08 Thread Jason Merrill
OK. Jason

Re: Updated to respond to various email comments from Jason, Diego and Cary (issue6197069)

2012-06-08 Thread Jakub Jelinek
On Fri, Jun 08, 2012 at 02:45:09PM -0400, Jason Merrill wrote: On 06/01/2012 01:58 PM, Sterling Augustine wrote: It also adds and documents a new option -g[no-]pubtypes which allows users to generate pubtypes even if the target disables them by default. Hmm, I thought the convention for this

Re: [PATCH] Add DWZ DWARF multifile extensions to dwarf2.{def,h}

2012-06-08 Thread Cary Coutant
+/* Extensions for DWZ multifile. +   See http://www.dwarfstd.org/ShowIssue.php?issue=120604.1type=open .  */ I'd suggest repeating this comment below: --- include/dwarf2.h.jj 2012-05-09 20:15:32.0 +0200 +++ include/dwarf2.h    2012-05-14 12:16:37.414204203 +0200 @@ -333,6 +333,9 @@

[lra] patch to speed LRA up

2012-06-08 Thread Vladimir Makarov
The following patch is to speed LRA up a bit. It also adds 2012 to copyright. The patch was sucessfully bootstrapped on x86-64. Committed as rev. 188341. 2012-06-08 Vladimir Makarov vmaka...@redhat.com * lra-assigns.c: Add 2012 to copyright header. Use lra_assert instead

Re: [PATCH][RFC] Recognize memcpy/memmove - fix PR53081

2012-06-08 Thread H.J. Lu
On Tue, Jun 5, 2012 at 7:36 AM, Richard Guenther rguent...@suse.de wrote: This adds memcpy/memmove recognition to loop distribution (and cleans it up some more).  Issues are similar to memset and not handled (and I just noticed we generate memset/memcpy even with -fno-builtin ...).

Re: [patch] PR29442 - split insn-attrtab.c in three parts

2012-06-08 Thread Mike Stump
On May 3, 2012, at 9:31 AM, Steven Bosscher wrote: How does one turn random volunteer GCC hackers into spoiled brats? So, brats aside, your new patch breaks my port... diff --git a/gcc/Makefile.in b/gcc/Makefile.in index 145e8b8..1eeeab9 100644 --- a/gcc/Makefile.in +++ b/gcc/Makefile.in @@

[PATCH] libgcc: If __GLIBC__ is defined, don't refer to pthread_cancel.

2012-06-08 Thread Roland McGrath
See http://gcc.gnu.org/ml/gcc-patches/2012-01/msg01427.html for the previous discussion about the motivation for this patch and the choice of __pthread_key_create (also the comment added in the code by this patch). This had no effect on testsuite results on x86_64-linux-gnu (Ubuntu EGLIBC

Re: Wider modes for partial int modes

2012-06-08 Thread Mike Stump
On Jun 8, 2012, at 4:11 PM, Mike Stump wrote: On Apr 17, 2012, at 2:08 PM, Bernd Schmidt wrote: This patch enables GET_MODE_WIDER_MODE for MODE_PARTIAL_INT (by setting the wider mode to the one the partial mode is based on), which is useful for the port I'm working on: I can avoid defining

[gimplefe] patch that fixes the bug for the failure test case

2012-06-08 Thread Sandeep Soni
Hi, This patch fixes the failure test case that I had submitted the last time. The changeLog is testsuite/gChangeLog.gimplefe is as follows. 2012-06-09 Sandeep Soni soni.sande...@gmail.com * gimple.dg/20120605-2.gimple : New. While the changelog in gimple/ChangeLog is as

Re: [RFC, ivopts] fix bugs in ivopts address cost computation

2012-06-08 Thread Hans-Peter Nilsson
On Tue, 5 Jun 2012, Sandra Loosemore wrote: (1) While the address cost computation is assuming in some situations that pre/post increment/decrement addressing will be used if supported by the target, it isn't actually using the target's address cost for such forms -- instead, just the cost of