Re: [PATCH, i386]: AMD bdver3 enablement

2012-11-05 Thread Uros Bizjak
On Mon, Nov 5, 2012 at 8:33 AM, Gopalasubramanian, Ganesh ganesh.gopalasubraman...@amd.com wrote: Couple of changes done with respect to the review comments. 1. sseshuf type attribute is handled in unit attribute calculation. 2. sseadd1 instruction attribute is handled in the new scheduler

[x86] Fix gcc.c-torture/compile/20080806-1.c failures

2012-11-05 Thread Richard Sandiford
This is a case where we had: (set (reg:HI foo) (plus:HI (reg:HI sp) (const_int X))) (clobber (reg:CC FLAGS_REG)) and the splitters decided to convert it to an LEA: (set (reg:SI foo) (plus:SI (subreg:SI (reg:HI sp) 0) (const_int X))) But this fails to match, because

Re: [x86] Fix gcc.c-torture/compile/20080806-1.c failures

2012-11-05 Thread Uros Bizjak
On Mon, Nov 5, 2012 at 9:30 AM, Richard Sandiford rdsandif...@googlemail.com wrote: This is a case where we had: (set (reg:HI foo) (plus:HI (reg:HI sp) (const_int X))) (clobber (reg:CC FLAGS_REG)) and the splitters decided to convert it to an LEA: (set (reg:SI foo) (plus:SI

[SH] PR 54089 - Add support for rotcl instruction

2012-11-05 Thread Oleg Endo
Hello, This patch adds support for SH's rotcl instruction. While working on it, I've noticed that the DImode left shift by one insn was not used anymore, and instead ended up as 'x + x'. This transformation was happening before/during RTL expansion. The fix for it was to adjust the costs for

Re: [PATCH, middle-end]: Fix mode-switching MODE_EXIT check with __builtin_apply/__builtin_return

2012-11-05 Thread Kaz Kojima
Uros Bizjak ubiz...@gmail.com wrote: 2012-11-04 Vladimir Yakovlev vladimir.b.yakov...@intel.com Uros Bizjak ubiz...@gmail.com * mode-switching.c (create_pre_exit): Added code for maybe_builtin_apply case. Patch was bootstrapped and regression tested on

Re: [PATCH, middle-end]: Fix mode-switching MODE_EXIT check with __builtin_apply/__builtin_return

2012-11-05 Thread Vladimir Yakovlev
Hellow, Kaz I've updated copyright. Is it Ok? Thanks, Vladimir --- a/gcc/mode-switching.c +++ b/gcc/mode-switching.c @@ -1,6 +1,6 @@ /* CPU mode switching Copyright (C) 1998, 1999, 2000, 2001, 2002, 2003, 2004, 2005, 2007, 2008, - 2009, 2010 Free Software Foundation, Inc. + 2009, 2010,

Re: [x86] Fix gcc.c-torture/compile/20080806-1.c failures

2012-11-05 Thread H.J. Lu
On Mon, Nov 5, 2012 at 1:01 AM, Uros Bizjak ubiz...@gmail.com wrote: On Mon, Nov 5, 2012 at 9:30 AM, Richard Sandiford rdsandif...@googlemail.com wrote: This is a case where we had: (set (reg:HI foo) (plus:HI (reg:HI sp) (const_int X))) (clobber (reg:CC FLAGS_REG)) and the splitters

[PATCH, testsuite]: Fix gcc.dg/tree-ssa/cunroll-9.c failure

2012-11-05 Thread Uros Bizjak
Hello! 2012-11-05 Uros Bizjak ubiz...@gmail.com * gcc.dg/tree-ssa/cunroll-9.c: Dump cunrolli details. Fix scan-tree-dump-times directive. Tested on x86_64-pc-linux-gnu, committed to mainline SVN. Uros. Index: gcc.dg/tree-ssa/cunroll-9.c

Re: exception safety validation

2012-11-05 Thread Jonathan Wakely
On 4 November 2012 16:01, Jonathan Wakely wrote: On 3 November 2012 19:05, Paolo Carlini wrote: On 11/03/2012 03:43 PM, François Dumont wrote: Note that I run exception tests to validate it and the recently introduced 23_containers/forward_list/allocator/noexcept.cc test doesn't work in

[PATCH] Fix up pr19105.c test regexp for ppc (PR testsuite/55188)

2012-11-05 Thread Jakub Jelinek
Hi! Depending on branch cost, e.g. on ppc v != d v != e isn't folded in the FE. On x86_64/i686 we have in *.original for range1: return (x != 0 (unsigned int) v != 2) (unsigned int) v + 4294967293 1; but on ppc return ((x != 0 (unsigned int) v != 2) (unsigned int) v != 3) (unsigned int)

Re: [RFH] subreg of a vector without going through memory

2012-11-05 Thread Richard Sandiford
Marc Glisse marc.gli...@inria.fr writes: * simplify-rtx.c (simplify_subreg): For vectors, create a VEC_SELECT. Probably not helpful, sorry, but a subreg-vec_select transformation feels like it's going in the wrong direction. Going from vec_select to subreg would be OK from a layering

Re: [PATCH] Fix final_scan_insn to handle all comparisons in non-jump and cmove insn

2012-11-05 Thread Richard Sandiford
Andreas Schwab sch...@linux-m68k.org writes: * final.c (final_scan_insn) [HAVE_cc0]: Handle all comparison codes in non-jump and cmove insn. OK, thanks. Richard

[PATCH, testsuite]: Fix gcc.dg/torture/pr55018.c testsuite failure

2012-11-05 Thread Uros Bizjak
Hello! Just another instance of testsuite PR51128. 2012-11-05 Uros Bizjak ubiz...@gmail.com PR testsuite/51128 * gcc.dg/torture/pr55018.c: Skip if -fno-fat-lto-objects was passed. Tested on x86_64-pc-linux-gnu, committed to mainline SVN. Uros. Index:

Re: [patch] Apply conditional down cast to cgraph.h et.al.

2012-11-05 Thread Eric Botcazou
Done and committed. That's great, thanks! -- Eric Botcazou

[PATCH] Fix up value_format ICE for DWARF2_ADDR_SIZE 4 targets (PR target/55194)

2012-11-05 Thread Jakub Jelinek
Hi! Apparently some targets have DWARF2_ADDR_SIZE 4, for dw_val_class_addr when DW_FORM_addr can't be used value_format already handles DWARF2_ADDR_SIZE 1, 2, 4 and 8, so I've committed this change as obvious. 2012-11-05 Jakub Jelinek ja...@redhat.com PR target/55194 *

Re: [PATCH] Stream cgraph_node.ipa_transforms_to_apply

2012-11-05 Thread Jan Hubicka
Hi, the following patch adds streaming ofcgraph_node.ipa_transforms_to_apply so that transformation phases of IPA passes are run in LTO too. It is done by simple streaming of pass.static_pass_number and then looking it up among all_regular_ipa_passes. Bootstrapped and tested on

Fix autopar testcase

2012-11-05 Thread Jan Hubicka
Hi, this testcase tests that we are about to bound number of iterations of for (k = 3; k NA_1; k++) for (i = 3; i MA_1; i++) for (j = 3; j MB_1; j++) { t = T[i][j]; T[i][j] = t+2+A[i][k]*B[j][k]; } based on #define MB 100 #define NA 450

Re: GCC for GNU Hurd: MACH built-in preprocessor macro (was: gdb: FTBFS on hurd-i386 (for review))

2012-11-05 Thread Samuel Thibault
Thomas Schwinge, le Mon 05 Nov 2012 07:09:43 +0100, a écrit : Samuel, is there any way you can unpack all Debian source packages on a Debian machine, and run a recursive grep command (I can work out a suitable regex) to see where removing the MACH or __MACH built-in preprocessor macros might

Relax limits of early inliner for the forwarder functions

2012-11-05 Thread Jan Hubicka
Hi, in 4.6 timeframe I limited early inlier growth to apply only for leaf functions. This does not work really well, because with less propagation of address expressions we are really not 100% succesfull on detecting C++ forwarders and predicting them zero cost. This patch simply makes the

Minor fixes to niter and ivcanon

2012-11-05 Thread Jan Hubicka
Hi, while proofreading niter and ivcanon code I noticed that we give up on exits in inner loops. I discussed this with Zdenek and it was done only because at the time it was implemented we did nothing useful on outer loops. We can now unloop them, so I removed the restriction. I also noticed

Re: [PATCH] Update source location for PRE inserted stmt

2012-11-05 Thread Eric Botcazou
Those compiler generated statements do not have source origins, but they need to have debug location information attached so that information collected for them can be correlated with program constructs (such as CFG). One of the natural way is to use the source location associated with the

Re: [RFH] subreg of a vector without going through memory

2012-11-05 Thread Marc Glisse
On Mon, 5 Nov 2012, Richard Sandiford wrote: Marc Glisse marc.gli...@inria.fr writes: * simplify-rtx.c (simplify_subreg): For vectors, create a VEC_SELECT. Probably not helpful, sorry, but a subreg-vec_select transformation feels like it's going in the wrong direction. Going from

Re: [PATCH] Do not change scope for unknown locations

2012-11-05 Thread Eric Botcazou
As Richard suggested, if a location is UNKNOWN_LOCATION, instead of change_scope to DECL_INITIAL (cfun-decl), we should let it inherit location from its previous instructions. This is implemented in the attached patch. Bootstrapped and passed gcc regression tests and gdb regression tests.

Re: GCC 4.8.0 Status Report (2012-10-29), Stage 1 to end soon

2012-11-05 Thread Peter Bergner
On Mon, 2012-10-29 at 18:56 +0100, Jakub Jelinek wrote: Status == I'd like to close the stage 1 phase of GCC 4.8 development on Monday, November 5th. If you have still patches for new features you'd like to see in GCC 4.8, please post them for review soon. Patches posted before the

Re: GCC 4.8.0 Status Report (2012-10-29), Stage 1 to end soon

2012-11-05 Thread Jakub Jelinek
On Mon, Nov 05, 2012 at 06:41:47AM -0600, Peter Bergner wrote: On Mon, 2012-10-29 at 18:56 +0100, Jakub Jelinek wrote: I'd like to close the stage 1 phase of GCC 4.8 development on Monday, November 5th. If you have still patches for new features you'd like to see in GCC 4.8, please post

Re: [x86] Fix gcc.c-torture/compile/20080806-1.c failures

2012-11-05 Thread H.J. Lu
On Mon, Nov 5, 2012 at 2:21 AM, H.J. Lu hjl.to...@gmail.com wrote: On Mon, Nov 5, 2012 at 1:01 AM, Uros Bizjak ubiz...@gmail.com wrote: On Mon, Nov 5, 2012 at 9:30 AM, Richard Sandiford rdsandif...@googlemail.com wrote: This is a case where we had: (set (reg:HI foo) (plus:HI (reg:HI sp)

New badness metric for inliner

2012-11-05 Thread Jan Hubicka
Hi, this patch implements new badness metric for inliner. It is now based on relative speedup, but not of calee, but of the caller + callee combo. This nicely encompasses the edge frequency and other parameters simplifying the actual badness code. I re-added code considering growth after

Re: GCC 4.8.0 Status Report (2012-10-29), Stage 1 to end soon

2012-11-05 Thread Kenneth Zadeck
On 11/04/2012 11:54 AM, Richard Biener wrote: On Thu, Nov 1, 2012 at 2:10 PM, Richard Sandiford rdsandif...@googlemail.com wrote: Kenneth Zadeck zad...@naturalbridge.com writes: I would like you to respond to at least point 1 of this email. In it there is code from the rtl level that was

Re: [Committed] Fix 54524: Wrong code with some 64bit addition with registers as 32bits

2012-11-05 Thread Eric Botcazou
Hmm, like you said originally in the PR, I think generating the invalid PLUS is the bug here. I agree (as Richard B said) that an invalid PLUS shouldn't cause us to generate wrong code, but an assert seems better than a check. Eric, what do you think? I think that this boils down to

Re: [RFA:] PR55186 - gcc.dg/const-uniq-1.c fails due to vector not in the constant pool

2012-11-05 Thread Eric Botcazou
But, for cris-elf (and reasonably the same for other targets) there might not be such a constant-pool entry in the first place: the vectors are too short to rule out piecewise initialization as optimal for size (counting the vectors once per use). Let's increase them. Twice might just

Re: Ping / update: RFA: replace #ifdef with if/#if for HAVE_ATTR_*

2012-11-05 Thread Bernd Schmidt
On 10/31/2012 01:12 PM, Richard Sandiford wrote: OK with those changes for the rtl bits. Can't approve the generator stuff though. That's also OK. Bernd

Re: GCC 4.8.0 Status Report (2012-10-29), Stage 1 to end soon

2012-11-05 Thread Peter Bergner
On Mon, 2012-11-05 at 13:53 +0100, Jakub Jelinek wrote: On Mon, Nov 05, 2012 at 06:41:47AM -0600, Peter Bergner wrote: I'd like to post later today (hopefully this morning) a very minimal configure patch that adds the -mcpu=power8 and -mtune=power8 compiler options to gcc. Currently,

Re: GCC 4.8.0 Status Report (2012-10-29), Stage 1 to end soon

2012-11-05 Thread Jakub Jelinek
On Mon, Nov 05, 2012 at 08:40:00AM -0600, Peter Bergner wrote: Well we also patch config.in and configure.ac/configure. If those are acceptable to be patched later too, then great. If not, the patch That is the same thing as config.gcc bits. isn't really very large. We did do this for

[committed] 3 backports to 4.7 branch

2012-11-05 Thread Jakub Jelinek
Hi! I've backported 3 patches of mine to 4.7 branch, and committed them after bootstrap/regtest on x86_64-linux and i686-linux. Jakub 2012-11-05 Jakub Jelinek ja...@redhat.com Backported from mainline 2012-10-10 Jakub Jelinek ja...@redhat.com PR

Re: [PATCH] Do not change scope for unknown locations

2012-11-05 Thread Dehao Chen
OK, thanks. What happens if you go one step farther and always continue? Because we do want location and its block to be tightly coupled. I may want to add an assertion that if location is known, the block should *not* be NULL. Dehao -- Eric Botcazou

Re: [PATCH] Update source location for PRE inserted stmt

2012-11-05 Thread Dehao Chen
No, there is nothing natural (and this can even be wrong). The statements must have the source location corresponding to the source construct they are generated for, which may be totally different from that of the insertion point. Yes, that can generate jumpiness in GDB, but this is far

Re: [PATCH 08/10] Factorize condition insertion code out of build_check_stmt

2012-11-05 Thread Jakub Jelinek
On Sat, Nov 03, 2012 at 12:03:45AM +0100, Dodji Seketeli wrote: + int fallthrough_probability = +then_more_likely_p +? PROB_VERY_UNLIKELY +: PROB_ALWAYS - PROB_VERY_UNLIKELY; Just a formatting nit, I think = needs to go on the next line, so int fallthrough_probability =

Re: [PATCH] Minor ipa-prop.c internal interface tweak

2012-11-05 Thread Jan Hubicka
2012-10-31 Martin Jambor mjam...@suse.cz * ipa-prop.c (ipa_get_param_decl_index_1): New function. (ipa_get_param_decl_index): Just call ipa_get_param_decl_index_1. (ipa_populate_param_decls): Accept descriptors parameter rather than the whole info.

Re: [PATCH] Disable aggregate jump functions for bit-field stores

2012-11-05 Thread Jan Hubicka
Hi, the patch below disables generation of aggregate jump functions from bit-field stores because currently we depend on type size of the value to determine the size of the stored value and that does not work with bit-fields, making it impossible for IPA-CP to organize them in their

Re: GCC 4.8.0 Status Report (2012-10-29), Stage 1 to end soon

2012-11-05 Thread David Malcolm
On Wed, 2012-10-31 at 11:13 +0100, Richard Biener wrote: On Mon, Oct 29, 2012 at 6:56 PM, Jakub Jelinek ja...@redhat.com wrote: Status == I'd like to close the stage 1 phase of GCC 4.8 development on Monday, November 5th. If you have still patches for new features you'd like to

patch to fix PR55151

2012-11-05 Thread Vladimir Makarov
The following patch fixes PR55151. The patch affects a sensitive part of LRA code. So it took some time to find a PR solution. For the test there were to many reloads into hard registers for an insn and LRA failed to assign hard registers to all reload pseudos. There is another more

Implement simple peeling on tree level; remove RTL peeling code

2012-11-05 Thread Jan Hubicka
Hi, this patch removes RTL loop peeling code and makes tree-cunroll pass to also perform simple peeling (i.e. one done with profile feedback when loop is expected to iterate just few times). The motivation is 1) I want to do some re-tunning of tree peeling heuristics first week(s) of stage

Re: [PR54693] loss of debug info in jump threading and loop ivopts

2012-11-05 Thread Jeff Law
On 11/03/2012 10:11 AM, Alexandre Oliva wrote: On Nov 2, 2012, Alexandre Oliva aol...@redhat.com wrote: On Nov 1, 2012, Jakub Jelinek ja...@redhat.com wrote: Even for stmt frontiers it is IMHO undesirable to duplicate (perhaps many times) the whole sequence, as one would then reply the var

Re: GCC 4.8.0 Status Report (2012-10-29), Stage 1 to end soon

2012-11-05 Thread Kenneth Zadeck
Jakub and Richi, At this point I have decided to that i am not going to get the rest of the wide-int patches into a stable enough form for this round. The combination of still living without power at my house and some issues that i hit with the front ends has made it impossible to get this

Re: [PATCH] Update source location for PRE inserted stmt

2012-11-05 Thread Xinliang David Li
For the example I listed, the new statement is generated for source construct at program point (2). However unlike simple code motion, (2) is not going away after PRE. How would setting the location of the new statement at the insertion point break coverage? Besides, the new statement won't create

committed: Fix dbr processing of conditional returns

2012-11-05 Thread Joern Rennecke
fill_simple_delay_slots would call optimize_skip with a conditional return, which would cause a segfault when it calls next_active_insn (RETURN). Other places in reorg.c check that they are not operating on a return insn before doing branch-insn only processing, so I added this check there too.

Re: [PATCH] New configuration options to enable additional executable/startfile/shared library prefixes

2012-11-05 Thread Michael Meissner
On Sat, Nov 03, 2012 at 07:01:04PM -0400, David Edelsohn wrote: On Thu, Nov 1, 2012 at 5:17 PM, Michael Meissner meiss...@linux.vnet.ibm.com wrote: This patch adds 4 additional configuration switches, that allow the person building the compiler to add additional prefixes to search for

libgo patch committed

2012-11-05 Thread Ian Lance Taylor
This patch from Shenghou Ma supports the os.Stat function on some more operating systems. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian diff -r 5d8ca2f79dba libgo/Makefile.am --- a/libgo/Makefile.am Fri Nov 02 16:37:20 2012 -0700 +++

Re: [PATCH] Disable aggregate jump functions for bit-field stores

2012-11-05 Thread Martin Jambor
On Mon, Nov 05, 2012 at 05:20:46PM +0100, Jan Hubicka wrote: Hi, the patch below disables generation of aggregate jump functions from bit-field stores because currently we depend on type size of the value to determine the size of the stored value and that does not work with

libgo patch committed: NetBSD fix

2012-11-05 Thread Ian Lance Taylor
This patch from Shenghou Ma fixes gccgo on libgo by not losing the TLS pointer. This works around what I would describe as a bug in the NetBSD libc. Bootstrapped on x86_64-unknown-linux-gnu, not that that proves much. Committed to mainline. Ian diff -r df61836f495f libgo/runtime/proc.c ---

Re: [PING^2] [C++ PATCH] Add overflow checking to __cxa_vec_new[23]

2012-11-05 Thread Florian Weimer
On 11/02/2012 01:14 PM, Paolo Carlini wrote: On 11/02/2012 01:09 PM, Florian Weimer wrote: I looked at this again and made a new copy of the test case instead. It has been successfully tested on x86_64-redhat-linux-gnu. Is this okay for trunk? Looks very nice to me, and after all the issue

PR 54805: __gthread_tsd* in vxlib-tls.c

2012-11-05 Thread rbmj
Hello all, Since nobody has commented on bug 54805, and I'm pretty sure this is valid (and obvious), I'm just submitting it to the list. This removes warnings about implicit declarations and fixes one of the function calls in vxlib-tls.c for vxworks targets. I got the old prototypes from

Re: [PATCH, middle-end]: Fix mode-switching MODE_EXIT check with __builtin_apply/__builtin_return

2012-11-05 Thread Eric Botcazou
This sequence breaks assumption in mode-switching.c, that final function return register load operates in MODE_EXIT mode and triggere following code: for (j = n_entities - 1; j = 0; j--) { int e = entity_map[j];

Re: [patch][RFC] Filename based shared library versioning on AIX

2012-11-05 Thread Michael Haubenwallner
On 11/02/2012 12:16 AM, David Edelsohn wrote: I would like to introduce filename-based shared library versioning (known as the soname in ELF world) for AIX, activated by the '--enable-aix-soname' configure flag. As discussed in the Bugzilla, I think this is a good feature and a nice trick

Re: [patch][RFC] Filename based shared library versioning on AIX

2012-11-05 Thread David Edelsohn
On Mon, Nov 5, 2012 at 1:10 PM, Michael Haubenwallner michael.haubenwall...@salomon.at wrote: Well, as long as the old sharedlibs were not created as standalone shared objects (lib.so), this is similar to a normal soname-bump on AIX, in that it is still possible for the package manager to

Re: [PATCH, middle-end]: Fix mode-switching MODE_EXIT check with __builtin_apply/__builtin_return

2012-11-05 Thread Uros Bizjak
On Mon, Nov 5, 2012 at 7:05 PM, Eric Botcazou ebotca...@adacore.com wrote: This sequence breaks assumption in mode-switching.c, that final function return register load operates in MODE_EXIT mode and triggere following code: for (j = n_entities - 1; j = 0; j--)

Re: [PATCH] Do not change scope for unknown locations

2012-11-05 Thread Eric Botcazou
Because we do want location and its block to be tightly coupled. I may want to add an assertion that if location is known, the block should *not* be NULL. Yes, that would be a first step towards always continuing. But that's 4.9 material, let's install the posted patch as-is for 4.8. --

[patch][google/gcc-4_7] Extend expiration date for pr54127 (issue6817091)

2012-11-05 Thread Paul Pluzhnikov
Greetings, This patch is for google/gcc-4_7 branch. Thanks, 2012-11-05 Paul Pluzhnikov ppluzhni...@google.com * contrib/testsuite-management/powerpc-grtev3-linux-gnu.xfail: extend expiration date for pr54127. Index:

Re: [PATCH, middle-end]: Fix mode-switching MODE_EXIT check with __builtin_apply/__builtin_return

2012-11-05 Thread Uros Bizjak
On Mon, Nov 5, 2012 at 7:34 PM, Uros Bizjak ubiz...@gmail.com wrote: On Mon, Nov 5, 2012 at 7:05 PM, Eric Botcazou ebotca...@adacore.com wrote: This sequence breaks assumption in mode-switching.c, that final function return register load operates in MODE_EXIT mode and triggere following code:

Re: [patch][google/gcc-4_7] Extend expiration date for pr54127 (issue6817091)

2012-11-05 Thread Ollie Wild
OK. Ollie On Mon, Nov 5, 2012 at 12:40 PM, Paul Pluzhnikov ppluzhni...@google.com wrote: Greetings, This patch is for google/gcc-4_7 branch. Thanks, 2012-11-05 Paul Pluzhnikov ppluzhni...@google.com * contrib/testsuite-management/powerpc-grtev3-linux-gnu.xfail:

Re: [x86] Fix gcc.c-torture/compile/20080806-1.c failures

2012-11-05 Thread Richard Sandiford
Uros Bizjak ubiz...@gmail.com writes: On Mon, Nov 5, 2012 at 9:30 AM, Richard Sandiford rdsandif...@googlemail.com wrote: gcc/ PR target/55204 * config/i386/i386.c (ix86_address_subreg_operand): Remove stack pointer check. (print_reg): Use true_regnum rather

Re: [PATCH, middle-end]: Fix mode-switching MODE_EXIT check with __builtin_apply/__builtin_return

2012-11-05 Thread Uros Bizjak
On Mon, Nov 5, 2012 at 7:43 PM, Uros Bizjak ubiz...@gmail.com wrote: As discussed above, modes of loads, generated from __builtin_apply have no connection to function return mode. mode-switching.c does detect __builtin_apply situation and raises maybe_builtin_apply flag, but doesn't use it to

Re: [PATCH/MIPS] Use ins/dins instruction when written manually

2012-11-05 Thread Richard Sandiford
Hi Andrew, Andrew Pinski andrew.pin...@caviumnetworks.com writes: On Fri, Oct 5, 2012 at 8:43 PM, Andrew Pinski andrew.pin...@caviumnetworks.com wrote: On Sun, Aug 19, 2012 at 10:13 AM, Richard Sandiford rdsandif...@googlemail.com wrote: Andrew Pinski andrew.pin...@caviumnetworks.com writes:

[patch, testsuite, mips] Fix gcc.dg/torture/mips-sdata-1.c

2012-11-05 Thread Steve Ellcey
The test gcc.dg/torture/mips-sdata-1.c fails when compiled with -fno-fat-lto-objects because when that option is used no assembly code is output and thus the scan does not find the '.sdata' line. Checked with the mips-mti-elf target. OK to checkin? Steve Ellcey sell...@mips.com 2012-11-05

Re: [PATCH/MIPS] Use ins/dins instruction when written manually

2012-11-05 Thread Andrew Pinski
On Mon, 2012-11-05 at 19:19 +, Richard Sandiford wrote: Hi Andrew, Andrew Pinski andrew.pin...@caviumnetworks.com writes: On Fri, Oct 5, 2012 at 8:43 PM, Andrew Pinski andrew.pin...@caviumnetworks.com wrote: On Sun, Aug 19, 2012 at 10:13 AM, Richard Sandiford

[v3] libstdc++/55215

2012-11-05 Thread Paolo Carlini
Hi, tested x86_64-linux, committed to mainline (and soon 4_7-branch too). Thanks, Paolo. // 2012-11-05 Paolo Carlini paolo.carl...@oracle.com PR libstdc++/55215 * include/bits/random.tcc (mersenne_twister_engine::seed(_Sseq)): Assign state_size to

Re: New badness metric for inliner

2012-11-05 Thread David Miller
This broke the bootstrap on sparc: /home/davem/src/GIT/GCC/build-sparc32-linux/./prev-gcc/g++ -B/home/davem/src/GIT/GCC/build-sparc32\ -linux/./prev-gcc/ -B/usr/local/sparc-unknown-linux-gnu/bin/ -nostdinc++ -B/home/davem/src/GIT/GCC\

Re: [PATCH, middle-end]: Fix mode-switching MODE_EXIT check with __builtin_apply/__builtin_return

2012-11-05 Thread Eric Botcazou
Unfortunately the proposed patch fails the testcase from PR41993: --cut here-- short retframe_short (void *rframe) { __builtin_return (rframe); } OK, so that's not only an issue with the mode of the return register, but with the return register itself. Then the original patch is OK

Re: [PATCH] Vzeroupper placement/47440

2012-11-05 Thread Uros Bizjak
Hello! 2012-11-04 Vladimir Yakovlev vladimir.b.yakov...@intel.com * mode-switching.c (create_pre_exit): Added code for maybe_builtin_apply case. The part above is already committed. * config/i386/i386-protos.h (emit_i387_cw_initialization): Deleted. (emit_vzero):

[google] Add attributes: always_patch_for_instrumentation and never_patch_for_instrumentation (issue6821051)

2012-11-05 Thread Harshit Chopra
2012-11-05 Harshit Chopra hars...@google.com * gcc/c-family/c-common.c (handle_always_patch_for_instrumentation_attribute): Handle always_patch_for_instrumentation attribute and turn inlining off for the function. (handle_never_patch_for_instrumentation_attribute): Handle

Re: [google] Add attributes: always_patch_for_instrumentation and never_patch_for_instrumentation (issue6821051)

2012-11-05 Thread Harshit Chopra
Thanks David for the review. My comments are inline. On Sat, Nov 3, 2012 at 12:38 PM, Xinliang David Li davi...@google.com wrote: Harshit, Nov 5 is the gcc48 cutoff date. If you want to have the x-ray instrumentation feature into this release, you will need to port your patch and submit for

Re: [PATCH 08/10] Factorize condition insertion code out of build_check_stmt

2012-11-05 Thread Dodji Seketeli
Jakub Jelinek ja...@redhat.com writes: On Sat, Nov 03, 2012 at 12:03:45AM +0100, Dodji Seketeli wrote: + int fallthrough_probability = +then_more_likely_p +? PROB_VERY_UNLIKELY +: PROB_ALWAYS - PROB_VERY_UNLIKELY; Just a formatting nit, I think = needs to go on the next line,

Re: [google] Add attributes: always_patch_for_instrumentation and never_patch_for_instrumentation (issue6821051)

2012-11-05 Thread Xinliang David Li
It does not hurt to submit the patch for review -- you need to provide more background and motivation for this work 1) comparison with -finstrument-functions (runtime overhead etc) 2) use model difference (production binary ..) 3) Interesting examples of use cases (with graphs). thanks, David

Re: [patch, testsuite, mips] Fix gcc.dg/torture/mips-sdata-1.c

2012-11-05 Thread Richard Sandiford
Steve Ellcey sell...@mips.com writes: diff --git a/gcc/testsuite/gcc.dg/torture/mips-sdata-1.c b/gcc/testsuite/gcc.dg/torture/mips-sdata-1.c index 8ffd4d8..53c9e4f 100644 --- a/gcc/testsuite/gcc.dg/torture/mips-sdata-1.c +++ b/gcc/testsuite/gcc.dg/torture/mips-sdata-1.c @@ -1,6 +1,7 @@

Re: [patch, testsuite, mips] Fix gcc.dg/torture/mips-sdata-1.c

2012-11-05 Thread Richard Sandiford
Richard Sandiford rdsandif...@googlemail.com writes: Steve Ellcey sell...@mips.com writes: diff --git a/gcc/testsuite/gcc.dg/torture/mips-sdata-1.c b/gcc/testsuite/gcc.dg/torture/mips-sdata-1.c index 8ffd4d8..53c9e4f 100644 --- a/gcc/testsuite/gcc.dg/torture/mips-sdata-1.c +++

[PATCH] fix libgomp.c++/pr24455.C failures on darwin

2012-11-05 Thread Jack Howarth
Currently the following testcases are failing on x86_64-apple-darwin11/12... FAIL: libgomp.c++/pr24455.C -O0 (test for excess errors) WARNING: libgomp.c++/pr24455.C -O0 compilation failed to produce executable FAIL: libgomp.c++/pr24455.C -O1 (test for excess errors) WARNING:

Re: [PATCH Version 2][RFA]Improving register pressure directed hoist

2012-11-05 Thread Jeff Law
On 11/02/2012 02:34 AM, Bin Cheng wrote: Also I don't understand why the bogus patch can catch more hoist opportunities and improve code size, so please help if you have any idea about this. Well, perturbing the code, particularly in a way which is supposed to change the amount of register

Re: exception safety validation

2012-11-05 Thread François Dumont
I apply the attached patch. 2012-10-05 François Dumont fdum...@gcc.gnu.org * include/ext/throw_allocator.h (__throw_value_base): Add move semantic, not throwing. (__throw_value_limit): Likewise. (__throw_value_random): Likewise. * testsuite/util/exception/safety.h: Add

Re: [v3] Fix allocator-aware container requirements for forward_list

2012-11-05 Thread Jonathan Wakely
This updates the debug-mode and profile-mode forward_list code to match my recent allocator changes to the default mode. It also adds checking for unswappable allocators, and fixes some tests that those new checks showed were broken. * include/profile/forward_list: Update to meet

Re: [patch, testsuite, mips] Fix gcc.dg/torture/mips-sdata-1.c

2012-11-05 Thread Steve Ellcey
On Mon, 2012-11-05 at 20:38 +, Richard Sandiford wrote: This sort of thing should usually be handled automatically by scan-assembler, and is for me: /foo/gcc/xgcc -B/foo/gcc/ /bar/gcc/testsuite/gcc.dg/torture/mips-sdata-1.c -fno-diagnostics-show-caret -O2 -flto -fuse-linker-plugin

[patch] Fix PR tree-optimization/54986

2012-11-05 Thread Eric Botcazou
Hi, this is a regression present on the 4.7 branch and caused by my fix for another regression: http://gcc.gnu.org/ml/gcc-patches/2012-07/msg00825.html It turns out that canonicalize_constructor_val has side effects: on the 4.7 branch, it calls add_referenced_var on the base variable of an

Re: [patch] Fix PR tree-optimization/54986

2012-11-05 Thread Jakub Jelinek
On Mon, Nov 05, 2012 at 10:16:31PM +0100, Eric Botcazou wrote: Bootstrapped/regtested on x86_64-suse-linux, OK for mainline and 4.7 branch? 2012-11-05 Eric Botcazou ebotca...@adacore.com PR tree-optimization/54986 * gimple-fold.c (canonicalize_constructor_val): Strip again

Re: [patch, testsuite, mips] Fix gcc.dg/torture/mips-sdata-1.c

2012-11-05 Thread Richard Sandiford
Steve Ellcey sell...@mips.com writes: On Mon, 2012-11-05 at 20:38 +, Richard Sandiford wrote: This sort of thing should usually be handled automatically by scan-assembler, and is for me: /foo/gcc/xgcc -B/foo/gcc/ /bar/gcc/testsuite/gcc.dg/torture/mips-sdata-1.c

Re: [patch, testsuite, mips] Fix gcc.dg/torture/mips-sdata-1.c

2012-11-05 Thread Steve Ellcey
On Mon, 2012-11-05 at 21:28 +, Richard Sandiford wrote: Does it work for you if you run it separately? E.g. with: make check-gcc RUNTESTFLAGS=dg-torture.exp=mips-sdata-1.c Richard Hm, I configured GCC with '--enable-languages=c,c++', I think this might set ENABLE_LTO to '0' and

[v3] 55028

2012-11-05 Thread Benjamin De Kosnik
Missing exports for unordered_* containers in debug mode. The patches between mainline/4.7 are a slight bit different, but equivalent. tested x86/linux tested x86/linux --enable-symvers=gnu-versioned-namespace 2012-11-05 Benjamin Kosnik b...@redhat.com Oleg Smolsky

RE: [Ping]FW: [PATCH] Cilk Plus merging to trunk (2 of n)

2012-11-05 Thread Iyer, Balaji V
Hello Joseph, Here is the fixed patch with all your changes and the ChangeLog entries below. gcc/ChangeLog 2012-11-05 Balaji V. Iyer balaji.v.i...@intel.com * Makefile.in (C_COMMON_OBJS): Added c-family/array-notation-common.o. * doc/passes.texi (Cilk Plus

Re: patch to fix PR55151

2012-11-05 Thread H.J. Lu
On Mon, Nov 5, 2012 at 8:43 AM, Vladimir Makarov vmaka...@redhat.com wrote: The following patch fixes PR55151. The patch affects a sensitive part of LRA code. So it took some time to find a PR solution. For the test there were to many reloads into hard registers for an insn and LRA failed

Re: RFC: PR middle-end/55142: Check Pmode instead of ptr_mode for address mode

2012-11-05 Thread H.J. Lu
On Sat, Nov 3, 2012 at 1:14 AM, H.J. Lu hjl.to...@gmail.com wrote: Hi, The testcase shows -O -mx32 -maddress-mode=long -fPIC -S generates; x.i:22:37: internal compiler error: in plus_constant, at explow.c:88 info[0x6eff - dyn-d_tag + 12] = dyn; expand_expr_real_2 has /* No

[Cilkplus] Merged with trunk at revision 193151

2012-11-05 Thread Iyer, Balaji V
Cilk Plus branch was merged with trunk at revision 193151. Committed as revision 193192. Thanks, Balaji V. Iyer.

[patch] PR55191 - ICE in tree-ssa-pre.c due to missing fake edge for an infinite loop

2012-11-05 Thread Steven Bosscher
Hello, I hadn't expected that cfganal.c's reverse-CFG DFS would actually depend on the order of the basic blocks. The attached patch fixes that small oversight... Bootstrappedtested on powerpc64-unknown-linux-gnu. OK for trunk? Ciao! Steven * cfganal.c (connect_infinite_loops_to_exit):

[Patch, Fortran, OOP] PR 54917: [4.7/4.8 Regression] TRANSFER on polymorphic variable causes ICE

2012-11-05 Thread Janus Weil
Hi all, the attached patch implements support for polymorphic arguments to TRANSFER. For details and discussion see the PR. The patch was regtested successfully on x86_64-unknown-linux-gnu. Ok for trunk? Btw, as part of the PR is a regression in 4.7 and trunk, I would like to backport the

Re: New badness metric for inliner

2012-11-05 Thread Jan Hubicka
This broke the bootstrap on sparc: /home/davem/src/GIT/GCC/build-sparc32-linux/./prev-gcc/g++ -B/home/davem/src/GIT/GCC/build-sparc32\ -linux/./prev-gcc/ -B/usr/local/sparc-unknown-linux-gnu/bin/ -nostdinc++ -B/home/davem/src/GIT/GCC\

Re: [RFA:] PR55186 - gcc.dg/const-uniq-1.c fails due to vector not in the constant pool

2012-11-05 Thread Hans-Peter Nilsson
From: Eric Botcazou ebotca...@adacore.com Date: Mon, 5 Nov 2012 15:29:11 +0100 But, for cris-elf (and reasonably the same for other targets) there might not be such a constant-pool entry in the first place: the vectors are too short to rule out piecewise initialization as optimal for

Re: New badness metric for inliner

2012-11-05 Thread David Miller
From: Jan Hubicka hubi...@ucw.cz Date: Mon, 5 Nov 2012 23:19:16 +0100 Is line 784 for you gcc_checking_assert (uninlined_call_time = 0);? Yes.

Re: [patch, testsuite, mips] Fix gcc.dg/torture/mips-sdata-1.c

2012-11-05 Thread Steve Ellcey
On Mon, 2012-11-05 at 13:40 -0800, Steve Ellcey wrote: Hm, I configured GCC with '--enable-languages=c,c++', I think this might set ENABLE_LTO to '0' and cause check_effective_target_lto to return FALSE and cause the -ffat-lto-objects flag to not be added. Of course this raises the question

Re: [patch, testsuite, mips] Fix gcc.dg/torture/mips-sdata-1.c

2012-11-05 Thread Richard Sandiford
Steve Ellcey sell...@mips.com writes: I am not sure how ld is supposed to find crt0.o in the uninstalled setup, it exists in obj-mips-mti-elf/newlib/mips-mti-elf/libgloss/mips. The newlib and libgloss flags are usually added by the dejagnu baseboard file. E.g. for mips-sim.exp: set_board_info

[patch RFA middle-end] Fix PR target/41993

2012-11-05 Thread Kaz Kojima
Hi, The attached patch is to solve PR target/41993 which will affect targets using MODE_EXIT. Without it, we can't find all return registers for __builtin_return in mode-switching.c:create_pre_exit. See the trail #4 by Uros in the PR for the details. The patch is tested with bootstrap and

Re: [patch, testsuite, mips] Fix gcc.dg/torture/mips-sdata-1.c

2012-11-05 Thread Steve Ellcey
On Mon, 2012-11-05 at 22:55 +, Richard Sandiford wrote: Steve Ellcey sell...@mips.com writes: I am not sure how ld is supposed to find crt0.o in the uninstalled setup, it exists in obj-mips-mti-elf/newlib/mips-mti-elf/libgloss/mips. The newlib and libgloss flags are usually added by

Re: RFC: PR middle-end/55142: Check Pmode instead of ptr_mode for address mode

2012-11-05 Thread Richard Sandiford
H.J. Lu hjl.to...@gmail.com writes: On Sat, Nov 3, 2012 at 1:14 AM, H.J. Lu hjl.to...@gmail.com wrote: Hi, The testcase shows -O -mx32 -maddress-mode=long -fPIC -S generates; x.i:22:37: internal compiler error: in plus_constant, at explow.c:88 info[0x6eff - dyn-d_tag + 12] = dyn;

Re: [Dwarf Fission] Implement Fission Proposal (issue6305113)

2012-11-05 Thread Cary Coutant
+/* %:replace-extension spec function. Replaces the extension of the + first argument with the second argument. */ + +const char * +replace_extension_spec_func (int argc, const char **argv) +{ + char *name; + char *p; + char *result; + + if (argc != 2) +fatal_error (too few

  1   2   >