Re: [PATCH] Fix vectorizable_reduction (PR tree-optimization/56350)

2013-02-19 Thread Richard Biener
On Mon, 18 Feb 2013, Jakub Jelinek wrote: > Hi! > > On the following testcase, a = a / 8; looks like reasonable reduction > statement, but we pattern recognize it as > patt_1 = a < 0 ? 7 : 0; > patt_2 = a + patt_1; > a = patt2 >> 3; > and in the first pattern stmt the reduction operand is in the

Re: [PATCH] Fix create_expression_by_pieces (PR tree-optimization/56381)

2013-02-19 Thread Richard Biener
On Mon, 18 Feb 2013, Jeff Law wrote: > On 02/18/13 15:32, Jakub Jelinek wrote: > > Hi! > > > > This accesses one past the last initialized argument. > > Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux, ok for > > trunk? > > > > 2013-02-18 Jakub Jelinek > > > > PR tree

Re: [patch] [libffi] do not install libffi library, headers and documentation

2013-02-19 Thread Richard Biener
On Mon, Feb 18, 2013 at 6:02 PM, Matthias Klose wrote: > Am 12.02.2013 13:45, schrieb Richard Biener: >> On Tue, Feb 12, 2013 at 1:44 PM, Richard Biener >> wrote: >>> On Tue, Feb 12, 2013 at 1:30 PM, Matthias Klose wrote: The libffi library, headers and documentation are still installed, al

Re: [PATCH, PR 55334] Disable IPA-CP on restrict pointers to arrays

2013-02-19 Thread Richard Biener
On Mon, Feb 18, 2013 at 9:06 PM, Martin Jambor wrote: > Hi, > > after much pondering about PR 55334 I came to conclusion that no nice > fix to the regression could be introduced in stage4. So for the sake > of the SPEC 200 benchmark I decided to cripple IPA-CP on restrict > pointers to arrays so

Re: [PATCH, PR 55334] Disable IPA-CP on restrict pointers to arrays

2013-02-19 Thread Martin Jambor
Hi, On Tue, Feb 19, 2013 at 10:21:32AM +0100, Richard Biener wrote: > On Mon, Feb 18, 2013 at 9:06 PM, Martin Jambor wrote: > > Hi, > > > > after much pondering about PR 55334 I came to conclusion that no nice > > fix to the regression could be introduced in stage4. So for the sake > > of the SP

Re: [PATCH, PR 55334] Disable IPA-CP on restrict pointers to arrays

2013-02-19 Thread Richard Biener
On Tue, Feb 19, 2013 at 12:22 PM, Martin Jambor wrote: > Hi, > > On Tue, Feb 19, 2013 at 10:21:32AM +0100, Richard Biener wrote: >> On Mon, Feb 18, 2013 at 9:06 PM, Martin Jambor wrote: >> > Hi, >> > >> > after much pondering about PR 55334 I came to conclusion that no nice >> > fix to the regres

[PATCH] Fix PR56384

2013-02-19 Thread Richard Biener
This fixes PHI value-numbering which did not correctly distinguish types in case of still VN_TOP PHI arguments. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2013-02-19 Richard Biener PR tree-optimization/56384 * tree-ssa-sccvn.h (struct vn_phi_s): A

[PATCH] Fix PR 55889

2013-02-19 Thread Andrey Belevantsev
Hello, As we discussed in the PR, the problem here is that the selective scheduler does not expect that renaming a hard register to a pseudo would result in extra dependencies. The dependencies come from implicit clobbers code of sched-deps.c, so I've made a minimal patch that checks only for

[4.7, testsuite] Clear hwcap_2 with Sun ld

2013-02-19 Thread Rainer Orth
I'd like to backport [testsuite] Clear hwcap_2 with Sun ld http://gcc.gnu.org/ml/gcc-patches/2012-06/msg01246.html 2012-06-19 Rainer Orth * gcc.target/i386/clearcapv2.map: New file. * gcc.target/i386/i386.exp: Try it first before clearcap.map. to the 4.7 branc

