[PATCH, testsuite]: Fix builtin-complex-1.c failure on alpha

2011-10-05 Thread Uros Bizjak
Hello! 2011-10-05 Uros Bizjak ubiz...@gmail.com * gcc.dg/torture/builtin-complex-1.c: Use dg-add-options ieee. Tested on alphaev68-pc-linux-gnu, committed to mainline. Uros. Index: gcc.dg/torture/builtin-complex-1.c ===

[PATCH] Add an intermediate coverage format for gcov

2011-10-05 Thread शरद सिंघई
This patch was earlier submitted to google/main, but I propose itfor the trunk as well. This patch adds an intermediate coverage format (enabled via 'gcov-i'). This is a compact format as it does not require source files. The new option ('gcov -i') outputs .gcov files in an intermediate textformat

[PING] PR50325 store_bit_field: Fix for big endian targets

2011-10-05 Thread Andreas Krebbel
This fixes many C++ tests on s390x and PPC64: http://gcc.gnu.org/ml/gcc-patches/2011-09/msg01220.html

Re: [PATCH] Add an intermediate coverage format for gcov

2011-10-05 Thread शरद सिंघई
Sorry about the garbled message. My mistake with the mailer. Here is what I really intended to send. This patch was earlier submitted to google/main, but I propose itfor trunk as well. This patch adds an intermediate coverage format (enabled via 'gcov-i'). This is a compact format as it does

Re: [PATCH] Add an intermediate coverage format for gcov

2011-10-05 Thread Sharad Singhai
Sorry about the badly formatted mail. Here is another version with a different mailer. -Sharad This patch was earlier submitted to google/main, but I propose it for the trunk as well. This patch adds an intermediate coverage format (enabled via 'gcov -i'). This is a compact format as it

[PATCH, i386]: Introduce ix86_emit_binop

2011-10-05 Thread Uros Bizjak
Hello! No functional change. 2011-10-05 Uros Bizjak ubiz...@gmail.com * config/i386/i386.c (ix86_emit_binop): New static function. (ix86_split_lea_for_addr): Use ix86_emit_binop to emit add and shl instructions. (x86_output_mi_thunk): Use ix86_emit_binop to

Re: [PATCH] Handle side-effects of EH optimizations of callees

2011-10-05 Thread Richard Guenther
On Tue, Oct 4, 2011 at 9:02 PM, Maxim Kuvyrkov ma...@codesourcery.com wrote: On 5/10/2011, at 1:49 AM, Richard Guenther wrote: On Tue, Oct 4, 2011 at 9:17 AM, Maxim Kuvyrkov ma...@codesourcery.com wrote: Richard, The following patch fixes a CFG consistency problem. When early IPA

[PATCH] Use widening_optab_handler when expanding highpart mults

2011-10-05 Thread Andreas Krebbel
Hi, the optab_handler uses in expand_mult_highpart_optab haven't been replaced with the widening_optab_handler yet. Fixed with attached patch. Tested on s390x and x86_64. Bye, -Andreas- 2011-10-05 Andreas Krebbel andreas.kreb...@de.ibm.com * expmed.c (expand_mult_highpart_optab):

Re: New warning for expanded vector operations

2011-10-05 Thread Richard Guenther
On Wed, Oct 5, 2011 at 12:18 AM, Artem Shinkarov artyom.shinkar...@gmail.com wrote: Hi Here is a patch to inform a programmer about the expanded vector operation. Bootstrapped on x86-unknown-linux-gnu. ChangeLog:        * gcc/tree-vect-generic.c (expand_vector_piecewise): Adjust to        

Re: Avoid optimized out references to appear in lto symbol table

2011-10-05 Thread Jan Hubicka
Jan Hubicka hubi...@ucw.cz writes: Hi, GNU LD has bug about reporting references to hidden symbol sthat has been optimized out. This however made me notice that we do output into LTO symbol tables things that do not belong there. In partiuclar we often output extern inline

Re: [PATCH] Use widening_optab_handler when expanding highpart mults

