Re: [PATCH] Fix PR54458

2012-09-08 Thread Markus Trippelsdorf
On 2012.09.04 at 11:22 +0200, Richard Guenther wrote: This fixes PR54458 where DOM jump threading turns a loop into one with multiple latches but does not mark it so. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. Richard. 2012-09-04 Richard Guenther

Re: combine vec_perm_expr with constructor

2012-09-08 Thread Marc Glisse
On Mon, 3 Sep 2012, Richard Guenther wrote: You do work above and then bail late here. Always do early exists early to reduce useless compile-time. [...] You need to verify that fold_ternary returns something that is valid GIMPLE. fold () in general happily returns trees that are in the need

Re: combine BIT_FIELD_REF and VEC_PERM_EXPR

2012-09-08 Thread Marc Glisse
On Mon, 3 Sep 2012, Richard Guenther wrote: Please do the early outs where you compute the arguments. Thus, right after getting op0 in this case or right after computing n for the n != 1 check. I think you need to verify that the type of 'op' is actually the element type of op0. The

Re: symbolic names for processor IDs

2012-09-08 Thread Uros Bizjak
Hello! The x86 cpuid instruction returns a processor ID and the __get_cpuid_max function even explicitly makes the %ebx value directly available. But users of that function have to use a cryptic constant. How about adding a few macros to make this more transparent? There are some other

Re: [ping][PATCH] Power: Reorder a sign-extend RTL pattern for readability

2012-09-08 Thread David Edelsohn
While examining the Power MD file seeking the explanation for a problem I saw I have noticed a change in the past separated one of the instruction splitters from its corresponding instruction pattern. Several unrelated patterns were inserted between the two, presumably by accident where the

Re: [patch, mips] New mips triplet for multilib linux builds

2012-09-08 Thread Richard Sandiford
Steve Ellcey sell...@mips.com writes: On Fri, 2012-09-07 at 07:52 +0100, Richard Sandiford wrote: MIPS_ABI_DEFAULT and MIPS_ISA_DEFAULT are better set in config.gcc. That also allows you to handle (say) mipsisa32-mti-linux-gnu vs. mipsisa64-mti-linux-gnu. I think in general we want more

Re: symbolic names for processor IDs

2012-09-08 Thread Ulrich Drepper
On Sat, Sep 8, 2012 at 7:17 AM, Uros Bizjak ubiz...@gmail.com wrote: There are some other cpuid vendor signatures than AMD and Intel, and probably there will be some more. Sure, if there are still people using those they can easily add those. IMO, anybody using __get_cpuid_max call should

Re: [Patch, PR 54128] ira.c change to fix mips bootstrap

2012-09-08 Thread Richard Sandiford
Don't have time to look at this in detail, but FWIW: Steve Ellcey sell...@mips.com writes: On Wed, 2012-09-05 at 08:15 +0200, Jakub Jelinek wrote: On Fri, Aug 31, 2012 at 10:58:51AM -0700, Steve Ellcey wrote: Here is my patch to fix the bootstrap comparision failure (PR 54128) on MIPS.

Re: [PATCH] Fix C++ bootstrap ICE

2012-09-08 Thread Mark Kettenis
Date: Fri, 07 Sep 2012 14:12:29 -0400 From: Jason Merrill ja...@redhat.com OK, thanks. I don't have commit access (or at least I'm not on the write after approval list). Would you (or somebody else) be so kind to commit this fix for me? Thanks, Mark

Re: [PATCH] Add -fmem-report-wpa

2012-09-08 Thread Hans-Peter Nilsson
On Thu, 6 Sep 2012, Andi Kleen wrote: From: Andi Kleen a...@linux.intel.com Passed bootstrap and testsuite on x86_64. +++ b/gcc/lto/lto.c @@ -2016,6 +2016,8 @@ do_whole_program_analysis (void) /* Show the LTO report before launching LTRANS. */ if (flag_lto_report)

Re: [PATCH] Add -fmem-report-wpa

2012-09-08 Thread Andi Kleen
On Sat, Sep 08, 2012 at 02:37:18PM -0400, Hans-Peter Nilsson wrote: On Thu, 6 Sep 2012, Andi Kleen wrote: From: Andi Kleen a...@linux.intel.com Passed bootstrap and testsuite on x86_64. +++ b/gcc/lto/lto.c @@ -2016,6 +2016,8 @@ do_whole_program_analysis (void) /* Show the LTO

Re: [PATCH] Fix C++ bootstrap ICE

2012-09-08 Thread Gerald Pfeifer
On Sat, 8 Sep 2012, Mark Kettenis wrote: I don't have commit access (or at least I'm not on the write after approval list). Would you (or somebody else) be so kind to commit this fix for me? I'll take care of it. Gerald

Re: [middle-end] Add machine_mode to address_cost target hook

2012-09-08 Thread Hans-Peter Nilsson
Oleg Endo wrote: On Thu, 2012-09-06 at 14:41 +0200, Georg-Johann Lay wrote: The change is definitely in the right direction, but I wonder how it helps to fix code bloats of 300%-400% as in PR52543? I'm not familiar with the AVR parts. BTW, There was a small change in lower-subreg which

Ping: [PATCH] top-level libtool / configure.ac PIC support for AIX

2012-09-08 Thread David Edelsohn
As mentioned in http://gcc.gnu.org/ml/gcc-patches/2012-08/msg01364.html GCC now is performing optimizations in binds_local_p() that necessitates adding -fPIC when compiling objects for shared libraries on AIX. The libtool patch has been accepted upstream. -fPIC creates unique names for some

Re: [PATCH] Set correct source location for deallocator calls

2012-09-08 Thread Dehao Chen
Hi, I've added a libjava unittest which verifies that this patch will not break Java debug info. I've also incorporated Richard's review in the previous mail. Attached is the new patch, which passed bootstrap and all gcc/libjava testsuites on x86. Is it ok for trunk? Thanks, Dehao

[committed] PA cost update

2012-09-08 Thread John David Anglin
The attached patch improves the cost estimate for multiplication, etc, on parisc. It was noted debugging various PRs that the cost for multiplication of long longs was seriously underestimated. Tested on hppa2.0w-hp-hpux11.11, hppa64-hp-hpux11.11 and hppa-unknown-linux-gnu. Committed to trunk.

Re: [middle-end] Add machine_mode to address_cost target hook

2012-09-08 Thread Georg-Johann Lay
Hans-Peter Nilsson schrieb: Oleg Endo wrote: Georg-Johann Lay wrote: The change is definitely in the right direction, but I wonder how it helps to fix code bloats of 300%-400% as in PR52543? I'm not familiar with the AVR parts. BTW, There was a small change in lower-subreg which required some

Re: symbolic names for processor IDs

2012-09-08 Thread Ulrich Drepper
On Sat, Sep 8, 2012 at 7:17 AM, Uros Bizjak ubiz...@gmail.com wrote: There are some other cpuid vendor signatures than AMD and Intel, How about a patch with this complete list? d-gcc-cpuid-signature Description: Binary data