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

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

Re: [RFA:] Caveat for ARM in gcc-4.7/changes.html: unaligned accesses

> From: Hans-Peter Nilsson > Date: Fri, 8 Jun 2012 06:29:04 +0200 > > From: Michael Hope > > Date: Fri, 8 Jun 2012 05:50:52 +0200 > > The combination of > > older Linux ARM kernels and GCC 4.7 gives a faulty kernel. > > We're in agreement! Oh wait sorry, my bad, I misread. Instead of "gives

Re: [RFA:] Caveat for ARM in gcc-4.7/changes.html: unaligned accesses

> From: Michael Hope > Date: Fri, 8 Jun 2012 05:50:52 +0200 > On 8 June 2012 15:20, Hans-Peter Nilsson wrote: > > So the default for ALIGNMENT_TRAP changed in >3.1? > > ALIGNMENT_TRAP is on by default but the early boot time trap is now > conditional on the architecture level: > > __enable_mmu

Re: [RFA:] Caveat for ARM in gcc-4.7/changes.html: unaligned accesses

On 8 June 2012 15:20, Hans-Peter Nilsson wrote: >> From: Michael Hope >> Date: Fri, 8 Jun 2012 04:42:30 +0200 > >> On 8 June 2012 12:15, Hans-Peter Nilsson wrote: >> > The "some source >> > codes" was in the analyzed case a strcpy of a five-byte string >> > (busybox/libbb/procps.c:365 'strcpy(fi

Re: [RFA:] Caveat for ARM in gcc-4.7/changes.html: unaligned accesses

> From: Michael Hope > Date: Fri, 8 Jun 2012 04:42:30 +0200 > On 8 June 2012 12:15, Hans-Peter Nilsson wrote: > > The "some source > > codes" was in the analyzed case a strcpy of a five-byte string > > (busybox/libbb/procps.c:365 'strcpy(filename_tail, "stat");' of > > some unknown busybox-versi

Re: [RFA:] Caveat for ARM in gcc-4.7/changes.html: unaligned accesses

On 8 June 2012 12:15, Hans-Peter Nilsson wrote: > (CC to ARM maintainer approving the original patch.) > > I'm listing this under "caveats" rather than "improvements" and > before the current top ARM-related caveat (as this one is more > important :) because I don't see performance figures in the

Ping*2: [RFA:] doc: TARGET_LEGITIMIZE_ADDRESS needs to be defined for native TLS

> From: Hans-Peter Nilsson > Date: Wed, 16 May 2012 02:24:02 +0200 > gcc: > * doc/tm.texi.in (Addressing Modes) : > Mention that this hook needs to be defined for native TLS. > * doc/tm.texi: Regenerate. brgds, H-P

Ping: [RFA:] fix bug in configure header-probing for stack protector support in target C library

> From: Hans-Peter Nilsson > Date: Fri, 1 Jun 2012 01:38:22 +0200 > gcc: > Fix configure test for "stack protector support in target C library". > * configure.ac (test_prefix, test_exec_prefix): Move setting from > inside sysroot handling to before and outside it. > * conf

[RFA:] Caveat for ARM in gcc-4.7/changes.html: unaligned accesses

(CC to ARM maintainer approving the original patch.) I'm listing this under "caveats" rather than "improvements" and before the current top ARM-related caveat (as this one is more important :) because I don't see performance figures in the context of the original patch (r178852) backing up this as

Re: [google-4_6] indirect_call promotion for streaming LIPO (issue 6306054)

ok for google branches -- LTO specific bugs should be isolated and submitted upstream. David http://codereview.appspot.com/6306054/diff/5001/value-prof.c File value-prof.c (right): http://codereview.appspot.com/6306054/diff/5001/value-prof.c#newcode1720 value-prof.c:1720: direct_call2 = 0; Ok

[google-4_6] indirect_call promotion for streaming LIPO (patchset 2) (issue6306054)

Hi, This patch is for google-4_6 branch only. Patch Set 2. Tested is undergoing. 2012-06-07 Rong Xu * lto-symtab.c (lto_cgraph_replace_node): indirect-call promotion support in streaming LIPO. (Entry_BB_profile_count): Ditto. (COMDAT_prevailing_node_p): Ditt

Re: [google-4_6] indirect_call promotion for streaming LIPO (issue 6306054)

On Thu, Jun 7, 2012 at 3:22 PM, wrote: > > will send the new patch set in a few minutes. > > > > http://codereview.appspot.com/6306054/diff/1/cgraph.h > File cgraph.h (right): > > http://codereview.appspot.com/6306054/diff/1/cgraph.h#newcode410 > cgraph.h:410: > They are referenced in lto-cgraph.c

[patch] Move split_double from final.c to rtlanal.c, do not include output.h in expr.c

Hello, Another patch to move around some code to break up a strange dependency. expr.c isn't supposed to emit assembly so it shouldn't need output.h, but it needs split_double from final.c. This split_double has nothing to do with writing out assembly, so it shouldn't be in final.c, and I moved it

Re: [google-4_6] indirect_call promotion for streaming LIPO (issue 6306054)

will send the new patch set in a few minutes. http://codereview.appspot.com/6306054/diff/1/cgraph.h File cgraph.h (right): http://codereview.appspot.com/6306054/diff/1/cgraph.h#newcode410 cgraph.h:410: They are referenced in lto-cgraph.c etc. On 2012/06/07 21:46:53, davidxl wrote: Why not put

Re: [google-4_6] indirect_call promotion for streaming LIPO (issue 6306054)

http://codereview.appspot.com/6306054/diff/1/cgraph.h File cgraph.h (right): http://codereview.appspot.com/6306054/diff/1/cgraph.h#newcode410 cgraph.h:410: Why not putting this into value-prof.h? http://codereview.appspot.com/6306054/diff/1/ipa-split.c File ipa-split.c (right): http://coderevi

Re: [Dwarf Patch] Improve pubnames and pubtypes generation. (issue 6197069)

On 2012/06/01 17:58:41, saugustine wrote: The enclosed patch updates the earlier patch to address all of the feedback I have gotten regarding generating pubnames. It fixes the offset problem in the pubtypes table; switches DW_AT_pubtypes to a flag and so on. It also adds and documents a ne

Committed: xfail gcc.dg/pr46647.c for cris-* and crisv32-*

See the PR. Note well, there is no effect on the emitted assembly code; the failure is in this pass but fixed in another. Thus it does not pass the work/benefit threshold for adding a separate test-case from the code in the PR, just for xfailing this one; and for every xfail there should be a PR.

[google-4_6] indirect_call promotion for streaming LIPO (issue6306054)

Hi, This patch is for google-4_6 branch only. It implements the multi-target indirect-call promotion as a regular IPA passes. I took a shortcut by opening and modifying the IR in IPA execute stage. The early version pushes the candidates to obstack and makes cgraph_node clones , and performs the

C++ PATCH for c++/53599 (ICE with local class in template)

The problem here was that when we instantiate G::bar, we want to instantiate F, and end up improperly pushing the declaration inside G, which leads to the ICE. Adjusting the call to pushclass is enough to fix this, which is what I propose doing for the 4.7 branch. For 4.8 I'm going to do more

Re: [PATCH][1/n][C] Do not sign-extend sizetypes

On Mon, Apr 11, 2011 at 7:25 AM, Richard Guenther wrote: > > This is another try at making sizetype behavior more consistent with > other integral types.  In particular this series will eventually > succeed in making TYPE_UNSIGNED tell the truth for sizetypes ... > > This first patch replaces a ha

[PATCH, libgcc]: Put soft-FP exception handler out-of-line for x86

Hello! Attached patch rewrites x86 soft-FP as an out-of-line function that gets conditionally called when exception occurs. This rewrite removes 17 instances of the same code from libgcc. In addition, the patch unifies a lot of code between 32bit and 64bit x86 target, and also introduces SSE instr

[lra] patch fixing spec2000 mesa code generation failure

The following patch mostly fixes spec2000 mesa code generation failure on x86 triggered by committing hard reg splitting patch. The patch was successfully bootstrapped on x86-64. Committed as rev. 188316. 2012-06-07 Vladimir Makarov * lra-constraints.c (split_reg): Fix comments and

Re: [wwwdocs] Buildstat update for 4.6

On Sun, 3 Jun 2012, Tom G. Christensen wrote: > Latest results for 4.6.x > > -tgc > > Testresults for 4.6.3: > i386-pc-solaris2.8 > i386-pc-solaris2.10 > x86_64-unknown-linux-gnu Thanks, Tom! Online now. Gerald

Re: [patch][PCH] Do not write/read asm_out_file

Steven Bosscher writes: > On Thu, Jun 7, 2012 at 7:52 PM, Richard Sandiford > wrote: >> Steven Bosscher writes: >>> On Wed, Jun 6, 2012 at 7:15 PM, Paolo Bonzini wrote: Il 06/06/2012 09:15, Steven Bosscher ha scritto: > +    add_asm_printf ("%s\n\t.ascii \"%s\"\n", > +            

Re: [patch][PCH] Do not write/read asm_out_file

On Thu, Jun 7, 2012 at 7:52 PM, Richard Sandiford wrote: > Steven Bosscher writes: >> On Wed, Jun 6, 2012 at 7:15 PM, Paolo Bonzini wrote: >>> Il 06/06/2012 09:15, Steven Bosscher ha scritto: +    add_asm_printf ("%s\n\t.ascii \"%s\"\n", +                 READONLY_DATA_SECTION_ASM_OP,

Re: [patch][PCH] Do not write/read asm_out_file

Steven Bosscher writes: > On Wed, Jun 6, 2012 at 7:15 PM, Paolo Bonzini wrote: >> Il 06/06/2012 09:15, Steven Bosscher ha scritto: >>> +    add_asm_printf ("%s\n\t.ascii \"%s\"\n", >>> +                 READONLY_DATA_SECTION_ASM_OP, string); >>> +  else >> >> Need to escape the string as in mips_

Re: [google] New fdo summary-based icache sensitive unrolling (issue 6282045)

On Thu, Jun 7, 2012 at 12:05 AM, wrote: > other than the update issue, good for google branch. > > David > > > http://codereview.appspot.com/6282045/diff/4003/libgcc/libgcov.c > File libgcc/libgcov.c (right): > > http://codereview.appspot.com/6282045/diff/4003/libgcc/libgcov.c#newcode1127 > libgc

Re: [google/gcc-4_6] Fix to use correct form for DW_AT_low_pc in compile_unit DIE. (issue6298055)

On Wed, Jun 6, 2012 at 11:23 PM, Cary Coutant wrote: > This patch is for the google/gcc-4_6 branch. > > In porting the earlier patch to use constant forms for DW_AT_high_pc, > the code in dwarf2out_finish was inadvertently changed to use an > indirect form for DW_AT_low_pc in the main compile_unit

[patch, fortran,committed] Fix regression introduced by PR 52861 patch

Hello world, my recent patch introduced a regression with deferred-length characters. I have committed the attached patch as obvious to fix this. Regards Thomas 2012-06-07 Thomas König PR fortran/52861 * frontend-passes.c (optimize_assignment): Don't set the

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

Il 07/06/2012 12:21, Dinar Temirbulatov ha scritto: > oh, I found typo in comment in the end of patch. fixed. Great improvement, thanks! Unfortunately we're not there yet, but much closer! I could understand the new code much better so I suggest some more improvements below, both to the comments

Re: [patch][gcc47] Deprecate -fconserve-space

On Mon, 4 Jun 2012, Steven Bosscher wrote: > How about this: Looks good to me. Do I understand correctly that this is a change in GCC 4.7.1 and later (but not GCC 4.7.0)? If that's the case, can you note this, for example by prepending "[GCC 4.7.1 and later]" to your snippet? Gerald

[PATCH] genhooks: always bail out if tm.texi lacks place for hook

So that it doesn't happen again. Committed as obvious. Andreas. * genhooks.c (main): Set progname. (emit_documentation): Remove variable found_start, always bail out when a place is missing. * doc/tm.texi.in (C++ ABI): Add @hook TARGET_CXX_DECL_MANGLING_CO

Re: [PATCH, i386]: Back port Fix PR 52908 - xop-mul-1:f9 miscompiled on bulldozer (-mxop) to 4.7

On Thu, Jun 07, 2012 at 06:07:18AM -0500, venkataramanan.ku...@amd.com wrote: > Please find the patch below that backports PR target/52908 to GCC 4.7. > > The patch passed bootstrap and regression test. > > Ok to commit? Please wait with it until 4.7.1 is released. > +2012-06-07 Venkataramanan

Re: [patch] Deal with #ident without

On Thu, Jun 7, 2012 at 12:42 PM, Andreas Schwab wrote: > Steven Bosscher writes: > >> I am not sure what you mean with misplaced. > > A nested @deftypefn doesn't look right.  Try make info.  You need to add > a @hook in tm.texi.in at the right place (output_ident has nothing to do > with anchors)

[PATCH, i386]: Back port Fix PR 52908 - xop-mul-1:f9 miscompiled on bulldozer (-mxop) to 4.7

Hi Maintainers, Please find the patch below that backports PR target/52908 to GCC 4.7. The patch passed bootstrap and regression test. Ok to commit? regards, Venkat. Index: ChangeLog === --- ChangeLog (revision 187449) +++ Chan

Re: [patch] Deal with #ident without

Steven Bosscher writes: > I am not sure what you mean with misplaced. A nested @deftypefn doesn't look right. Try make info. You need to add a @hook in tm.texi.in at the right place (output_ident has nothing to do with anchors). Andreas. -- Andreas Schwab, sch...@linux-m68k.org GPG Key fing

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

oh, I found typo in comment in the end of patch. fixed. thanks, Dinar. On Thu, Jun 7, 2012 at 2:14 PM, Dinar Temirbulatov wrote: > Hi, > Here is new version of patch based up on Paolo review, again tested on > arm-7l, mips-32r2 (74k), i686 without new regressions. >      

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

Hi, Here is new version of patch based up on Paolo review, again tested on arm-7l, mips-32r2 (74k), i686 without new regressions. thanks, Dinar. On Sat, May 26, 2012 at 4:45 PM, Paolo Bonzini wrote: > Il 26/05/2012 14:35, Paolo Bonzini ha scritto: >>        /* We have to

Re: [patch] Deal with #ident without

On Thu, Jun 7, 2012 at 8:16 AM, Andreas Schwab wrote: > Steven Bosscher writes: > >> Index: doc/tm.texi >> === >> --- doc/tm.texi       (revision 188182) >> +++ doc/tm.texi       (working copy) >> @@ -5847,6 +5847,10 @@ value is 0. >

Re: [patch] Deal with #ident without

Hi Steven, This is the patch to deal with #ident. It removes the ASM_OUTPUT_IDENT and IDENT_ASM_OP target macros, and replaces them with a single target hook. The RX parts of this patch are approved. Cheers Nick

Re: libgo patch RFA: Fix printing of names in stack dumps

On Thu, Jun 7, 2012 at 8:35 AM, Ian Lance Taylor wrote: > Go strings are not necessarily NUL terminated.  The stack dump code was > erroneously using %s to print the contents of a Go string.  The > runtime_printf function that it uses supports %S to print a Go string. > This patch fixes the code t

Re: [patch] Deal with #ident without

On Thu, Jun 7, 2012 at 8:16 AM, Andreas Schwab wrote: > Steven Bosscher writes: > >> Index: doc/tm.texi >> === >> --- doc/tm.texi       (revision 188182) >> +++ doc/tm.texi       (working copy) >> @@ -5847,6 +5847,10 @@ value is 0. >

Re: [patch, fortran] Optimize assignment of empty strings

Thomas Koenig wrote: his rather simple patch makes sure that only memset is used for assigning empty strings when front-end optimization is used. Regression-tested. OK for trunk? OK, thanks for the quick patch. Tobias 2012-06-06 Thomas König PR fortran/52861 * frontend-

[Patch][ARM] Add arm-linux-gnueabihf triplet support.

Hi, The patch adds arm-linux-gnueabihf triplet support. No regression for arm-linux-gnueabi tests. There are some differences between testsuite results on softfp Natty builders and the new hard float Precise builders. But none are due to the change in triplet. Thanks! -Zhenqiang gcc/ada/Change

Go patch RFA: Fix taking the address of a field of a local variable

This patch to the Go compiler fixes the case of taking the address of a field of a local variable. Previously taking the address of the field was not treated as taking the address of the variable. But, of course, it is. This patch fixes the problem. Bootstrapped and ran Go testsuite on x86_64-u

Re: [PATCH] Add option for dumping to stderr (issue6190057)

On Wed, Jun 6, 2012 at 10:58 PM, Sharad Singhai wrote: > Sorry about the delay. I have finally incorporated all the suggestions > and reorganized the dump infrastructure a bit. The attached patch > updates vectorizer passes so that instead of accessing global > dump_file directly, these passes cal

Re: [google] New fdo summary-based icache sensitive unrolling (issue 6282045)

other than the update issue, good for google branch. David http://codereview.appspot.com/6282045/diff/4003/libgcc/libgcov.c File libgcc/libgcov.c (right): http://codereview.appspot.com/6282045/diff/4003/libgcc/libgcov.c#newcode1127 libgcc/libgcov.c:1127: cs_prg->num_hot_counters = cs_tprg->num