Re: [PATCH, i386] RTM support

2012-03-14 Thread Kirill Yukhin
Thanks! K On Tue, Mar 13, 2012 at 11:14 PM, Uros Bizjak ubiz...@gmail.com wrote: On Sun, Mar 11, 2012 at 10:16 AM, Kirill Yukhin kirill.yuk...@gmail.com wrote: The patch is OK for mainline, if there are no further comments in next 24h. According to Tobias's input, I've added few lines

Re: [RFC]: Add support for pragma pointer_size

2012-03-14 Thread Tristan Gingold
On Mar 13, 2012, at 9:57 PM, Gary Funck wrote: On 03/06/12 14:09:23, Tristan Gingold wrote: The patch is simple: the C front-end will now calls c_build_pointer_type (instead of build_pointer_type), which in turn calls build_pointer_type_for_mode using the right mode. [...] Joining this

Re: [PATCH, i386]: Declare constant_call_address_operand as special predicate

2012-03-14 Thread Uros Bizjak
On Wed, Mar 14, 2012 at 12:13 AM, H.J. Lu hjl.to...@gmail.com wrote: Similar to tls_symbolic_operand and tls_modbase_operand, we are not interested in the mode of the operand in the predicate. 2012-03-13  Uros Bizjak  ubiz...@gmail.com        * config/i386/predicates.md

[Patch]: Update fixincludes/README

2012-03-14 Thread Tristan Gingold
Hi, fixinc.sh is not anymore in gcc/ directory. So I propose to adjust fixincludes/README. Ok for trunk ? Tristan. fixincludes/ 2012-03-14 Tristan Gingold ging...@adacore.com * README (EXAMPLES OF FIXES): Update. diff --git a/fixincludes/README b/fixincludes/README index

Re: [PATCH] Fix PR 50971 and PR 35629: Only one loop detected when there should be two

2012-03-14 Thread Richard Guenther
On Tue, Mar 13, 2012 at 7:31 PM, Andrew Pinski pins...@gmail.com wrote: Ping?  Rebootstrapped on x86_64-linux-gnu with no regressions. Zdenek, can you have a look here? I think the patch is reasonable, but you should have a better idea ;) Thanks, Richard. Thanks, Andrew Pinski On Sat, Jan

Re: PATCH RFA: Update Go frontend on gcc 4.7 branch

2012-03-14 Thread Rainer Orth
Ian Lance Taylor i...@google.com writes: I would like to update the Go support on the 4.7 branch. As I've mentioned before, Go is working toward a stable Go 1 release. That release is not complete, but it is quite close. The 4.7 branch was made at a slightly unstable point in the process.

[PATCH] Fix PR52571

2012-03-14 Thread Richard Guenther
This fixes PR52571, we should not align DECL_COMMON variables as they might be pre-empted by a definition with lower alignment. At LTO/WPA level we might recover from missed optimizations by promoting DECL_COMMON variables to non-common. Not sure if we do that already. Bootstrapped on

[PATCH] Fix part of PR52582

2012-03-14 Thread Richard Guenther
This fixes part of PR52582, we need to create a cgraph node when folding from an initializer. Bootstrapped on x86_64-unknown-linux-gnu, testing in progress. Richard. 2012-03-14 Richard Guenther rguent...@suse.de PR middle-end/52582 * gimple-fold.c

Re: [Patch]: Update fixincludes/README

2012-03-14 Thread Bruce Korb
I suppose after a decade of being gone, it's time to stop referencing it. OK for *ALL* active branches. On Wed, Mar 14, 2012 at 2:36 AM, Tristan Gingold ging...@adacore.com wrote: Hi, fixinc.sh is not anymore in gcc/ directory.  So I propose to adjust fixincludes/README. Ok for trunk ?

Re: [PATCH] Fix PR52571

2012-03-14 Thread Richard Guenther
On Wed, 14 Mar 2012, Richard Guenther wrote: This fixes PR52571, we should not align DECL_COMMON variables as they might be pre-empted by a definition with lower alignment. At LTO/WPA level we might recover from missed optimizations by promoting DECL_COMMON variables to non-common. Not

[v3] Fix some comments in forward_list wrongly referring to size()

2012-03-14 Thread Paolo Carlini
Hi, highly misleading. Committed to mainline (suitable for 4.7.1 too of course) Thanks, Paolo. /// 2012-03-14 Paolo Carlini paolo.carl...@oracle.com * include/bits/forward_list.h: Fix comments. Index: include/bits/forward_list.h

[committed] Workaround for PR c++/52582

