Re: [GCC 4.8 wwwdocs] PATCH: Mention several user-visible changes for x86

2013-02-20 Thread Kirill Yukhin
Hi, Checked in Thanks, K On Mon, Feb 18, 2013 at 10:28 AM, Igor Zamyatin izamya...@gmail.com wrote: Gerald, Thanks a lot for your remarks! Below is updated patch which will be checked in. Thanks, Igor On Mon, Feb 18, 2013 at 3:07 AM, Gerald Pfeifer ger...@pfeifer.com wrote: On Fri,

Re: [PATCH, x86, AVX2] FP reassociation enabling for AVX2 targets

2013-02-20 Thread Kirill Yukhin
OK (it is a tuning patch). Hi, Checked in: http://gcc.gnu.org/ml/gcc-cvs/2013-02/msg00540.html Thanks, K

Re: Speedup recognizing multi-letter constraints

2013-02-20 Thread Richard Biener
On Tue, Feb 19, 2013 at 4:10 PM, Michael Matz m...@suse.de wrote: Hi, from IRC: [15:45:21] richi 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

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

2013-02-20 Thread Richard Biener
On Tue, 19 Feb 2013, Jakub Jelinek wrote: 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

Re: Version specific onlinedocs page

2013-02-20 Thread Tobias Burnus
Gerald Pfeifer wrote: On Mon, 18 Feb 2013, Tobias Burnus wrote: 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 you can add a note for the release manager how to

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

