[PATCH] Fix SLP vectorization of shifts (PR tree-optimization/48616)

2011-04-16 Thread Jakub Jelinek
Hi! As the attached testcase shows, while the current detection of what shifts are by scalar and what shifts are by vector shift count may work well for loop vectorizer (vect_internal_def being vector shift, vect_external_def or vect_constant_def scalar shift), it is incorrect for SLP, where

Re: [Patch, Fortran] PR 48624 - fix DECL for external procedures with proc arguments

2011-04-16 Thread Tobias Burnus
Steve Kargl wrote: The comment comes from r170414, which suggest there is a problem if a procedure is in an argument list, and that procedure gets in-lined. [...] Perhaps, looking at the -fdump-tree-original on whole_file_32.f90 may shed light on the situation. Well, it does not. The

[PATCH 9/n, i386]: Merge SSE and AVX patterns using enable attribute.

2011-04-16 Thread Uros Bizjak
Hello! Attached patch converts move patterns. 2011-04-15 Uros Bizjak ubiz...@gmail.com * config/i386/sse.md (V16): New mode iterator. (VI1, VI8): Ditto. (AVXMODEQI, AVXMODEDI): Remove. (sse2, sse3): New mode attribute. (movmode): Use V16 mode iterator.

Re: [Patch, Fortran] PR 48624 - fix DECL for external procedures with proc arguments

2011-04-16 Thread Richard Guenther
On Sat, Apr 16, 2011 at 8:34 AM, Tobias Burnus bur...@net-b.de wrote: Steve Kargl wrote: The comment comes from r170414, which suggest there is a problem if a procedure is in an argument list, and that procedure gets in-lined. [...] Perhaps, looking at the -fdump-tree-original on

Re: Remember/restore ALLOCA_FOR_VAR_P over tuples

2011-04-16 Thread Eric Botcazou
r172516, for reference also below. I later saw that my patch causes cxg2001 to fail. I've analyzed it enough to be sure that it's only exposed by this patch (due to inlining now happening), in fact it's a problem in IRA that is reproducible with a C++ testcase even without the patch. I've

Re: [PATCH] doubled words

2011-04-16 Thread Gerald Pfeifer
On Fri, 15 Apr 2011, Mike Stump wrote: I think these are obvious. Which means that you can commit them without getting explicit approval, Jim. Gerald

[PATCH, i386]: Macroize and simplify vector integer pack/unpack patterns.

2011-04-16 Thread Uros Bizjak
Hello! 2011-04-16 Uros Bizjak ubiz...@gmail.com * config/i386/sse.md (sseunpackmode): New mode attribute. (ssepackmode): Ditto. (vec_pack_trunc_mode): Macroize expander from vec_pack_trunc_{v8hi,v4si,v2di} using VI248_128 mode iterator.

Re: [committed] PA predicate cleanup

2011-04-16 Thread John David Anglin
On Mon, 28 Mar 2011, Richard Henderson wrote: (define_predicate ior_operand (and (match_code const_int) (match_test ior_mask_p (INTVAL (op) (define_predicate reg_or_ior_operand (ior (match_operand 0 register_operand) (match_operand 0 ior_operand))) You've

New German PO file for 'gcc' (version 4.6.0)

2011-04-16 Thread Translation Project Robot
Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'gcc' has been submitted by the German team of translators. The file is available at: http://translationproject.org/latest/gcc/de.po (This file, 'gcc-4.6.0.de.po', has just

Re: [pph] Macro Validation Correction (issue4425041)

2011-04-16 Thread Diego Novillo
On Thu, Apr 14, 2011 at 22:01, Lawrence Crowl cr...@google.com wrote: Unfortunately, five bits are not sufficient for the alternate use of cpp_hashnode.directive_index as a named operator index.  So, I have reverted the number of bits from five back to seven.  As a result, we now have 34 bits

Re: [PATCH] doubled words

2011-04-16 Thread Mike Stump
On Apr 16, 2011, at 4:04 AM, Gerald Pfeifer ger...@pfeifer.com wrote: On Fri, 15 Apr 2011, Mike Stump wrote: I think these are obvious. Which means that you can commit them without getting explicit approval Well, technically, it means nothing... It only means something if the maintainer

New German PO file for 'gcc' (version 4.6.0)

2011-04-16 Thread Translation Project Robot
Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'gcc' has been submitted by the German team of translators. The file is available at: http://translationproject.org/latest/gcc/de.po (This file, 'gcc-4.6.0.de.po', has just

[google] Install additional gcov files (issue4442052)

2011-04-16 Thread Diego Novillo
I'm committing this patch from Rong Xu. These added files are used for FDO kernel builds. Rong, could you please provide more details? Will you be submitting this patch to trunk? Tested on x86_64. Committed to google/main branch. Thanks. Diego. 2011-04-15 Rong Xu x...@google.com

Re: [google] Initialize fields in default ctor for _Hashtable_iterator (issue4430051)

2011-04-16 Thread Diego Novillo
Geez, working on too many different branches. This is *not* a pph patch. On Sat, Apr 16, 2011 at 16:30, Diego Novillo dnovi...@google.com wrote: I am committing this patch for David Li on google/main. The patch adds explicit initialization to the fields of _Hashtable_iterator in its default

Fix gengtype-state string hashtable

2011-04-16 Thread Nicola Pero
While reading GCC code, I noticed that in gengtype-state.c the equality function in a string hashtable is set to strcmp. But that returns 0 (ie, false for hashtable.c) when the strings are equal! I can't see how that hashtable would ever work. Do we have any tests for gengtype-state ? Am I

Re: [google] Handle NULL return values in setlocale calls (issue4444046)

2011-04-16 Thread Jing Yu
Thanks Diego for committing this patch. Yes, I would like to submit the patch to trunk. The reason is for this patch is that setlocale() in bionicC always returns NULL (bionicC does not support setlocale()), however libstdc++ does not handle the NULL return value of setlocale(xxx,NULL) and thus

Fix PR48622 (lto ICE, lto bootstrap)

2011-04-16 Thread Michael Matz
Hi, since r172430 lto bootstrap is broken, as well as the attached testcase (pr48622) and cpu2006 compilation (pr48645). The inline summary writer used a different order for size and time than the reader expected. I've committed the below patch as obvious (r172603) after verifying that lto