2012-03-14 Thread Jakub Jelinek
Hi! I've committed the following workaround for PR52582 to 4.7 branch (acked on IRC by David and Richard) after bootstrapping/regtesting it on powerpc64-linux (--with-cpu=default32 as well as -m64 defaulted build), and for trunk committed just the testcase. Richard committed some fix to the

Re: [Patch ARM/ configury] Add fall-back check for gnu_unique_object

2012-03-14 Thread Ramana Radhakrishnan
On 12 March 2012 18:10, DJ Delorie d...@redhat.com wrote: Looks OK to me. I was about to commit this into my svn checkout and then realized the patch p4 didn't have the changes to configure - So I regenerated it again and then began a journey into the depths of m4 and autoconf for a bit before

[dwarf]:

2012-03-14 Thread Tristan Gingold
Hi, the dwarf3 DW_OP_call_frame_cfa implicitly refers to .debug_frame section. But if we aren't emitting dwarf unwinding info, it isn't anymore well defined. There is one target that doesn't emit dwarf unwind info: ia64. And gdb is not able to deal with DW_OP_call_frame_cfa on ia64, thus

[C] Handle #pragma before a parameter

2012-03-14 Thread Tristan Gingold
Hi, it happens that some system headers on VMS have #pragma between parameters. This is spotted by building the Ada runtime. This patch simply handles them. Manually tested by building for ia64-hp-openvms. No regressions for c on x86_64-darwin. Ok for trunk ? Tristan. 2012-03-14 Tristan

Re: [Patch ARM/ configury] Add fall-back check for gnu_unique_object

2012-03-14 Thread Paolo Bonzini
Il 14/03/2012 16:37, Ramana Radhakrishnan ha scritto: Empirically I spotted this odd behaviour with gcc_GAS_CHECK_FEATURE and comments - Attached are the 2 alternate patches that I tried and the difference in the configure scripts themselves . I am no m4 expert but it does look like the

[Patch/cfgexpand]: also consider assembler_name to call expand_main_function