2011-10-05 Thread Andrew Stubbs
On Wed 05 Oct 2011 09:33:09 BST, Andreas Krebbel wrote: Hi, the optab_handler uses in expand_mult_highpart_optab haven't been replaced with the widening_optab_handler yet. Apologies, I don't know how I missed that one? :( Andrew

Re: [PATCH, PR50527] Don't assume alignment of vla-related allocas.

2011-10-05 Thread Richard Guenther
On Tue, Oct 4, 2011 at 10:58 PM, Richard Henderson r...@redhat.com wrote: On 10/04/2011 01:17 PM, Tom de Vries wrote: In general, to fold vlas (which are lowered to allocas) to normal declarations, if the alloca argument is constant. Ah.  Ok, I suppose.  How often are you seeing this

Re: [PATCH, PR50527] Don't assume alignment of vla-related allocas.

2011-10-05 Thread Richard Guenther
On Tue, Oct 4, 2011 at 6:28 PM, Tom de Vries tom_devr...@mentor.com wrote: On 10/04/2011 03:03 PM, Richard Guenther wrote: On Tue, Oct 4, 2011 at 9:43 AM, Tom de Vries tom_devr...@mentor.com wrote: On 10/01/2011 05:46 PM, Tom de Vries wrote: On 09/30/2011 03:29 PM, Richard Guenther wrote: On

Re: [PATCH 2/2] allow certain kinds of inputs to top level asm()-s

2011-10-05 Thread Jan Hubicka
we have, like specifying the set of symbols _defined_ by a toplevel asm, right? I might misremember but sth like extern void foo (void); asm( foo); was supposed to do the trick. Or should we treat those as outputs (given you use inputs for symbol uses)? I don't recall any discussion

Re: [PATCH] Handle side-effects of EH optimizations of callees

2011-10-05 Thread Jan Hubicka
On Tue, Oct 4, 2011 at 9:02 PM, Maxim Kuvyrkov ma...@codesourcery.com wrote: On 5/10/2011, at 1:49 AM, Richard Guenther wrote: On Tue, Oct 4, 2011 at 9:17 AM, Maxim Kuvyrkov ma...@codesourcery.com wrote: Richard, The following patch fixes a CFG consistency problem. When early

Re: Ping #1: [Patch,AVR]: Clean-up some SP insns

2011-10-05 Thread Georg-Johann Lay
Ping #1: http://gcc.gnu.org/ml/gcc-patches/2011-09/msg01690.html Georg-Johann Lay wrote: This is just a code clean-up. The bulky code from *addhi3_sp_R_pc2 and *addhi3_sp_R_pc3 is done by a small C function that does the same (except that it prints some comment depending on -dp or

Re: [wwwdocs] IA-32/x86-64 Changes for upcoming 4.7.0 series

2011-10-05 Thread Kirill Yukhin
Thank you guys for your support! K

Commit: RX: Codegen bug fixes

2011-10-05 Thread Nick Clifton
Hi Guys, I am applying the patch below to fix a couple of bugs in the RX's machine description patterns. The first concerns the tablejump pattern, which needs to include a label to be referenced by the ASM_OUTPUT_ADDR_DIFF_ELT macro. The second problem is the ADDDI3 spiltter which was

Re: Ping #1: [Patch,AVR]: Clean-up some SP insns

2011-10-05 Thread Denis Chertykov
2011/10/5 Georg-Johann Lay a...@gjlay.de: Ping #1: http://gcc.gnu.org/ml/gcc-patches/2011-09/msg01690.html Georg-Johann Lay wrote: This is just a code clean-up. The bulky code from *addhi3_sp_R_pc2 and *addhi3_sp_R_pc3 is done by a small C function that does the same (except that it

[C++ Patch] PR 38980

2011-10-05 Thread Paolo Carlini
Hi, as analyzed by Jakub in the audit trail, after changes made by Mark back in 2005, constant_value_1 doesn't return aggregate constants for fear of inadvertent copies (in general their addresses must be the same everywhere). However, for the purpose of format checking in check_format_arg

[PATCH] Fix ICE in find_equal_ptrs (PR tree-optimization/50613)

2011-10-05 Thread Jakub Jelinek
Hi! I didn't consider that the rhs1 of a gimple cast might be something other than SSA_NAME. Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? 2011-10-05 Jakub Jelinek ja...@redhat.com PR tree-optimization/50613 * tree-ssa-strlen.c

Re: New warning for expanded vector operations

2011-10-05 Thread Richard Guenther
On Wed, Oct 5, 2011 at 1:28 PM, Artem Shinkarov artyom.shinkar...@gmail.com wrote: On Wed, Oct 5, 2011 at 9:40 AM, Richard Guenther richard.guent...@gmail.com wrote: On Wed, Oct 5, 2011 at 12:18 AM, Artem Shinkarov artyom.shinkar...@gmail.com wrote: Hi Here is a patch to inform a programmer

Re: [PATCH] Fix ICE in find_equal_ptrs (PR tree-optimization/50613)

2011-10-05 Thread Richard Guenther
On Wed, Oct 5, 2011 at 1:31 PM, Jakub Jelinek ja...@redhat.com wrote: Hi! I didn't consider that the rhs1 of a gimple cast might be something other than SSA_NAME.  Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? Ok. Thanks, Richard. 2011-10-05  Jakub

[PATCH] Propagate vector CONSTRUCTOR and element extract in SCCVN

2011-10-05 Thread Richard Guenther
Noticed from vector lowering testcases. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. Richard. 2011-10-05 Richard Guenther rguent...@suse.de * tree-ssa-sccvn.c (vn_get_expr_for): Handle CONSTRUCTOR of vector type. (simplify_unary_expression):

[PATCH, testsuite]: Fix avx256-unaligned-{load,store}-3.c scan-assembler failures

2011-10-05 Thread Uros Bizjak
Hello! Atom does not vectorize DFmode arrays by default, so add -mtune=generic to dg-options to fix scan-assembler failures [1]. 2011-10-05 Uros Bizjak ubiz...@gmail.com * gcc.target/i386/avx256-unaligned-load-3.c (dg-options): Add -mtune=generic. *

Re: Modify gcc for use with gdb (issue5132047)

2011-10-05 Thread Diego Novillo
On 11-09-27 03:37 , Richard Guenther wrote: On Tue, Sep 27, 2011 at 9:35 AM, Richard Guenther richard.guent...@gmail.com wrote: On Tue, Sep 27, 2011 at 9:14 AM, Jakub Jelinekja...@redhat.com wrote: On Mon, Sep 26, 2011 at 03:05:00PM -0700, Lawrence Crowl wrote: There a non-transparent

[PATCH] Fix PR38885

2011-10-05 Thread Richard Guenther
I'm testing a pair of patches to fix PR38885 (for constants) and PR38884 (for non-constants) stores to complex/vector memory and CSE of component accesses from SCCVN. This is the piece that handles stores from constants and partial reads of it. We can conveniently re-use fold-const native

Re: Modify gcc for use with gdb (issue5132047)

2011-10-05 Thread Diego Novillo
On Wed, Oct 5, 2011 at 09:45, Richard Guenther richard.guent...@gmail.com wrote: On Wed, Oct 5, 2011 at 3:18 PM, Diego Novillo dnovi...@google.com wrote: On 11-09-27 03:37 , Richard Guenther wrote: On Tue, Sep 27, 2011 at 9:35 AM, Richard Guenther richard.guent...@gmail.com  wrote: On Tue,

Re: Modify gcc for use with gdb (issue5132047)

2011-10-05 Thread Jakub Jelinek
On Wed, Oct 05, 2011 at 10:10:44AM -0400, Diego Novillo wrote: Why not use the artificial attribute on them instead?  At least what is documented is exactly what we want (well, at least it seems to me). Yes, I forgot to mention in my reply. I tried it, but you still step into the

Re: Modify gcc for use with gdb (issue5132047)

2011-10-05 Thread Ian Lance Taylor
Diego Novillo dnovi...@google.com writes: It is much more important to optimize my debugging time as experienced developer resources are more scarce than some random unexperienced guy that happens to dig into GCC. ;) or not really ;). You are being facetious, I hope. Part of the reason