2013-02-20 Thread Jakub Jelinek
On Wed, Feb 20, 2013 at 10:52:52AM +0100, Richard Biener wrote: *** get_value (tree var) *** 295,301 { prop_value_t *val; ! if (const_val == NULL) return NULL; val = const_val[SSA_NAME_VERSION (var)]; --- 296,303 { prop_value_t *val;

[PATCH][RFC] Less TODO_remove_unused_locals

2013-02-20 Thread Richard Biener
Hunting for the we're getting slower bits I noticed that TODO_remove_unused_locals is a big part of execute_function_todo (and accounts for 1% of compile-time of ac.f90). The following patch removes most of the remove_unused_locals calls based on the fact that with anonymous SSA names now

Re: RFC: [PATCH,ARM] Fix 56110

2013-02-20 Thread Richard Earnshaw
On 19/02/13 22:26, Tilman Sauerbeck wrote: I don't get why relaxing the restrictions for the andsi3_compare0_scratch pattern results in a mismatch for the zeroextractsi_compare0_scratch one. Any ideas? Because of the way combine works. It first tries to find a pattern that doesn't have a

Re: [PATCH] Remove broken powf hack

2013-02-20 Thread Tobias Burnus
Am 18.02.2013 18:49, schrieb John David Anglin: This patch removes the broken powf hack. This problem is now fixed in the PA backend. Tested on hppa2.0w-hp-hpux11.11 and hppa64-hp-hpux11.11. OK for trunk? OK. Thanks for the patch and for fixing the problem more properly.* Tobias

Fix for 56175

2013-02-20 Thread Yuri Rumyantsev
Hi All, This patch is aimed to recognize (A C) ^ (B C) - (A ^ B) C pattern in simpify_bitwise_binary for short integer types. The fix is very simple - we simply turn off short type sinking at the first pass of forward propagation allows to get +10% speedup for important benchmark Coremark 1.0

Re: [PATCH][RFC] Less TODO_remove_unused_locals

2013-02-20 Thread Richard Biener
On Wed, 20 Feb 2013, Richard Biener wrote: Hunting for the we're getting slower bits I noticed that TODO_remove_unused_locals is a big part of execute_function_todo (and accounts for 1% of compile-time of ac.f90). The following patch removes most of the remove_unused_locals calls based on

Re: Fix for 56175

2013-02-20 Thread Richard Biener
On Wed, Feb 20, 2013 at 1:00 PM, Yuri Rumyantsev ysrum...@gmail.com wrote: Hi All, This patch is aimed to recognize (A C) ^ (B C) - (A ^ B) C pattern in simpify_bitwise_binary for short integer types. The fix is very simple - we simply turn off short type sinking at the first pass of

[PATCH] Fix PR56398

2013-02-20 Thread Richard Biener
This fixes an ICE because gimple_bb of a default def stmt is NULL. Just don't do anything here, we're not going to adjust anything anyway for them. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2013-02-20 Richard Biener rguent...@suse.de PR

[PATCH] Another simple dumping fix in IPA-CP

2013-02-20 Thread Martin Jambor
Hi, when debugging a PR I noticed that dumped numbers do not correspond to changed PARAM_IPA_CP_EVAL_THRESHOLD and defaults are hard-wired to the fprintf. Fixed by the patch below, which bootstraps and tests fine on x86_64-linux. Unless there are objections, I will commit it tomorrow as

[PATCH, PR 56294] Fix omissions in intersect_aggregates_with_edge

2013-02-20 Thread Martin Jambor
Hi, this patch fixes an omission in IPA-CP's agg_replacements_to_vector which needs to filter the vector by index and offset and a typo in intersect_aggregates_with_edge which in one call passed the wrong index to agg_replacements_to_vector. This combined lead to empty intersections which were

[PR middle-end/56108] handle transactions with ASMs in the first block

2013-02-20 Thread Aldy Hernandez
In the following test, the first statement of a relaxed transaction is an inline asm: __transaction_relaxed { __asm__(); } Since we bypass inserting BUILT_IN_TM_IRREVOCABLE at the beginning of transactions that are sure to be irrevocable, later when we try to expand the transaction, we ICE

Re: Fix for 56175

2013-02-20 Thread Yuri Rumyantsev
Richard, First of all, your proposal to move type sinking to the end of function does not work since we handle each statement in function and we want that 1st type folding of X C will not happen. Note that we have the following sequence of gimple before forwprop1: x.0_10 = (signed char) x_8;

Fix ICE in ipa_make_edge_direct_to_target

2013-02-20 Thread Jan Hubicka
Hi, in the testcase bellow we get an ICE in ipa_make_edge_direct_to_target. There is virtual call that gets devirtualized only while inlining functions called once. At this point however we already removed bodies for virtual functions from the callgraph, so we need to update it and re-create its

libgo patch committed: Fix x86_64 Solaris pointer usage

2013-02-20 Thread Ian Lance Taylor
This patch from Janne Snabb should fix PR 56320 about random failures on Solaris x86_64. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian diff -r 29b742a2ed37 -r 3b1c3cceaf02 libgo/runtime/lfstack.c --- a/libgo/runtime/lfstack.c Fri Feb 15 10:54:51 2013

libgo patch committed: Solaris net fixes

2013-02-20 Thread Ian Lance Taylor
This patch, mainly from Rainer Orth, in PR 56171 fixes passing a file descriptor on Solaris. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian diff -r 3b1c3cceaf02 libgo/Makefile.am --- a/libgo/Makefile.am Wed Feb 20 11:41:05 2013 -0800 +++

Re: [PATCH, PR 56294] Fix omissions in intersect_aggregates_with_edge

2013-02-20 Thread Jan Hubicka
Hi, this patch fixes an omission in IPA-CP's agg_replacements_to_vector which needs to filter the vector by index and offset and a typo in intersect_aggregates_with_edge which in one call passed the wrong index to agg_replacements_to_vector. This combined lead to empty intersections which

[patch] Tweak two libstdc++ tests

2013-02-20 Thread Jonathan Wakely
* testsuite/23_containers/unordered_set/55043.cc: Add missing namespace qualification. * testsuite/23_containers/unordered_multiset/55043.cc: Likewise. Tested x86_64-linux, committed to trunk. commit a6d9aa71b453ad2cf7b1cbd581fa04603728bad6 Author: Jonathan Wakely

[Patch, Fortran] PR 56385: [4.6/4.7/4.8 Regression] [OOP] ICE with allocatable function result in a procedure-pointer component

2013-02-20 Thread Janus Weil
Hi all, here is a straightforward patch which fixes a regression with procedure-pointer components which have an allocatable result. Regtests cleanly on x86_64-unknown-linux-gnu. Ok for trunk/4.7/4.6? [In absence of any reviews I will commit as obvious on the weekend.] Cheers, Janus

[patch] Fix spelling in libstdc++ docs and comments

2013-02-20 Thread Jonathan Wakely
* doc/html/faq.html: Fix spelling. * doc/xml/faq.xml: Likewise. * include/bits/basic_ios.h: Likewise. * include/bits/regex.h: Likewise. * include/std/istream: Likewise. * include/std/streambuf: Likewise. Tested x86_64-linux, committed to trunk

[patch] Fix libstdc++ doxygen page for basic_streambuf

2013-02-20 Thread Jonathan Wakely
This removes an unclosed @{ group marker so that Doxygen doesn't put half the members of basic_streambuf in the Friends section (see http://stackoverflow.com/q/14988997/981959) I also changed uses of __streambuf_type to basic_streambuf, which makes the Doxygen page look better because it doesn't

Re: PATCH: Correctly configure all big-endian ARM archs, not just arm*-*-linux-*.

2013-02-20 Thread Seth LaForge
On Fri, Feb 15, 2013 at 3:29 PM, Mike Stump mikest...@comcast.net wrote: No. Counter proposal, let's handle the cases that don't work. So, you said in your original email that armeb-unknown-eabi doesn't work. So, in the existing case statement for: arm*-*-eabi*) let's just add:

Re: PATCH: Correctly configure all big-endian ARM archs, not just arm*-*-linux-*.

2013-02-20 Thread Seth LaForge
On Sat, Feb 16, 2013 at 7:45 AM, Bernhard Reutner-Fischer rep.dot@gmail.com wrote: Sounds like a DUP of http://gcc.gnu.org/bugzilla/show_bug.cgi?id=16350 Is the missing hunk in by now (cannot look myself right now)? The commit you describe appears to be in trunk but not in gcc-4_7-branch.

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

2013-02-20 Thread Seth LaForge
On Tue, Feb 19, 2013 at 2:59 PM, Ramana Radhakrishnan ramana@googlemail.com wrote: This is not the correct form of a changelog entry. Sorry - another attempt below. Ok to backport provided no regressions when running the testsuite in big endian mode. Sorry to say I'm not sure how to run

[PATCH] MIPS: MIPS32r2 FP MADD instruction set support

2013-02-20 Thread Maciej W. Rozycki
Hi, This issue was originally raised here: http://gcc.gnu.org/ml/gcc-patches/2012-12/msg00863.html We have a shortcoming in GCC in that we only allow the use half of the FP MADD instruction subset (MADD.fmt and MSUB.fmt) in the 64-bit/32-register mode (CP0.Status.FR == 1) on MIPS32r2

[patch] df-scan: split df_insn_delete for clearer dumps and better speed

2013-02-20 Thread Steven Bosscher
Hello, The attached patch splits a new function df_insn_info_delete from df_insn_delete. The original motivation was to get rid of the silly deleting insn with uid = ... messages when re-scanning an insn, because the mentioned insn isn't deleted at all (it's just rescanned). But it turns out that

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

2013-02-20 Thread Jerry DeLisle
On 02/19/2013 02:40 PM, Janne Blomqvist wrote: 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()

closing PR's (was Re: [PATCH ARM iWMMXt 0/5] Improve iWMMXt support)

2013-02-20 Thread Hans-Peter Nilsson
On Mon, 28 Jan 2013, nick clifton wrote: Also, could you close its duplicates, bugs 36798 and 36966? Sorry no. I do not actually own these PRs, so I cannot close them. :-( Sorry if I misinterpret, but it seems a reminder is in order: magic powers are attached to whome...@gcc.gnu.org accounts

[PATCH] Don't expand *MEM_REF using extract_bit_field or movmisalign for EXPAND_MEMORY (PR inline-asm/56405)

2013-02-20 Thread Jakub Jelinek
Hi! If an input operand of inline-asm doesn't allow registers, but allows memory, we expand it with EXPAND_MEMORY modifier (the only case of using that modifier). But the movmisalign code added for 4.6 and especially the extract_bit_field code added for 4.8 results in getting a REG from

[PATCH] Small *.texi{,info} fixes for texinfo 5.0 (PR bootstrap/56258)

2013-02-20 Thread Jakub Jelinek
Hi! Currently it is not possible to bootstrap gcc with texinfo 5.0. This patch attempts to fix the errors that prevent bootstrap, there are tons of warnings this doesn't address and would be good if somebody more TeXinfo knowledgeable looked at it. Bootstrapped/regtested on x86_64-linux and