Fix PR48622 (lto ICE, lto bootstrap)

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 bo

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

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 ca

Fix gengtype-state string hashtable

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 miss

Re: More of ipa-inline housekeeping

On Sat, Apr 16, 2011 at 3:42 PM, H.J. Lu wrote: > On Wed, Apr 13, 2011 at 3:20 PM, Jan Hubicka wrote: >> Hi, >> this patch moves inline_summary from field in cgraph_node into its own on >> side >> datastructure. This moves it from arcane decision of mine to split all IPA >> data >> into global/

Re: More of ipa-inline housekeeping

On Wed, Apr 13, 2011 at 3:20 PM, Jan Hubicka wrote: > Hi, > this patch moves inline_summary from field in cgraph_node into its own on side > datastructure. This moves it from arcane decision of mine to split all IPA > data > into global/local datas stored in common datastructure into the scheme w

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

Am 16.04.2011 08:34, schrieb Tobias Burnus: Thus: OK for the trunk? OK. Thanks for the patch (which, incidentally, also fixes PR 48644). Thomas

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

Geez, working on too many different branches. This is *not* a pph patch. On Sat, Apr 16, 2011 at 16:30, Diego Novillo 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 constructor.  Da

[pph] Initialize fields in default ctor for _Hashtable_iterator (issue4430051)

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 constructor. David, could you describe why this change was needed? Will you be submitting for trunk? Tested on x86_64. Committed to google/main.

[google] Install cpu_defines.h (issue4440044)

I am committing this patch from Doug Kwan on google/main. This patch adds cpu_defines.h to the set of files to be installed. Doug, could you describe why we need to do this? Will you be submitting this patch for trunk? Tested on x86_64. Committed on google/main. Diego. 2011-03-15 Doug Kwan

[Patch, Fortran] PR 18918 - implement coarray's IMAGE_INDEX for nonconstant bounds

The attached patch implements IMAGE_INDEX(COARRAY, SUB) for nonconstant bounds; the patch (and the test case) work both with -fcoarray=single and with -fcoarray=lib (with any number of images - I tried up to 21). Note: If the image index would exceed the number of images, 0 is returned. Additi

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

I'm committing this patch for Jing Yu on google/main. The patch handles NULL values returned from setlocale. Jing, could you please describe why this was needed? Is this a patch that you will want to submit for trunk? Tested on x86_64. Committed to google/main. Diego. 2011-04-15 Jing Yu

[google] Move ChangeLog entries to the right file (issue4427046)

This patch moves some ChangeLog entries from ChangeLog into ChangeLog.google-main. This prevents merge conflicts. Committed to google/main. Diego. diff --git a/gcc/ChangeLog b/gcc/ChangeLog index eddbc67..086d1ba 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,22 +1,3 @@ -2011-04-11 Xinl

[google] Install additional gcov files (issue4442052)

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 * Makefile.

Fix PR other/48639

Merge glitch apparently. Tested on i586-suse-linux, applied on the 4.4 branch as obvious. 2011-04-16 Eric Botcazou PR other/48639 * tests/base/pthread.h [IRIX_PTHREAD_INIT_CHECK]: Move around. -- Eric Botcazou Index: tests/base/pthread.h ===

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

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: [PATCH] doubled words

On Apr 16, 2011, at 4:04 AM, Gerald Pfeifer 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 agrees with me,

Re: [pph] Clean positive tests. (issue4423044)

On 2011/04/15 18:22:19, Lawrence Crowl wrote: 2011-04-15 Lawrence Crowl * lib/dg-pph.exp (dg-pph-pos): Stop on first failure. Change names of assembly files to be clear on which is with and without PPH. OK. Diego. http://codereview.appspot.c

Re: [pph] Macro Validation Correction (issue4425041)

On Thu, Apr 14, 2011 at 22:01, Lawrence Crowl 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 > in small fie

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

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: [committed] PA predicate cleanup

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")))

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

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

Re: [PATCH] doubled words

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

Some proofreading in lto.c

Various nits and typos spotted while studying WPA, plus one redundant test. Tested on i586-suse-linux, applied on the mainline as obvious. 2011-04-16 Eric Botcazou * lto.c (lto_balanced_map): Fix typos in head comment. (lto_promote_cross_file_statics): Fix long lines and remo

Re: Remember/restore ALLOCA_FOR_VAR_P over tuples

> 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

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

On Sat, Apr 16, 2011 at 8:34 AM, Tobias Burnus 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 whole_file_32.f9

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

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