Re: Modify gcc for use with gdb (issue5132047)

2011-10-05 Thread Richard Guenther
On Wed, Oct 5, 2011 at 4:10 PM, Diego Novillo dnovi...@google.com wrote: On Wed, Oct 5, 2011 at 09:45, Richard Guenther richard.guent...@gmail.com wrote: On Wed, Oct 5, 2011 at 3:18 PM, Diego Novillo dnovi...@google.com wrote: On 11-09-27 03:37 , Richard Guenther wrote: On Tue, Sep 27, 2011

Re: [PATCH] Add support for more sparc VIS 3.0 instructions.

2011-10-05 Thread Marc Glisse
On Tue, 4 Oct 2011, David Miller wrote: There are only a few VIS 3.0 instructions left after this, Hello, I am happy to see all this work, but I was wondering: are these instructions documented somewhere? It makes sense for gcc to only provide a list in extend.texi, but I couldn't find the

Commit: RX: Add PID support

2011-10-05 Thread Nick Clifton
Hi Guys, I am applying the attached patch to add support for position independent data to the RX backend. When this mode is enabled constant data is referenced via an offset from a base address held in a fixed register. This allows the position of this data to be determined at

Re: Initial shrink-wrapping patch

2011-10-05 Thread Richard Guenther
On Wed, Oct 5, 2011 at 12:29 AM, Richard Henderson r...@redhat.com wrote: On 10/04/2011 03:10 PM, Bernd Schmidt wrote:       * doc/invoke.texi (-fshrink-wrap): Document.       * opts.c (default_options_table): Add it.       * common.opt (fshrink-wrap): Add.       * function.c

Re: Modify gcc for use with gdb (issue5132047)

