Re: Simple change in dot dumper -- Display profile count and branch probability

2013-04-21 Thread Xinliang David Li
Thanks. The patch is revised. David 2013-04-20 Xinliang David Li davi...@google.com * graph.c (draw_cfg_node_succ_edges): Add branch probility as label. * cfghhooks.c (dump_bb_for_graph): Dump profile count and frquency. * Makefile.in: New dependency. Index:

[4.7] Fix a couple of C++isms

2013-04-21 Thread Eric Botcazou
Bootstrapped/regtested on x86_64-suse-linux, applied as obvious. 2013-04-21 Eric Botcazou ebotca...@adacore.com * cfgexpand.c (avoid_complex_debug_insns): Fix C++ism. cp/ * parser.c (cp_parser_late_return_type_opt): Likewise. -- Eric BotcazouIndex: cp/parser.c

Re: [PATCH] Add explicit default constructors where required by the standard

2013-04-21 Thread Jonathan Wakely
On 19 April 2013 16:19, Evgeniy Stepanov wrote: Good point, thanks! Revised patch attached. I've committed that version, thanks very much. On Fri, Apr 19, 2013 at 7:00 PM, Jonathan Wakely jwakely@gmail.com wrote: On 19 April 2013 15:42, Evgeniy Stepanov wrote: Hi, according to

RFA: Fix use of extra_objs when configuring tilepro-linux and tilegx-linux

2013-04-21 Thread Nick Clifton
Hi Walter, The tilepro-gnu-linux and tilegx-gnu-linux currently fail to build in the FSF mainline sources because of: libbackend.a(tilepro.o):(.data+0x2e4): undefined reference to `linux_android_has_ifunc_p()' This happens because the entries for tilegx-linux and tilepro-linux in

New Swedish PO file for 'gcc' (version 4.8.0)

2013-04-21 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 Swedish team of translators. The file is available at: http://translationproject.org/latest/gcc/sv.po (This file, 'gcc-4.8.0.sv.po', has

Re: RFA: Fix use of extra_objs when configuring tilepro-linux and tilegx-linux

2013-04-21 Thread Walter Lee
Hi Nick. Looks good. Thanks for fixing it. Walter On 4/21/2013 7:03 AM, Nick Clifton wrote: Hi Walter, The tilepro-gnu-linux and tilegx-gnu-linux currently fail to build in the FSF mainline sources because of: libbackend.a(tilepro.o):(.data+0x2e4): undefined reference to

Re: Add std::unordered_* C++11 allocator support

2013-04-21 Thread Jonathan Wakely
On 21 April 2013 21:08, François Dumont wrote: Hi Here is another proposal with: - No attempt to remove const key - No attempt to use assignment operator - noexcept move constructor; I slightly modify a static assertion so that it checks that _M_bucket_index is noexcept qualified which

Re: patch to fix constant math - 4th patch - the wide-int class - patch ping for the next stage 1

2013-04-21 Thread Kenneth Zadeck
Richard, i pulled these two frags out of your comments because i wanted to get some input from you on it while i addressed the other issues you raised. + enum SignOp { +/* Many of the math functions produce different results depending + on if they are SIGNED or UNSIGNED. In

[Patch, Fortran] PR 57022: [4.7/4.8/4.9 Regression] Inappropriate warning for use of TRANSFER with arrays

2013-04-21 Thread Janus Weil
Hi all, the attached patch fixes an regression with TRANSFER, which was just reported today. The problem was that array-valued SOURCE arguments were not treated correctly. To fix it properly, I had to make 'gfc_target_expr_size' behave correctly for arrays and introduced an new function

Re: [4.7] Fix a couple of C++isms

2013-04-21 Thread Jeff Law
On 04/21/2013 01:59 AM, Eric Botcazou wrote: Bootstrapped/regtested on x86_64-suse-linux, applied as obvious. 2013-04-21 Eric Botcazou ebotca...@adacore.com * cfgexpand.c (avoid_complex_debug_insns): Fix C++ism. cp/ * parser.c (cp_parser_late_return_type_opt): Likewise. Umm,

Re: [4.7] Fix a couple of C++isms

2013-04-21 Thread Andrew Pinski
On Sun, Apr 21, 2013 at 2:13 PM, Jeff Law l...@redhat.com wrote: On 04/21/2013 01:59 AM, Eric Botcazou wrote: Bootstrapped/regtested on x86_64-suse-linux, applied as obvious. 2013-04-21 Eric Botcazou ebotca...@adacore.com * cfgexpand.c (avoid_complex_debug_insns): Fix C++ism.

Re: [4.7] Fix a couple of C++isms

2013-04-21 Thread Eric Botcazou
Umm, I thought declaring at first use (or near first use) was one of the approved C++-isms? http://gcc.gnu.org/wiki/CppConventions You probably missed the [4.7] in the subject. -- Eric Botcazou

[GOOGLE] Workaround a bug in AutoFDO which may leads to infinite loop for inlined assembly

2013-04-21 Thread Dehao Chen
This patch fixed a bug in getting inline stacks: if there is no location info attached to a block, we should *not* try to get its function name because it could result in infinite loop. Bootstrapped and passed all regression tests. Ok for gcc-4_7 branch? Thanks, Dehao Index: gcc/auto-profile.c

Re: [GOOGLE] Workaround a bug in AutoFDO which may leads to infinite loop for inlined assembly

2013-04-21 Thread Teresa Johnson
On Sun, Apr 21, 2013 at 4:51 PM, Dehao Chen de...@google.com wrote: This patch fixed a bug in getting inline stacks: if there is no location info attached to a block, we should *not* try to get its function name because it could result in infinite loop. Is the infinite loop within

Re: [GOOGLE] Workaround a bug in AutoFDO which may leads to infinite loop for inlined assembly

2013-04-21 Thread Dehao Chen
Thanks for the comment, new patch attached: Index: gcc/auto-profile.c === --- gcc/auto-profile.c (revision 198117) +++ gcc/auto-profile.c (working copy) @@ -623,6 +623,10 @@ static tree get_function_decl_from_block (tree block) {

Re: [GOOGLE] Workaround a bug in AutoFDO which may leads to infinite loop for inlined assembly

2013-04-21 Thread Teresa Johnson
Looks ok to me. Teresa On Sun, Apr 21, 2013 at 7:15 PM, Dehao Chen de...@google.com wrote: Thanks for the comment, new patch attached: Index: gcc/auto-profile.c === --- gcc/auto-profile.c (revision 198117) +++

[PATCH] Remove useless BIT_AND_EXPRs

2013-04-21 Thread Jeff Law
While looking at dumps I noticed a lot of sequences like t = a 1; x = (bool) t; The BIT_AND_EXPR is obviously redundant as the narrowing cast has the same effect. This can be trivially generalized for other integral types and an appropriate constant. At different times over the last

Re: RFA: enable LRA for rs6000 [patch for WRF]

2013-04-21 Thread Alan Modra
On Fri, Apr 19, 2013 at 05:16:43PM -0400, Vladimir Makarov wrote: If I change the above code to if (! lra_in_progress GET_MODE_SIZE (mode) UNITS_PER_WORD !(/* ??? Assume floating point reg based on mode? */ TARGET_HARD_FLOAT TARGET_FPRS