Re: [PR50764, PATCH] Fix for ICE in maybe_record_trace_start with -fsched2-use-superblocks

2011-10-30 Thread Tom de Vries
On 10/29/2011 09:17 PM, Tom de Vries wrote: Richard, I have a tentative fix for PR50764. In the example from the test-case, -fsched2-use-superblocks moves an insn from block 4 to block 3. 2 bar | ---+- / \ *

[PATCH] Properly limit backwards label scanning in reorg.

2011-10-30 Thread David Miller
My check-gcc runs are completely dominated by the time it takes to test compile.exp=20001226-1.c On sparc when compiling this function at -O1 the lowest hanging fruit is some sillyness in reorg. It is trying to test a very specific condition within a well contained code block, but the label

Re: [Patch, fortran] [06/66] inline sum and product: Prepare gfc_trans_preloop_setup

2011-10-30 Thread Paul Richard Thomas
Dear Mikael, I intend to work my way through your patches, starting this evening. I'll give you feedback as I go along and will OK the whole package, rather than bits. Is that alright with you? Cheers Paul On Fri, Oct 28, 2011 at 1:29 AM, Mikael Morin mikael.mo...@sfr.fr wrote: -- The

[PR50878, PATCH] Fix for verify_dominators in -ftree-tail-merge

2011-10-30 Thread Tom de Vries
Richard, I have a fix for PR50878. A simplified form of the problem from the test-case of the PR is shown in this cfg. Block 12 has as direct dominator block 5. 5 / \ / \ * * 6 7 | | | | * * 8 9 \ / \ /

Re: [PR50878, PATCH] Fix for verify_dominators in -ftree-tail-merge

2011-10-30 Thread Tom de Vries
On 10/30/2011 09:20 AM, Tom de Vries wrote: Richard, I have a fix for PR50878. Sorry, with patch this time. Thanks, - Tom A simplified form of the problem from the test-case of the PR is shown in this cfg. Block 12 has as direct dominator block 5. 5 / \ / \

Re: [PATCH] Pattern recognize shifts with different rhs1/rhs2 types

2011-10-30 Thread Ira Rosen
On 28 October 2011 20:44, Jakub Jelinek ja...@redhat.com wrote: Hi! Hi, This patch implements what I've talked about, with this we can now with -mavx2 as well as -mxop vectorize long long/unsigned long long shifts by int or long long/unsigned long long shifts by long long (where the FE

Re: [PATCH] Fix early inliner inlining uninlinable functions