2011-10-05 Thread Tom Tromey
Diego == Diego Novillo dnovi...@google.com writes: Diego Tom, Cary, Ian, any suggestions? We are trying to figure out a Diego compromise for tiny inline functions that are generally a nuisance Diego when debugging. The scenario is a call like this: big_function_foo Diego (inlined_f (x),

Re: Modify gcc for use with gdb (issue5132047)

2011-10-05 Thread Diego Novillo
On Wed, Oct 5, 2011 at 10:51, Richard Guenther richard.guent...@gmail.com wrote: Did you also mark the function with always_inline?  That's a requirement as artificial only works for inlined function bodies. Yeah. It doesn't quite work as I expect it to. It steps into the function at odd

Re: Modify gcc for use with gdb (issue5132047)

2011-10-05 Thread Paolo Carlini
On 10/05/2011 05:27 PM, Tom Tromey wrote: FWIW, I wrote the macro define stuff that Paolo posted back when I was actively hacking on gcc. Yes, thanks Tom! Actually, I suspected that, but couldn't remember where I actually got it from, maybe you posted it on a public discussion thread

Re: Modify gcc for use with gdb (issue5132047)

2011-10-05 Thread Diego Novillo
On Wed, Oct 5, 2011 at 11:27, Tom Tromey tro...@redhat.com wrote: Diego I proposed extending #pragma GCC options to bracket these functions Diego with -g0.  This would help reduce the impact of debug info size. I think this is fixing the wrong component: it means making a one-size-fits-all

RFC: ARM: Add comments to emitted .eabi_attribute directives

2011-10-05 Thread Nick Clifton
Hi Richard, Hi Paul, Hi Ramana, Well my idea to have a header file containing all of the possible binary file attributes does not seem to have taken off, so here is a much simpler patch for the GCC ARM backend. It just adds comments to the .eabi_directives (when -dA or -fverbose-asm are

Re: Modify gcc for use with gdb (issue5132047)

2011-10-05 Thread Jakub Jelinek
On Wed, Oct 05, 2011 at 11:42:51AM -0400, Diego Novillo wrote: Richi, Jakub, Lawrence, would you be OK with this approach? IIUC, this means we'd have to add a bunch of blacklist commands to gcc/gdbinit.in. I don't mind if it goes into gdb, but IMHO the blacklisting should definitely default

Re: Initial shrink-wrapping patch

2011-10-05 Thread Bernd Schmidt
On 10/05/11 17:13, Richard Guenther wrote: On Wed, Oct 5, 2011 at 12:29 AM, Richard Henderson r...@redhat.com wrote: On 10/04/2011 03:10 PM, Bernd Schmidt wrote: * doc/invoke.texi (-fshrink-wrap): Document. * opts.c (default_options_table): Add it. * common.opt

Ping! Re: [RFA/ARM][Patch 01/02]: Thumb2 epilogue in RTL

2011-10-05 Thread Sameera Deshpande
Ping! http://gcc.gnu.org/ml/gcc-patches/2011-09/msg01854.html On Wed, 2011-09-28 at 17:15 +0100, Sameera Deshpande wrote: Hi! This patch generates Thumb2 epilogues in RTL form. The work involves defining new functions, predicates and patterns along with few changes in existing code: *

Ping! Re: [RFA/ARM][Patch 02/02]: ARM epilogues in RTL

2011-10-05 Thread Sameera Deshpande
Ping! http://gcc.gnu.org/ml/gcc-patches/2011-09/msg01855.html On Wed, 2011-09-28 at 17:15 +0100, Sameera Deshpande wrote: Hi! This patch generates ARM epilogue in RTL form. The work defines new functions and reuses most of the static functions and patterns defined in the previous patch

[PATCH] Fix PR46556 (poor address generation)

2011-10-05 Thread William J. Schmidt
This patch addresses the poor code generation in PR46556 for the following code: struct x { int a[16]; int b[16]; int c[16]; }; extern void foo (int, int, int); void f (struct x *p, unsigned int n) { foo (p-a[n], p-c[n], p-b[n]); } Prior to the fix for PR32698, gcc calculated the

Re: Modify gcc for use with gdb (issue5132047)

2011-10-05 Thread Tom Tromey
Diego == Diego Novillo dnovi...@google.com writes: Tom    http://sourceware.org/bugzilla/show_bug.cgi?id=8287 Diego I think this could work. I'm not sure I like the idea of having to Diego specify all these blacklist commands, but I appreciate how it can make Diego debugging more flexible.

Re: [RFC] Split -mrecip

2011-10-05 Thread Michael Matz
Hi, On Mon, 19 Sep 2011, Uros Bizjak wrote: Looking at this topic again, I'd propose that x86 adopts approach from rs6000. The rs6000 approach is more extensible, and offers the same flexibility, due to !. So, x86 could have -mrecip=opt, with all, default, none, div, vec-div, divf,

Re: Modify gcc for use with gdb (issue5132047)

2011-10-05 Thread Tom Tromey
Jakub == Jakub Jelinek ja...@redhat.com writes: Jakub I don't mind if it goes into gdb, but IMHO the blacklisting should Jakub definitely default to blacklisting DW_AT_artificial inline functions Jakub (and allowing to unblacklist them), because the artificial attribute Jakub has been designed

Re: Initial shrink-wrapping patch

2011-10-05 Thread Richard Henderson
On 10/05/2011 08:59 AM, Bernd Schmidt wrote: Bootstrapping the following now. Ok? (Alternatively, could keep the redzone logic, but make it depend on !flag_shrink_wrap). It's a good space-saving optimization, that redzone logic. We should be able to keep it based on crtl-shrink_wrapped; that

Re: [PATCH] Fix PR46556 (poor address generation)

2011-10-05 Thread Paolo Bonzini
On 10/05/2011 06:13 PM, William J. Schmidt wrote: One other general question about the pattern-match transformation: Is this an appropriate transformation for all targets, or should it be somehow gated on available addressing modes on the target processor? Bootstrapped and regression tested on

Re: Fix pr50607 bconstp-3.c failure

2011-10-05 Thread Artem Shinkarov
On Wed, Oct 5, 2011 at 4:22 PM, Joseph S. Myers jos...@codesourcery.com wrote: On Wed, 5 Oct 2011, Artem Shinkarov wrote: On Tue, Oct 4, 2011 at 11:51 PM, Joseph S. Myers jos...@codesourcery.com wrote: On Tue, 4 Oct 2011, Artem Shinkarov wrote: Hi Here is the patch tho fix

Re: Fix pr50607 bconstp-3.c failure

2011-10-05 Thread Joseph S. Myers
On Wed, 5 Oct 2011, Artem Shinkarov wrote: Joseph, is it possible to commit the patch together with the space fixes? You should not commit whitespace fixes to lines not otherwise modified by a patch, except in a separate commit that *only* has whitespace or other formatting fixes, so that svn

Re: Fix pr50607 bconstp-3.c failure

2011-10-05 Thread Artem Shinkarov
On Wed, Oct 5, 2011 at 5:28 PM, Joseph S. Myers jos...@codesourcery.com wrote: On Wed, 5 Oct 2011, Artem Shinkarov wrote: Joseph, is it possible to commit the patch together with the space fixes? You should not commit whitespace fixes to lines not otherwise modified by a patch, except in a

Re: fix for c++/44473, mangling of decimal types, checked in

2011-10-05 Thread Peter Bergner
On Fri, 2011-09-30 at 10:37 -0700, Janis Johnson wrote: Patch http://gcc.gnu.org/ml/gcc-patches/2010-12/msg00625.html was approved by Jason last December but I never got around to checking it in. Paolo Carlini said in PR44473 that it was already approved and doesn't need a new approval, so I

Re: Modify gcc for use with gdb (issue5132047)

2011-10-05 Thread Diego Novillo
On Wed, Oct 5, 2011 at 11:28, Diego Novillo dnovi...@google.com wrote: On Wed, Oct 5, 2011 at 10:51, Richard Guenther richard.guent...@gmail.com wrote: Did you also mark the function with always_inline?  That's a requirement as artificial only works for inlined function bodies. Yeah.  It

Re: [PATCH] Add an intermediate coverage format for gcov

2011-10-05 Thread Mike Stump
On Oct 5, 2011, at 12:47 AM, Sharad Singhai wrote: This patch adds an intermediate coverage format (enabled via 'gcov -i'). This is a compact format as it does not require source files. I don't like any of the tags, I think if you showed them to 100 people that had used gcov before, very few

Re: [RFC] Split -mrecip

2011-10-05 Thread Uros Bizjak
On Wed, Oct 5, 2011 at 6:19 PM, Michael Matz m...@suse.de wrote: Looking at this topic again, I'd propose that x86 adopts approach from rs6000. The rs6000 approach is more extensible, and offers the same flexibility, due to !. So, x86 could have -mrecip=opt, with all, default, none, div,

Re: [PATCH] Fix PR46556 (poor address generation)

2011-10-05 Thread William J. Schmidt
On Wed, 2011-10-05 at 18:29 +0200, Steven Bosscher wrote: On Wed, Oct 5, 2011 at 6:13 PM, William J. Schmidt wschm...@linux.vnet.ibm.com wrote: * tree-ssa-loop-ivopts.c (copy_ref_info): Remove static token. Rather than this, why not move the function to common code somewhere?

Re: Initial shrink-wrapping patch

2011-10-05 Thread Bernd Schmidt
On 10/05/11 18:21, Richard Henderson wrote: On 10/05/2011 08:59 AM, Bernd Schmidt wrote: Bootstrapping the following now. Ok? (Alternatively, could keep the redzone logic, but make it depend on !flag_shrink_wrap). It's a good space-saving optimization, that redzone logic. We should be able

Re: Initial shrink-wrapping patch

2011-10-05 Thread Bernd Schmidt
On 10/05/11 00:29, Richard Henderson wrote: As a followup, I think this option needs to be disabled for profiling and profile_after_prologue. Should be a mere matter of frobbing the options at startup. The other code seems to test crtl-profile rather than an option flag, so how's this?

[PATCH, i386]: A couple of cleanups.

2011-10-05 Thread Uros Bizjak
Hello! No functional change. 2011-10-05 Uros Bizjak ubiz...@gmail.com * config/i386/i386.c (distance_non_agu_define): Simplify calculation of found. Simplify return value calculation. (distance_agu_use): Ditto. Tested on x86_64-pc-linux-gnu {,-m32} with

Re: [PATCH] Fix PR46556 (poor address generation)

2011-10-05 Thread William J. Schmidt
On Wed, 2011-10-05 at 18:21 +0200, Paolo Bonzini wrote: On 10/05/2011 06:13 PM, William J. Schmidt wrote: One other general question about the pattern-match transformation: Is this an appropriate transformation for all targets, or should it be somehow gated on available addressing modes on

[PATCH, testsuite]: Fix UNRESOLVED: gcc.dg/vect/vec-scal-opt*.c scan-tree-dump-times veclower

2011-10-05 Thread Uros Bizjak
Hello! 2011-10-05 Uros Bizjak ubiz...@gmail.com * gcc.dg/vect/vect.exp (VEC_CFLAGS): Move initialization after DEFAULT_VECTFLAGS initialization. Tested on x86_64-pc-linux-gnu {,-m32}, committed to mainline SVN. Uros. Index: gcc.dg/vect/vect.exp

[PATCH 0/3] Fix vector shuffle problems

2011-10-05 Thread Richard Henderson
The first problem is that the generic lowering to scalar implementation didn't match the documentation in that the shuffle indicies are to be masked to the range of the inputs. Or, perhaps more exactly, the generic lowering didn't match the SSE implementation which does do the masking. The OpenCL

[PATCH 1/3] Fix lower_vec_shuffle.

2011-10-05 Thread Richard Henderson
1: It can never fail. 2: It should mask the input indicies. --- gcc/ChangeLog |8 ++ gcc/tree-vect-generic.c | 276 +-- 2 files changed, 107 insertions(+), 177 deletions(-) diff --git a/gcc/ChangeLog b/gcc/ChangeLog index

[PATCH 3/3] Fix vect-shuffle-* test cases.

2011-10-05 Thread Richard Henderson
--- gcc/testsuite/ChangeLog| 11 ++ .../gcc.c-torture/execute/vect-shuffle-1.c | 98 +++--- .../gcc.c-torture/execute/vect-shuffle-2.c | 96 +++-- .../gcc.c-torture/execute/vect-shuffle-3.c | 90 ++--

Re: Initial shrink-wrapping patch

2011-10-05 Thread Richard Henderson
On 10/05/2011 10:19 AM, Bernd Schmidt wrote: * function.c (thread_prologue_and_epilogue_insns): Don't shrink-wrap if profiling after the prologue. Ok. r~

[PATCH] Fix memory leak in vect_pattern_recog_1

2011-10-05 Thread Jakub Jelinek
Hi! If vect_recog_func fails (or the other spot where vect_pattern_recog_1 returns early), the vector allocated in the function isn't freed, leading to memory leak. But, more importantly, doing a VEC_alloc + VEC_free num_stmts_in_loop * NUM_PATTERNS times seems to be completely unnecessary, the

Re: [PATCH, testsuite, i386] FMA3 testcases + typo fix in MD

2011-10-05 Thread Uros Bizjak
On Wed, Oct 5, 2011 at 12:49 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote: We're prepared and bunch of tests which checks autogeneration of FMA3 instructions family. FMA3 typo in .md file is fixed as well (it was catched by tests). ChangeLog entry: 2011-10-04  Kirill Yukhin

Re: Modify gcc for use with gdb (issue5132047)

2011-10-05 Thread Mike Stump
On Oct 5, 2011, at 6:18 AM, Diego Novillo wrote: I think we need to find a solution for this situation. The solution Apple found and implemented is a __nodebug__ attribute, as can be seen in Apple's gcc. We use it like so: #define __always_inline__ __always_inline__, __nodebug__ #undef

Re: Modify gcc for use with gdb (issue5132047)

2011-10-05 Thread Diego Novillo
On Wed, Oct 5, 2011 at 14:20, Mike Stump mikest...@comcast.net wrote: On Oct 5, 2011, at 6:18 AM, Diego Novillo wrote: I think we need to find a solution for this situation. The solution Apple found and implemented is a __nodebug__ attribute, as can be seen in Apple's gcc. We use it like

Re: [PATCH] Fix PR46556 (poor address generation)

2011-10-05 Thread Paolo Bonzini
On 10/05/2011 07:22 PM, William J. Schmidt wrote: I don't know off the top of my head -- I'll have to gather that information. The issue is that the profitability is really context-sensitive, so just the isolated costs of insns aren't enough. The forward propagation of the add into (mem (reg

Re: [PATCH, testsuite]: Fix UNRESOLVED: gcc.dg/vect/vec-scal-opt*.c scan-tree-dump-times veclower

2011-10-05 Thread Uros Bizjak
On Wed, Oct 5, 2011 at 7:42 PM, Uros Bizjak ubiz...@gmail.com wrote: 2011-10-05  Uros Bizjak  ubiz...@gmail.com        * gcc.dg/vect/vect.exp (VEC_CFLAGS): Move initialization after        DEFAULT_VECTFLAGS initialization. Actually, these testcases need a bit more surgery... 2011-10-05

Fix htab lookup bug in reregister_specialization (issue5190046)

2011-10-05 Thread Diego Novillo
I found this bug while debugging a failure in pph images with template classes. When writing the decl_specializations table, the writer calls htab_elements() to output the length of the table. It then traverses the table with htab_traverse_noresize() to emit all the entries. The reader uses

[pph] Fix template class tables streaming (issue5199041)

2011-10-05 Thread Diego Novillo
This fixes the problem I described in http://gcc.gnu.org/ml/gcc-patches/2011-10/msg00376.html The tests are now failing assembly comparisons because the elements in these tables are emitted in different sequence in the pph and non-pph case. Other than that, the assembly produced is identical.

Re: Fix htab lookup bug in reregister_specialization (issue5190046)

2011-10-05 Thread Jakub Jelinek
On Wed, Oct 05, 2011 at 03:49:38PM -0400, Diego Novillo wrote: Jason, I don't think this is affecting any bugs in trunk, but it looks worth fixing. OK for trunk? Well, it can cause problems. Consider 3 entries in the hash table with the same hash. (1) inserted first, then (2), then (3), then

Re: [PATCH] Fix PR46556 (poor address generation)

2011-10-05 Thread William J. Schmidt
On Wed, 2011-10-05 at 21:01 +0200, Paolo Bonzini wrote: On 10/05/2011 07:22 PM, William J. Schmidt wrote: I don't know off the top of my head -- I'll have to gather that information. The issue is that the profitability is really context-sensitive, so just the isolated costs of insns aren't

Find more shrink-wrapping opportunities

2011-10-05 Thread Bernd Schmidt
This adds a little mini-pass to shrink-wrapping, to eliminate a common case that often makes shrink-wrapping unavailable. If a move insn copies an argument registers to a call-saved register, the prologue must be emitted before this insn. We should therefore try to delay such moves for as long as

Re: [PATCH, PR50527] Don't assume alignment of vla-related allocas.

2011-10-05 Thread Tom de Vries
On 10/05/2011 10:46 AM, Richard Guenther wrote: On Tue, Oct 4, 2011 at 6:28 PM, Tom de Vries tom_devr...@mentor.com wrote: On 10/04/2011 03:03 PM, Richard Guenther wrote: On Tue, Oct 4, 2011 at 9:43 AM, Tom de Vries tom_devr...@mentor.com wrote: On 10/01/2011 05:46 PM, Tom de Vries wrote: On

Re: Fix htab lookup bug in reregister_specialization (issue5190046)

2011-10-05 Thread Jakub Jelinek
On Wed, Oct 05, 2011 at 05:03:45PM -0400, Jason Merrill wrote: On 10/05/2011 04:05 PM, Jakub Jelinek wrote: BTW, register_specialization has the same problems. If slot != NULL and fn == NULL, it can still return without storing non-NULL into *slot (when optimize_specialization_lookup_p (tmpl)

Re: Find more shrink-wrapping opportunities

2011-10-05 Thread Steven Bosscher
On Wed, Oct 5, 2011 at 10:48 PM, Bernd Schmidt ber...@codesourcery.com wrote: Bootstrapped and tested on i686-linux. Ok? +/* Return true if BB has any active insns. */ +static bool +bb_active_p (basic_block bb) +{ + rtx label; + + /* Test whether there are active instructions in the

[PATCH] Remove OUTPUT_ADDR_CONST_EXTRA macro

2011-10-05 Thread Anatoly Sokolov
Hi. No one back end does not use OUTPUT_ADDR_CONST_EXTRA macro now, this patch remove it. The TARGET_ASM_OUTPUT_ADDR_CONST_EXTRA target hook should be use instead. The patch has been bootstrapped on and regression tested on x86_64-unknown-linux-gnu for c and c++. This patch is

Re: Initial shrink-wrapping patch

2011-10-05 Thread Ian Lance Taylor
On Wed, Oct 5, 2011 at 10:17 AM, Bernd Schmidt ber...@codesourcery.com wrote: I've committed the following after a x86_64-linux bootstrap. This patch appears to have broken the Go bootstrap when compiling a C file in libgo.  Here is a reduced test case: static struct { int n; unsigned int

Re: [PATCH] Remove OUTPUT_ADDR_CONST_EXTRA macro

2011-10-05 Thread Richard Henderson
On 10/05/2011 02:28 PM, Anatoly Sokolov wrote: * system.h (OUTPUT_ADDR_CONST_EXTRA): Poison. * doc/tm.texi.in (OUTPUT_ADDR_CONST_EXTRA): Remove documentation. * doc/tm.texi: Regenerate. * target.def (output_addr_const_extra): Use

Re: Commit: RX: Codegen bug fixes

2011-10-05 Thread Richard Henderson
On 10/05/2011 03:23 AM, Nick Clifton wrote: The final fix was pointed out by Richard Henderson. The recently added support for narrow mode min and max instructions did not work for the SMAX insn, as the RX does not have narrow mode versions of this insn. The SMIN pattern has the same

Re: [v3] versioned-namespaces spelling/soname change

2011-10-05 Thread Benjamin Kosnik
I'm going to let this chill a bit on mainline and then check in to 4.6.x. Seems fine so I dropped this into the 4_6-branch tested x86/linux -benjamin 2011-10-05 Benjamin Kosnik b...@redhat.com Jonathan Wakely jwakely@gmail.com PR libstdc++/48698 *

Re: Initial shrink-wrapping patch

2011-10-05 Thread Bernd Schmidt
On 10/06/11 01:04, Ian Lance Taylor wrote: On Wed, Oct 5, 2011 at 10:17 AM, Bernd Schmidt ber...@codesourcery.com wrote: I've committed the following after a x86_64-linux bootstrap. This patch appears to have broken the Go bootstrap when compiling a C file in libgo. Here is a reduced

Re: [v3] versioned-namespaces spelling/soname change

2011-10-05 Thread Paolo Carlini
Hi, the below hunk seems spurious?!? Paolo. / Index: include/bits/locale_facets.tcc === --- include/bits/locale_facets.tcc (revision 179579) +++ include/bits/locale_facets.tcc (working copy) @@ -635,15

[v3] Avoid pod_char_traits.h warnings in C++0x mode

2011-10-05 Thread Paolo Carlini
Hi, committed to mainline. Paolo. 2011-10-05 Paolo Carlini paolo.carl...@oracle.com * include/ext/pod_char_traits.h: Avoid warnings in C++0x mode when int_type is unsigned. Index: include/ext/pod_char_traits.h

Re: [v3] add max_size and rebind to __alloc_traits

2011-10-05 Thread Paolo Carlini
Hi, 2011-10-04 Jonathan Wakelyjwakely@gmail.com * include/ext/alloc_traits.h (__alloc_traits::max_size): Define. (__alloc_traits::rebind): Define. * include/bits/stl_vector.h: Use them. * testsuite/util/testsuite_allocator.h (SimpleAllocator): Define.

Re: Fix pr50607 bconstp-3.c failure

2011-10-05 Thread Artem Shinkarov
On Wed, Oct 5, 2011 at 5:32 PM, Artem Shinkarov artyom.shinkar...@gmail.com wrote: On Wed, Oct 5, 2011 at 5:28 PM, Joseph S. Myers jos...@codesourcery.com wrote: On Wed, 5 Oct 2011, Artem Shinkarov wrote: Joseph, is it possible to commit the patch together with the space fixes? You should

Re: Fix pr50607 bconstp-3.c failure

2011-10-05 Thread Hans-Peter Nilsson
On Thu, 6 Oct 2011, Artem Shinkarov wrote: Successfully regtested on x86-unknown-linux-gnu. Committed to the mainline with the revision 179588. ChangeLog: 2011-10-06 Artjoms Sinkarovs artyom.shinkar...@gmail.com * c-tree.h (c_expr_t): New typedef for struct c_expr.

Re: Fix pr50607 bconstp-3.c failure

2011-10-05 Thread Artem Shinkarov
On Thu, Oct 6, 2011 at 3:27 AM, Hans-Peter Nilsson h...@bitrange.com wrote: On Thu, 6 Oct 2011, Artem Shinkarov wrote: Successfully regtested on x86-unknown-linux-gnu. Committed to the mainline with the revision 179588. ChangeLog: 2011-10-06  Artjoms Sinkarovs  artyom.shinkar...@gmail.com  

Merge to gccgo branch

2011-10-05 Thread Ian Lance Taylor
I merged mainlined revision 179565 to the gccgo branch. I included a patch disabling -fshrink-wrap if -fstack-prologue is enabled. Ian

Re: Initial shrink-wrapping patch

2011-10-05 Thread Ian Lance Taylor
Bernd Schmidt ber...@codesourcery.com writes: This appears to be because the split prologue contains a jump, which means the find_many_sub_blocks call reorders the block numbers, and our indices into bb_flags are off. I'm testing the following patch, but it won't finish today - feel free to

Re: Fix pr50607 bconstp-3.c failure

2011-10-05 Thread Hans-Peter Nilsson
On Thu, 6 Oct 2011, Artem Shinkarov wrote: On Thu, Oct 6, 2011 at 3:27 AM, Hans-Peter Nilsson h...@bitrange.com wrote: Write that changelog entry as:        PR middle-end/50607        * c-tree.h (c_expr_t): New typedef for struct c_expr.        (C_EXPR_APPEND): New macro.        *