Re: [PATCH] Fix PR 55889

2013-02-19 Thread Alexander Monakov
On Tue, 19 Feb 2013, Andrey Belevantsev wrote: > Hello, > > As we discussed in the PR, the problem here is that the selective scheduler > does not expect that renaming a hard register to a pseudo would result in > extra dependencies. The dependencies come from implicit clobbers code of > sched

[PATCH] Speedup IVOPTs

2013-02-19 Thread Richard Biener
This speeds up IVOPTs by optimizing its hottest function when compiling polyhedron linpk. The datastructure used for recording use, candidate costs (a hashtable) should make O(1) queries on average - but it turns out that for use, candidate queries that have no entry recorded in it it is O(n) cur

Re: [PATCH] Speedup IVOPTs

2013-02-19 Thread Jakub Jelinek
On Tue, Feb 19, 2013 at 02:59:46PM +0100, Richard Biener wrote: > This speeds up IVOPTs by optimizing its hottest function when compiling > polyhedron linpk. The datastructure used for recording use, candidate > costs (a hashtable) should make O(1) queries on average - but it turns > out that for

[PATCH] Speedup lookup_constraint for 2-letter constraints

2013-02-19 Thread Richard Biener
This speeds up lookup_constraint by allowing the generic folding for n == 1 strncmp calls to trigger, handling two-letter constraints by two char comparisons instead of one and a call to strncmp (which ends up not being inlined because it's considered cold ...). Bootstrap & regtest pending on x86

Re: [PATCH] Speedup IVOPTs

2013-02-19 Thread Richard Biener
On Tue, 19 Feb 2013, Jakub Jelinek wrote: > On Tue, Feb 19, 2013 at 02:59:46PM +0100, Richard Biener wrote: > > This speeds up IVOPTs by optimizing its hottest function when compiling > > polyhedron linpk. The datastructure used for recording use, candidate > > costs (a hashtable) should make O(1

Re: [PATCH] Speedup lookup_constraint for 2-letter constraints

2013-02-19 Thread Jakub Jelinek
On Tue, Feb 19, 2013 at 04:07:49PM +0100, Richard Biener wrote: > > This speeds up lookup_constraint by allowing the generic folding > for n == 1 strncmp calls to trigger, handling two-letter constraints > by two char comparisons instead of one and a call to strncmp (which > ends up not being inli

Re: [PATCH] Speedup IVOPTs

2013-02-19 Thread Jakub Jelinek
On Tue, Feb 19, 2013 at 04:08:57PM +0100, Richard Biener wrote: > Well, I used the lame variant to avoid a zero argument to ceil_log2 ... > I'll use yours instead. > > Ok? Yep. Jakub

Speedup recognizing multi-letter constraints

2013-02-19 Thread Michael Matz
Hi, from IRC: "[15:45:21] ick - lookup_constraint for multi-letter constraints is quite expensive ... strncmp is not expanded inline for some reason" Instead of fiddling with strncmp inlining, simply generate better code from the start for two character constraints: switch (str[0]) { ca

Re: RFC: [PATCH,ARM] Fix 56110

2013-02-19 Thread Richard Earnshaw
On 18/02/13 21:47, Tilman Sauerbeck wrote: Hi, adding the instruction pattern below fixes my testcase for PR 56110; however I'm not sure if adding a new pattern is the correct way to go. I duplicated the andsi3_compare0_scratch pattern, and lifted the requirement that the 2nd operand be an arm_n

Ping: [Patch,avr]: Add more devices take #2: 2/2

2013-02-19 Thread Georg-Johann Lay
CCing Richard in case this needs RM approval (I don't think so because the patch just changes ternary parts). Ping #1 for: http://gcc.gnu.org/ml/gcc-patches/2013-02/msg00579.html Georg-Johann Lay schrieb: And here 2/2 with the device -> arch mapping for gas. Ok for trunk? Johann

Ping: [Patch,avr]: Add more devices take #2: 1/2

2013-02-19 Thread Georg-Johann Lay
CCing Richard in case this needs RM approval (I don't think so because the patch just changes ternary parts). Ping #1 for: http://gcc.gnu.org/ml/gcc-patches/2013-02/msg00577.html http://gcc.gnu.org/ml/gcc-patches/2013-02/msg00576.html Georg-Johann Lay schrieb: Better attach the patch... Georg

Re: [PATCH] Temporarily revert Steven's PCH changes for 4.8 (PR pch/54117)

2013-02-19 Thread David Edelsohn
On Thu, Feb 14, 2013 at 7:09 PM, Joel Brobecker wrote: >> >> AFAICT, for gcc+gas it should already work with binutils that include >> >> the AdaCore patch for DWARF support. But this has apparently not been >> >> tested with AIX ld, and there are AdaCore local patches pending. >> >> http://sourcew

Re: [C++ Patch] PR 56373

2013-02-19 Thread Paolo Carlini
... in case, we may want to do something like the below too. Thanks, Paolo. /// Index: cp/cp-tree.h === --- cp/cp-tree.h(revision 196136) +++ cp/cp-tree.h(working copy) @@ -5834,6 +5834,7 @@ extern bo

Re: [PATCH] Temporarily revert Steven's PCH changes for 4.8 (PR pch/54117)

2013-02-19 Thread Joel Brobecker
> Thanks! That's great progress on the Binutils side. You are very welcome. > What is the status of patches for GCC to generate DWARF for AIX XCOFF > and do the AIX assembler and linker recognize, consume and process the > directives produced by GCC correctly? I haven't looked at this part in d

[Patch, fortran] [0/5] PR54730 ICE: confused by type-like fonctions

2013-02-19 Thread Mikael Morin
Hello, this is a fix for cases like: program main implicit none intrinsic :: real print *,(/ real(a = 1) /) end where `real(a = 1)' is initially parsed as a typespec, creating a symbol for 'a' along the way. The match fails, and then it is parsed as a constructor element and accepted that

[Patch, fortran] [2/5] PR54730 ICE: confused by type-like fonctions: Extract restore_old_symbol.

2013-02-19 Thread Mikael Morin
The ChangeLog says it all. 2013-02-15 Mikael Morin PR fortran/54730 * symbol.c (gfc_undo_symbols): Move code... (restore_old_symbol): ... here as a new function. diff --git a/symbol.c b/symbol.c index 2c122d0..4f1752d 100644 --- a/symbol.c +++ b/symbol.c @@ -2878,6 +28

[Patch, fortran] [1/5] PR54730 ICE: confused by type-like fonctions: Move to the vec API.

2013-02-19 Thread Mikael Morin
This packs symbol.c's 'changed_syms' and 'tentative_tbp' variables into a new 'gfc_change_set' structure using the vec API. 2013-02-15 Mikael Morin PR fortran/54730 * Make-lang.in (F95_PARSER_OBJS): Add dependency to vec.h * gfortran.h: Include vec.h (gfc_change

[Patch, fortran] [3/5] PR54730 ICE: confused by type-like fonctions: Fix restore_old_symbol

2013-02-19 Thread Mikael Morin
This is a fix for what looks like a thinko. The other patches don't depend on it, so it can well wait for 4.9 if that is preferred. 2013-02-15 Mikael Morin PR fortran/54730 * symbol.c (restore_old_symbol): Fix thinko. diff --git a/symbol.c b/symbol.c index 4f1752d..e4dbb41 100

[Patch, fortran] [4/5] PR54730 ICE: confused by type-like fonctions: Support multiple change sets.

2013-02-19 Thread Mikael Morin
A new field 'previous' is added to the 'gfc_change_set' structure so that it can be used as a stack. New procedures are added to use the new partial undoing feature, namely: gfc_new_checkpoint, gfc_drop_last_checkpoint and gfc_restore_last_checkpoint. They will be used in the next patch. 2013-02-1

[Patch, fortran] [5/5] PR54730 ICE: confused by type-like fonctions: Fix gfc_match_array_constructor.

2013-02-19 Thread Mikael Morin
This is the fix for the PR. 2013-02-15 Mikael Morin PR fortran/54730 * array.c (gfc_match_array_constructor): Set a checkpoint before matching a typespec. Drop it on success, restore it otherwise. diff --git a/array.c b/array.c index 6787c05..b4a028b 100644 --- a/arr

4.7 backports

2013-02-19 Thread Jakub Jelinek
Hi! I've backported a bunch of patches from trunk to 4.7, after bootstrapping/regtesting them on x86_64-linux and i686-linux. Jakub 2013-02-19 Jakub Jelinek Backported from mainline 2012-11-27 Jakub Jelinek PR tree-optimization/55110 * tree-vect-loo

Re: [4.7, testsuite] Clear hwcap_2 with Sun ld

2013-02-19 Thread Janis Johnson
On 02/19/2013 05:25 AM, Rainer Orth wrote: > I'd like to backport > > [testsuite] Clear hwcap_2 with Sun ld > http://gcc.gnu.org/ml/gcc-patches/2012-06/msg01246.html > > 2012-06-19 Rainer Orth > > * gcc.target/i386/clearcapv2.map: New file. > * gcc.target/i386/i386.e

Re: FW: [PATCH] [MIPS] microMIPS gcc support

2013-02-19 Thread Richard Sandiford
Thanks, this is looking much better now. "Moore, Catherine" writes: > I'd also like to support -mno-jals for backward compatibility. Are you > okay with that? If so, I'll submit as a separate patch. Hmm, to be honest, I'd rather you kept it local to the Mentor toolchain. The way the patch is w

Re: [PATCH] Temporarily revert Steven's PCH changes for 4.8 (PR pch/54117)

2013-02-19 Thread David Edelsohn
On Tue, Feb 19, 2013 at 11:11 AM, Joel Brobecker wrote: > I haven't looked at this part in details, but our recollection is that > it's a matter of configuring GCC with --with-gnu-as and --with-gnu-ld, > together with a small patch to make DWARF the default. AFAIK, the rest > should be working as

Re: [PATCH] Temporarily revert Steven's PCH changes for 4.8 (PR pch/54117)

2013-02-19 Thread Paolo Bonzini
Il 13/02/2013 22:12, Steven Bosscher ha scritto: > On Wed, Feb 13, 2013 at 4:54 PM, Jakub Jelinek wrote: >> Hi! >> >> As agreed on in the PR, here is the revertion of 3 commits, so that >> PCH works again for -gstabs and other debug info formats for 4.8 >> release. For 4.9 we should either remove

linaro/gcc-4_8-branch created and documented

2013-02-19 Thread Matthew Gretton-Dann
All, I have just created a distribution branch: 'linaro/gcc-4_8-branch'. I have committed the attached patch to the wwwdocs CVS repository to document this branch (and future Linaro branches). The branch will track the equivalent FSF release branch (once created) and also accept backports o

[committed] Fix libquadmath strtoflt128 at -O0 (PR libquadmath/56379)

2013-02-19 Thread Jakub Jelinek
Hi! I've committed to trunk what has been accepted earlier today for glibc strtod_l.c, a fix for -O0 compilation of the file. Bootstrapped/regtested on x86_64-linux and i686-linux. 2013-02-19 Jakub Jelinek PR libquadmath/56379 * strtod/strtod_l.c (mpn_lshift_1): Rewritten as

Re: RFC: [PATCH,ARM] Fix 56110

2013-02-19 Thread Tilman Sauerbeck
Richard Earnshaw [2013-02-19 15:12]: Hi, thanks for your reply. > [...] > However, the question you need to be asking is why the pattern immediately > before the one you've added is not matching. The compiler knows how to add > clobbers, so I'm surprised that you're finding a new pattern to be >

Re: FW: [PATCH] [MIPS] microMIPS gcc support

2013-02-19 Thread Maciej W. Rozycki
On Tue, 19 Feb 2013, Richard Sandiford wrote: > > Index: gcc.target/mips/umips-lwp-swp-2.c > > === > > --- gcc.target/mips/umips-lwp-swp-2.c (revision 0) > > +++ gcc.target/mips/umips-lwp-swp-2.c (revision 0) > > @@ -0,0 +

[PATCH] Fix ccp (PR tree-optimization/56396)

2013-02-19 Thread Jakub Jelinek
Hi! On the following patch gcc ICEs because malloc memory is corrupted. The problem is that const_val array is allocated at the start of the pass, but during the execution of ccp some new SSA_NAMEs are created (update_call_from_tree if I remember well). The following patch fixes that by making co

[Patch, libfortran] PR 30162 pipe I/O regression with 4.7/4.8

2013-02-19 Thread Janne Blomqvist
Hi, attached is an attempt to fix writing formatted sequential I/O to a pipe (The PR was reopened in comment #22, which refers to formatted I/O so the PR title is incorrect). I think the underlying reason was that the introduction of the ssize() member function led to a change in semantics (size o

Re: Patch for 4.7: Avoid subreg'ing VFP D registers in big-endian mode

2013-02-19 Thread Ramana Radhakrishnan
On Fri, Feb 15, 2013 at 10:40 PM, Seth LaForge wrote: > gcc 4.7.2 generates incorrect code for big-endian ARM VFP processors > when storing a local double to a packed memory location, as described > in bug 56351. > http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56351 > > A fix has been submitted to t

Re: [Patch, microblaze]: Handle 0x80000000 as 32bit signed dividend

2013-02-19 Thread Michael Eager
On 02/10/2013 10:38 PM, David Holsgrove wrote: Handle 0x8000 as 32bit signed dividend Changelog 2013-02-11 Edgar E. Iglesias * config/microblaze/modsi3.S (modsi3): Fix case with 0x8000 as dividend. Committed revision 196156. -- Michael Eagerea...@eagercon.com 1960 Park

Re: [Patch, microblaze]: Avoid PC relative branches between sections

2013-02-19 Thread Michael Eager
On 02/10/2013 10:38 PM, David Holsgrove wrote: Avoid PC relative branches between sections by further checking that the symbol we are calling is declared as a function before using PC relative calls. Changelog 2013-02-11 Edgar E. Iglesias * config/microblaze/microblaze.md (call_value_inte

Re: [Patch, microblaze]: Add support for the CLZ insn

2013-02-19 Thread Michael Eager
On 02/10/2013 10:39 PM, David Holsgrove wrote: Add support for the CLZ insn Will be used if pattern-compare is enabled and the targeted core is newer than v8.10.a. Changelog 2013-02-11 Edgar E. Iglesias * config/microblaze/microblaze.c: microblaze_has_clz = 0 Add version check for

[v3] correct doxygen parse

2013-02-19 Thread Benjamin De Kosnik
Fixups for doxygen. Since 2012-11-20, most of C++11 has been ignored by doxygen, resulting in some missing modules documentation even in HTML output. Jakub, this fixes the libstdc++.tag validation errors. I've regenerated the gcc.gnu.org docs for 4.8.0. They look equivalent or better than 4.7.2

Re: Version specific onlinedocs page

2013-02-19 Thread Gerald Pfeifer
On Mon, 18 Feb 2013, Tobias Burnus wrote: If "we" means that you are volunteering ;-), absolutely, go for it! ;-) How about the following patch? If it is okay, I will add the remaining index.html for 4.6 and 4.7 and update gcc-4.6/index.html – and then commit it. Looks good to me. Perhaps

Re: [C++ Patch] PR 56373

2013-02-19 Thread Jason Merrill
OK. Jason

Re: RFC: [PATCH,ARM] Fix 56110

2013-02-19 Thread Tilman Sauerbeck
Tilman Sauerbeck [2013-02-19 23:26]: > However it breaks the case where the 2nd operand is a const_int that > *can* be used as an immediate (eg 0x80), and ends up generating the > AND/CMP combination. ... and that would be because I changed the operand patterns in zeroextractsi_compare0_scratch a