2012-03-14 Thread Tristan Gingold
Hi, the code to call expand_main_function currently only checks DECL_NAME. This leads to a hack in ada/gcc-interface/utils.c to handle the gnatbind generated file that could declare: package ada_main is … function my_main (argc : Integer; argv : System.Address; envp :

Re: Remove DWARF_OFFSET_SIZE, DWARF_INITIAL_LENGTH_SIZE

2012-03-14 Thread Rainer Orth
Jason Merrill ja...@redhat.com writes: On 03/12/2012 01:41 PM, Joseph S. Myers wrote: As a *target macro* it makes sense to remove it - reomve it from defaults.h, make it purely internal to dwarf2out.c. But I think it makes sense to have it inside dwarf2out.c Agreed. Here's the revised

Re: [Patch/cfgexpand]: also consider assembler_name to call expand_main_function

2012-03-14 Thread Richard Guenther
On Wed, 14 Mar 2012, Tristan Gingold wrote: Hi, the code to call expand_main_function currently only checks DECL_NAME. This leads to a hack in ada/gcc-interface/utils.c to handle the gnatbind generated file that could declare: package ada_main is … function my_main (argc

Re: Remove non-GAS non-ELF support in alpha backend

2012-03-14 Thread Rainer Orth
Richard Henderson r...@redhat.com writes: On 03/12/12 09:22, Rainer Orth wrote: There are only two issues: * In alpha.c (alpha_option_override), it's unclear if the optimize 0 test can be removed completely. Yeah, I guess leave that for now. * During testing, I ran into this error:

Re: [Patch/cfgexpand]: also consider assembler_name to call expand_main_function

2012-03-14 Thread Tristan Gingold
On Mar 14, 2012, at 5:08 PM, Richard Guenther wrote: On Wed, 14 Mar 2012, Tristan Gingold wrote: Hi, the code to call expand_main_function currently only checks DECL_NAME. This leads to a hack in ada/gcc-interface/utils.c to handle the gnatbind generated file that could declare:

Re: [PATCH 3/3] Misaligned MEM_REF reads

2012-03-14 Thread Martin Jambor
Hi, On Mon, Mar 12, 2012 at 03:09:04PM +0100, Martin Jambor wrote: Hi, this patch is very similar to the one I posted before (http://gcc.gnu.org/ml/gcc-patches/2012-02/msg01377.html) except that it is now adjusted to sit on top of the new one before this and does not ignore complex

Re: [C] Handle #pragma before a parameter

2012-03-14 Thread Joseph S. Myers
On Wed, 14 Mar 2012, Tristan Gingold wrote: Hi, it happens that some system headers on VMS have #pragma between parameters. This is spotted by building the Ada runtime. This patch simply handles them. Manually tested by building for ia64-hp-openvms. No regressions for c on

Re: [Patch ARM/ configury] Add fall-back check for gnu_unique_object

2012-03-14 Thread Andreas Schwab
Paolo Bonzini bonz...@gnu.org writes: Yes, the # comment is actually part of the macro argument. If you want to write a real comment (i.e. at the m4 rather than shell level) use dnl instead of #. Actually both are part of the macro argument and act like comment introducers at the m4 level,

Re: Remove DWARF_OFFSET_SIZE, DWARF_INITIAL_LENGTH_SIZE

2012-03-14 Thread Joseph S. Myers
On Wed, 14 Mar 2012, Rainer Orth wrote: Jason Merrill ja...@redhat.com writes: On 03/12/2012 01:41 PM, Joseph S. Myers wrote: As a *target macro* it makes sense to remove it - reomve it from defaults.h, make it purely internal to dwarf2out.c. But I think it makes sense to have it

Re: Remove DWARF_OFFSET_SIZE, DWARF_INITIAL_LENGTH_SIZE

2012-03-14 Thread Rainer Orth
Joseph S. Myers jos...@codesourcery.com writes: On Wed, 14 Mar 2012, Rainer Orth wrote: Jason Merrill ja...@redhat.com writes: On 03/12/2012 01:41 PM, Joseph S. Myers wrote: As a *target macro* it makes sense to remove it - reomve it from defaults.h, make it purely internal to

Re: Remove non-GAS non-ELF support in alpha backend

2012-03-14 Thread Richard Henderson
On 03/14/12 09:09, Rainer Orth wrote: Nearly ok. + targetm.asm_file_start_file_directive = 0; This is default and may be deleted. r~

[patch, gcc RFA] dg-extract-results.sh: Handle KFAILs.

2012-03-14 Thread Doug Evans
The results of running the testsuite in parallel should match the results when run serially. This patch adds KFAIL counts so that happens. [There's still a nit that the order of the results don't precisely match, but that's a separate issue.] I will check this into the gdb tree if there are no

[PATCH] Optimize in VRP if ((x cst1) cmp cst2) (PR tree-optimization/52267)

2012-03-14 Thread Jakub Jelinek
Hi! This patch adds ASSERT_EXPRs derived from tmp_N = var_M cst2; if (tmp_N cmp val) where cst2 and val are constants. Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? 2012-03-14 Jakub Jelinek ja...@redhat.com PR tree-optimization/52267 * tree-vrp.c

Re: PATCH RFA: Update Go frontend on gcc 4.7 branch

2012-03-14 Thread Ian Lance Taylor
Rainer Orth r...@cebitec.uni-bielefeld.de writes: Ian Lance Taylor i...@google.com writes: I would like to update the Go support on the 4.7 branch. As I've mentioned before, Go is working toward a stable Go 1 release. That release is not complete, but it is quite close. The 4.7 branch was

Re: [dwarf]:

2012-03-14 Thread Jason Merrill
OK. Jason

Re: PATCH RFA: Update Go frontend on gcc 4.7 branch

2012-03-14 Thread Rainer Orth
Ian Lance Taylor i...@google.com writes: Unfortunately, this patch introduced several go and/or libgo testsuite failures on both Solaris/SPARC and x86 on the 4.7 branch while the results were clean before. I've filed PR go/52583 for this, but not yet investigated in detail. Hmmm, the patch

Re: Remove non-GAS non-ELF support in alpha backend

2012-03-14 Thread Rainer Orth
Richard Henderson r...@redhat.com writes: On 03/14/12 09:09, Rainer Orth wrote: Nearly ok. + targetm.asm_file_start_file_directive = 0; This is default and may be deleted. Or would be if alpha.c didn't override the default. This is what I actually committed after retesting.

Re: Remove MIPS_DEBUGGING_INFO support

2012-03-14 Thread Rainer Orth
Joseph S. Myers jos...@codesourcery.com writes: On Mon, 5 Mar 2012, Rainer Orth wrote: The only two users of MIPS_DEBUGGING_INFO are on their way out: I've just submitted a patch to remove the OpenBSD/MIPS configuration, and IRIX removal will follow soon. There seems to be no point in

Re: [gimplefe] [patch] splits cpp_lshift and cpp_rshift tokens into discrete cpp_less and cpp_greater tokens respectively

2012-03-14 Thread Sandeep Soni
On Tue, Mar 13, 2012 at 9:42 PM, Diego Novillo dnovi...@google.com wrote: On 08/03/12 20:47 , Sandeep Soni wrote: +/* Splits the token TOKEN into two tokens FIRST_TOKEN and SECOND_TOKEN. +   Note that the split should work only if the type of the TOKEN is +   either CPP_RSHIFT or CPP_LSHIFT

[google/4.6] Fix problem where -gfission emits duplicate strings (issue5824050)

2012-03-14 Thread Cary Coutant
This is for the google/gcc-4_6 branch only. Fix output_indirect_string so that it does not output strings that have already been written to the .debug_str.dwo section. This is a backport of part of an unrelated change from upstream 4.7: http://gcc.gnu.org/ml/gcc-cvs/2011-05/msg00130.html

Re: RFC: PATCH: Add -maddress-mode=short|long for x86

2012-03-14 Thread Uros Bizjak
On Mon, Mar 12, 2012 at 6:48 PM, H.J. Lu hongjiu...@intel.com wrote: Trunk can now use SImode or DImode for Pmode with -mx32.  Pmode == SImode will avoid many conversions from SImode to DImode: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50797 It will use 0x67 address size prefix instead.  

Re: [google/4.6] Fix problem where -gfission emits duplicate strings (issue 5824050)

2012-03-14 Thread saugustine
On 2012/03/14 18:15:25, Cary wrote: This is for the google/gcc-4_6 branch only. Fix output_indirect_string so that it does not output strings that have already been written to the .debug_str.dwo section. This is a backport of part of an unrelated change from upstream 4.7:

Re: RFC: PATCH: Add -maddress-mode=short|long for x86

2012-03-14 Thread Joseph S. Myers
On Wed, 14 Mar 2012, Uros Bizjak wrote: The patch is technically OK for mainline, Joseph - can you please review option stuff? The option changes look fine to me. -- Joseph S. Myers jos...@codesourcery.com

Re: [patch, gcc RFA] dg-extract-results.sh: Handle KFAILs.

2012-03-14 Thread Mike Stump
On Mar 14, 2012, at 10:21 AM, Doug Evans wrote: The results of running the testsuite in parallel should match the results when run serially. This patch adds KFAIL counts so that happens. [There's still a nit that the order of the results don't precisely match, but that's a separate issue.]

New Swedish PO file for 'gcc' (version 4.7-b20120128)

2012-03-14 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.7-b20120128.sv.po',

[C++ Patch] PR 44783

2012-03-14 Thread Paolo Carlini
Hi, Manuel provided this patchlet improving his template instantiation notes work to use a template_backtrace_limit parameter, as requested in c++/44783. I booted and tested it on x86_64-linux. Is it Ok for mainline? Anything else we want to do here? Thanks, Paolo. ///

[google][4.6]Make option -freorder-functions= invoke function reordering linker plugin (issue5825054)

2012-03-14 Thread Sriraman Tallam
This patch overloads GCC option -freorder-functions with -freoder-functions=* which will invoke the linker plugin libfunction_reordering_plugin.so. For now, the only accepted option is -freoder-functions=cgedge, where the functions which are connected by hot callgraph edges are placed closer.

Re: [google][4.6]Make option -freorder-functions= invoke function reordering linker plugin (issue5825054)

2012-03-14 Thread Sriraman Tallam
On Wed, Mar 14, 2012 at 1:56 PM, Sriraman Tallam tmsri...@google.com wrote: This patch overloads GCC option -freorder-functions with -freoder-functions=* which will invoke the linker plugin libfunction_reordering_plugin.so. For now, the only accepted option is -freoder-functions=cgedge, where

Re: RFC: PATCH: Add -maddress-mode=short|long for x86

2012-03-14 Thread Gerald Pfeifer
On Wed, 14 Mar 2012, Uros Bizjak wrote: I have also added Gerald to CC, to check if everything is OK with added documentation. Joseph is better with that than I am, but I'll give it a try. :-) +@item -maddress-mode=long +@opindex maddress-mode=long +Generate code for long address mode. It is

Re: [PATCH] Fix PR49484, gthr requirements update (target maintainers have a looksee)

2012-03-14 Thread Gerald Pfeifer
On Tue, 13 Mar 2012, Richard Guenther wrote: Goes back to rev. 5880 by rms, at which time tsystem.h did not exist. I'm going to remove those two lines, bootstrap test it and commit as obvious. Thanks, Richard. I could not do the tests you suggested yesterday before seeing this other mail.

[SH] Add FPUL usage testcase

2012-03-14 Thread Oleg Endo
Hi, I'd like to add an SH target testcase which is supposed to check the usage of the FPUL register when float values are treated as int and vice versa. Does this make sense? Tested against rev 185360 with the usual make -k check RUNTESTFLAGS=--target_board=sh-sim

Re: [google][4.6]Make option -freorder-functions= invoke function reordering linker plugin (issue5825054)

2012-03-14 Thread Xinliang David Li
thanks. This greatly improves the usability of the plugin based function reordering feature and is in a shape that can be pushed to trunk. Regarding the sub-options, cgedge does not seem user friendly. How about just -freorder-functions=callgraph or -freorder-function=clustering? What is the

Re: RFC: PATCH: Add -maddress-mode=short|long for x86

2012-03-14 Thread H.J. Lu
On Wed, Mar 14, 2012 at 2:57 PM, Gerald Pfeifer ger...@pfeifer.com wrote: On Wed, 14 Mar 2012, Uros Bizjak wrote: I have also added Gerald to CC, to check if everything is OK with added documentation. Joseph is better with that than I am, but I'll give it a try. :-) +@item

Re: [SH] Add FPUL usage testcase

2012-03-14 Thread Kaz Kojima
Oleg Endo oleg.e...@t-online.de wrote: I'd like to add an SH target testcase which is supposed to check the usage of the FPUL register when float values are treated as int and vice versa. Does this make sense? Tested against rev 185360 with the usual make -k check

Re: [SH] Add FPUL usage testcase

2012-03-14 Thread Oleg Endo
On Thu, 2012-03-15 at 08:12 +0900, Kaz Kojima wrote: Oleg Endo oleg.e...@t-online.de wrote: I'd like to add an SH target testcase which is supposed to check the usage of the FPUL register when float values are treated as int and vice versa. Does this make sense? Tested against rev

Re: [Patch, libfortran] Reduce default precision for list-directed and G0 real output

2012-03-14 Thread Jerry DeLisle
On 03/12/2012 05:28 PM, Janne Blomqvist wrote: Hi, currently when writing a value of type real or complex using list-directed output, the G0 edit descriptor, or namelist output, gfortran chooses the number of significant digits such that a binary-ascii-binary roundtrip recovers the original

Re: [SH] Add FPUL usage testcase

2012-03-14 Thread Kaz Kojima
Oleg Endo oleg.e...@t-online.de wrote: I'd like to add an SH target testcase which is supposed to check the usage of the FPUL register when float values are treated as int and vice versa. Does this make sense? Tested against rev 185360 with the usual make -k check

[pph] Identify and diagnose one class of bad merges (issue5822054)

2012-03-14 Thread Diego Novillo
Detect one class of invalid merges. When merging two different EXPRs, we are currently merging some trees that should not be merged. This patch tries to identify one case where we can certainly detect that a bad merge is about to happen. This fixes no tests, but diagnoses quite a few bad merges

Re: [Patch, libfortran] PR 52434/48878/38199 Improve floating point formatted writes

2012-03-14 Thread Jerry DeLisle
On 03/08/2012 05:40 PM, Janne Blomqvist wrote: Hi, the attached patch implements some improvements for formatted writes of floating point values. Currently libgfortran uses snprintf() to fill a buffer with a fixed amount of digits, regardless of the digits required, and then rounding or zero

[pph] Add a GC root for the lexer (issue5822055)

2012-03-14 Thread Diego Novillo
Add a GC root for the lexer. This GC root is required to support running the garbage collector during lexing. This happens when the lexer loads pre-parsed headers. Loading these headers calls into the middle-end, which may run the garbage collector. This was triggering in some of the bigger

[google][4.6]Make option -freorder-functions= invoke function reordering linker plugin (issue5825054)

2012-03-14 Thread Sriraman Tallam
Made the following changes: * -ffunction-sections turned on with -freorder-functions= * Change name from cgedge to callgraph. * The plugin should not dump final layout to file by default. * cgraphbuild.c (remove_cgraph_callee_edges): Replace flag_callgraph_profiles_sections with

Re: [google][4.6]Make option -freorder-functions= invoke function reordering linker plugin (issue5825054)

2012-03-14 Thread Sriraman Tallam
On Wed, Mar 14, 2012 at 6:56 PM, Sriraman Tallam tmsri...@google.com wrote: Made the following changes: * -ffunction-sections turned on with -freorder-functions= * Change name from cgedge to callgraph. * The plugin should not dump final layout to file by default.        * cgraphbuild.c

Re: [C++ Patch] PR 44783

2012-03-14 Thread Jason Merrill
OK. Jason