2011-10-30 Thread Richard Guenther
On Sat, Oct 29, 2011 at 2:36 AM, Maxim Kuvyrkov ma...@codesourcery.com wrote: On 29/10/2011, at 1:57 AM, Richard Guenther wrote: We fail to keep the cannot-inline flag up-to-date when turning indirect to direct calls.  The following patch arranges to do this during statement folding (which

Re: [PATCH] Properly limit backwards label scanning in reorg.

2011-10-30 Thread Richard Guenther
On Sun, Oct 30, 2011 at 8:49 AM, David Miller da...@davemloft.net wrote: My check-gcc runs are completely dominated by the time it takes to test compile.exp=20001226-1.c On sparc when compiling this function at -O1 the lowest hanging fruit is some sillyness in reorg.  It is trying to test a

Re: [PR50878, PATCH] Fix for verify_dominators in -ftree-tail-merge

2011-10-30 Thread Richard Guenther
On Sun, Oct 30, 2011 at 9:27 AM, Tom de Vries tom_devr...@mentor.com wrote: On 10/30/2011 09:20 AM, Tom de Vries wrote: Richard, I have a fix for PR50878. Sorry, with patch this time. Ok for now, but see Davids mail and the complexity issue with iteratively updating dominators. It seems to

New Japanese PO file for 'gcc' (version 4.6.1)

2011-10-30 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 Japanese team of translators. The file is available at: http://translationproject.org/latest/gcc/ja.po (This file, 'gcc-4.6.1.ja.po', has

Re: PING: [C++-11 PATCH] Trailing comma in enum

2011-10-30 Thread Magnus Fromreide
On Sat, 2011-10-29 at 20:48 +0300, Ville Voutilainen wrote: Could someone please review this? + if (cxx_dialect cxx0x !in_system_header) + pedwarn (input_location, OPT_pedantic, + comma at end of enumerator list); Why not use maybe_warn_cpp0x there?

Re: [PATCH, committed] Update wwwdocs for PR target/47272 fixes (altivec vec_ld/vec_st builtins)

2011-10-30 Thread Gerald Pfeifer
On Fri, 4 Feb 2011, Michael Meissner wrote: After updating the mainline and 4.5 trees with the fix for PR target/47272 that restores vec_ld and vec_st to generate only Altivec memory instructions, I added the following to the documentation: Here are some minor editorial changes I applied on

Re: building binutils from same directory as gcc

2011-10-30 Thread Gerald Pfeifer
I see this was not applied, and also did not get any review comments. Let me add our build system maintainers. If something like this goes in, I suggest to avoid direct references to SVN (which is a technical detail) and use something more generic and spell GCC as opposed to gcc when it refers

[C++ patch] Use GNU/Linux in NEWS

2011-10-30 Thread Gerald Pfeifer
Installed based on a request by RMS. Gerald 2011-10-25 Gerald Pfeifer ger...@pfeifer.com * NEWS (GCC 2.95): Refer to GNU/Linux instead of Linux. (EGCS 1.0): Ditto. Index: NEWS === --- NEWS(revision

Re: [dwarf2out, elfos] Output assembly faster

2011-10-30 Thread Dimitrios Apostolou
Hello list, I updated this patch to latest trunk and also incorporated apinski's suggestion to use stpcpy() instead of a custom loop. Bootstrapped and tested again on i386, no regressions. The comments from my previous email still apply, I never got a reply: On Mon, 22 Aug 2011, Dimitrios

[committed] Avoid common in some tests on hppa*-*-hpux*

2011-10-30 Thread John David Anglin
Add -fno-common to some more tests on hppa*-*-hpux* to avoid limited alignment capabilities. Tested on hppa2.0w-hp-hpux11.11. Committed to trunk. Dave -- J. David Anglin dave.ang...@nrc-cnrc.gc.ca National Research Council of Canada (613) 990-0752

[PATCH, i386]: Merge and macroize some more patterns

2011-10-30 Thread Uros Bizjak
Hello! 2011-10-30 Uros Bizjak ubiz...@gmail.com * config/i386/i386.md (avx2_vec_dupmode): Macroize insn from avx2_vec_dup{v8sf,v4sf} using VF1 mode iterator. (vec_dupv4sf): Remove expander. (vec_dupv4sf): Merge from *vec_dupv4sf and *vec_dupv4sf_avx.

[libstdc++, doc] Update prerequisites documentation

2011-10-30 Thread Gerald Pfeifer
At first this was only triggered by Linux vs GNU/Linux, but then I noticed GCC being lower-cased and the Red Hat Linux reference when we actually do not support that old version of glibc anymore at all. Committed to mainline. Gerald 2011-10-30 Gerald Pfeifer ger...@pfeifer.com *

Re: PING: [C++-11 PATCH] Trailing comma in enum

2011-10-30 Thread Ville Voutilainen
On 30 October 2011 12:17, Magnus Fromreide ma...@lysator.liu.se wrote: Attached is a variation where maybe_warn_cpp0x is used. Please note how my new pedwarn breaks the pattern in maybe_warn_cpp0x. Looks reasonable to me, but I'm not a maintainer. :) Jason?

[PING] New port for TILEPro and TILE-Gx

2011-10-30 Thread Walter Lee
Ping? I believe I have addressed all the reviewer's (namely Joseph Myers') comments to date. Thanks, Walter Lee

[PATCH] Straight-line strength reduction, stage 1

2011-10-30 Thread William J. Schmidt
Greetings, IVOPTS handles strength reduction of induction variables, but GCC does not currently perform strength reduction in straight-line code. This has been noted before in PR22586 and PR35308. PR46556 is also a case that could be handled with strength reduction. This patch adds a pass to

Re: [libstdc++, doc] Update prerequisites documentation

2011-10-30 Thread Jonathan Wakely
On 30 October 2011 15:36, Gerald Pfeifer wrote: At first this was only triggered by Linux vs GNU/Linux, but then I noticed GCC being lower-cased and the Red Hat Linux reference when we actually do not support that old version of glibc anymore at all. Are those instructions not valid for Fedora

Re: [PATCH][2/n] LTO option handling/merging rewrite

2011-10-30 Thread Joseph S. Myers
On Sat, 29 Oct 2011, Richard Guenther wrote: I tried to look at what we get for -fno-pic vs. -fpic and -fno-pic is completely dropped from the decoded options list (not sure what happens on targets with -fpic as default). So it seems at most one state (the non-default one) survives here.

Re: [C++-11] User defined literals

2011-10-30 Thread Ed Smith-Rowland
On 10/27/2011 03:47 PM, Jason Merrill wrote: On 10/27/2011 03:32 PM, Ed Smith-Rowland wrote: + if (TREE_CODE (TREE_TYPE (decl)) != LANG_TYPE) +argtypes = TYPE_ARG_TYPES (TREE_TYPE (decl)); If you have multiple overloaded operator _foo, you need to iterate over them looking for the one

Re: [libstdc++, doc] Update prerequisites documentation

2011-10-30 Thread Gerald Pfeifer
On Sun, 30 Oct 2011, Jonathan Wakely wrote: Are those instructions not valid for Fedora though? And RHEL? Anything using rpm -e --nodeps is not valid anywhere, I'd claim. And rpm -i --define _install_langs all glibc-common-2.2.5-34.i386.rpm definitely won't work on anything released in the

[C++ preview patch] PR 44277

2011-10-30 Thread Paolo Carlini
Hi, thus I'm working on a new warning for 0 as null ptr constant, which should be useful for people moving to C++11' nullptr. Looks like a good place to check for that is cp_convert_to_pointer, then the difficulty is coping correctly with things like: int* p; if (p) ; where

Re: [libstdc++, doc] Update prerequisites documentation

2011-10-30 Thread Jonathan Wakely
On 30 October 2011 17:16, Gerald Pfeifer wrote: I hope this addresses your questions? It certainly does - thanks.

Re: addition to http://gcc.gnu.org/faq.html

2011-10-30 Thread Gerald Pfeifer
On Sun, 17 Apr 2011, Axel Freyn wrote: I propose to add a phrase to the rpath-section (Dynamic linker is unable to find GCC libraries), which explains how to use -R as compiler-option (by using -Wl,) This is a good idea, thank you. I applied the small variation of your original patch below

Re: [Patch, fortran] [06/66] inline sum and product: Prepare gfc_trans_preloop_setup

2011-10-30 Thread Mikael Morin
On Sunday 30 October 2011 09:16:09 Paul Richard Thomas wrote: Dear Mikael, I intend to work my way through your patches, starting this evening. I'll give you feedback as I go along and will OK the whole package, rather than bits. Is that alright with you? Sure. Mikael

[wwwdocs] Simplify FAQ around rpath

2011-10-30 Thread Gerald Pfeifer
While looking into Axel Freyn's suggestion, I noticed how that answer really is too long winded. This cuts down things quite a bit, making this more digestable hopefully. Applied. Gerald Index: faq.html === RCS file:

Re: [patch, Fortran] Fix PR 50690

2011-10-30 Thread Mikael Morin
On Tuesday 25 October 2011 07:39:44 Thomas Koenig wrote: Ping ** 0.571428 Let's keep Jakub CC-ed for mixes of OpenMP and frontend optimizations. ;-) There are two commented lines in the testcase. Is it expected? Otherwise doesn't look too bad... Mikael Jakub Jelinek wrote: Though, what

[PATCH] Fix AIX breakage from gcc-ar/nm/ranlib wrappers

2011-10-30 Thread David Edelsohn
This patch breaks bootstrap on AIX because it includes stdio.h before config.h. config.h header can affect system headers. I am not sure why stdio.h should be included because system.h itself includes stdio.h. The appended patch moves stdio.h after config.h, but maybe gcc-ar.h should not

Re: [4.5.1,PR42776] Backport LTO-COFF implementation.

2011-10-30 Thread Gerald Pfeifer
On Mon, 14 Jun 2010, Dave Korn wrote: * htdocs/gcc-4.5/changes.html: Remove explicit mentions of ELF format from LTO documentation. Ouch, I noticed this patch of yours never was applied, perhaps some misunderstanding between Richi and you (him assuming you'd go ahead and you waiting

Re: [PATCH][2/n] LTO option handling/merging rewrite

2011-10-30 Thread Richard Guenther
On Fri, Oct 28, 2011 at 4:48 PM, Diego Novillo dnovi...@google.com wrote: Isn't this the same patch as http://gcc.gnu.org/ml/gcc-patches/2011-10/msg02348.html? No, it's updated to do exactly what we do now. Changes to behavior to be split to 3/n (like warning about mismatches or fixing the way

Re: [PATCH] Fix AIX breakage from gcc-ar/nm/ranlib wrappers

2011-10-30 Thread Richard Guenther
On Sun, Oct 30, 2011 at 9:39 PM, David Edelsohn dje@gmail.com wrote: This patch breaks bootstrap on AIX because it includes stdio.h before config.h.  config.h header can affect system headers.  I am not sure why stdio.h should be included because system.h itself includes stdio.h.  The

Re: [PATCH] PR fortran/50573 -- Fix checking of dshift{lr} arguments

2011-10-30 Thread Steve Kargl
On Fri, Oct 28, 2011 at 07:43:58AM +0200, Tobias Burnus wrote: Am 28.10.2011 01:27, schrieb Steve Kargl: The attached patch tightens the checking of the arguments of the dshiftl and dshiftr argument. It also does a kind type conversion when either I or J is a BOZ to the kind type of the

Factor mangled id computation (issue5311075)

2011-10-30 Thread Diego Novillo
This factors the computation of mangled names so it can be called without actually setting the assembler name on the DECL. It is used in the pph branch for merging symbols that are replicated in multiple images. On trunk, it changes nothing, but it sets it up for when we are ready to merge the

[lto] Also stream TYPE_ADDR_SPACE. (issue5308067)

2011-10-30 Thread Diego Novillo
I found this on the PPH branch. We were not handling TYPE_ADDR_SPACE in the streamer. Fixed with this patch. Richi, I will apply it on trunk unless you remember some reason why we never streamed this? It does not seem to affect any LTO tests. Tested with lto profiledbootstrap on x86_64.

RFA: Enable multi-step transformation in peephole2

2011-10-30 Thread Joern Rennecke
Where there can be multiple things wrong with a pattern sequence, performance-wise, and multiple things might be fixed up, depending on factors like available registers, providing patterns for every combination leads to combinatorial explosion. It is simpler to describe the available

[COMMITTED,fortran] small whitespace patch

2011-10-30 Thread Steve Kargl
I've committed the following small whitespace patch that allows one to do 'grep -E ^gfc_free' to find functions starting with this pattern. 2011-10-30 Steven G. Kargl ka...@gcc.gnu.org * symbol.c (gfc_free_charlen): Whitespace. Index: symbol.c

Re: [patch, Fortran] Fix PR 50690

2011-10-30 Thread Tobias Burnus
Mikael Morin wrote: Let's keep Jakub CC-ed for mixes of OpenMP and frontend optimizations. ;-) There are two commented lines in the testcase. Is it expected? Otherwise doesn't look too bad... I had also a glance at the patch - and it looks reasonable; in particular, I failed to generate a

[PATCH] PR fortran/50404 -- CLOSE requires unit number

2011-10-30 Thread Steve Kargl
I've had this patch in my i386-*-freebsd and x86_64-*-freebsd for more than a week. Several 'gmake check-gfortran' have been run during that time without a regression. OK for trunk? 2011-10-30 Steven G. Kargl ka...@gcc.gnu.org PR fortran/50404 * io.c (gfc_resolve_close): A

[PATCH] Handle many consecutive location notes more efficiently in dwarf2.

2011-10-30 Thread David Miller
Tests such as limits-fndefn.c spend %97 of their time in next_real_insn() when -g is on the command line. It's the invocation from dwarf2out_var_location(). These test cases are basically 100,000 consecutive var location notes, followed by a real instruction. This is similar to the reorg issue

Re: [PATCH] PR fortran/50404 -- CLOSE requires unit number

2011-10-30 Thread Jerry DeLisle
On 10/30/2011 07:50 PM, Steve Kargl wrote: I've had this patch in my i386-*-freebsd and x86_64-*-freebsd for more than a week. Several 'gmake check-gfortran' have been run during that time without a regression. OK for trunk? OK, thanks for patch. Jerry

Re: [C++ preview patch] PR 44277

2011-10-30 Thread Paolo Carlini
... this is another option: use an 'integer_zero_or_nullptr_node (tree)' defined differently for C and C++. Certainly the whole thing becomes smaller. Paolo. Index: c-family/c.opt === --- c-family/c.opt

Re: Factor mangled id computation (issue5311075)

2011-10-30 Thread Jason Merrill
OK. Jason

Re: C++ PATCH to add -std=c++11 ??

2011-10-30 Thread Jason Merrill
Here's my start at adjusting things to use the C++11 name; feel free to run with it. Looking at it again, I think adding __GXX_EXPERIMENTAL_CXX11__ is a mistake, we should just set __cplusplus to the C++11 value. Jason commit 9c331b6de02e796fa25b04ed206b8570c83ee237 Author: Jason Merrill