Re: Minor fixes to ipa-inline-analysis.c

2012-11-25 Thread Richard Biener
On Wed, Nov 7, 2012 at 10:25 AM, Jan Hubicka hubi...@ucw.cz wrote: Hi, while analyzing c-ray I noticed two issues. First is that I originally set number of size/time entries to 32. Once we reach this limit we conservatively account everything as unconditional. This limit is not met on

[patch] reorg.c janitor patch: use return hard_reg_set_intersect_p

2012-11-25 Thread Steven Bosscher
One to be committed as obvious. * reorg.c (resource_conflicts_p): Use hard_reg_set_intersect_p. Index: reorg.c === --- reorg.c (revision 193787) +++ reorg.c (working copy) @@ -292,18 +292,7 @@ resource_conflicts_p

[patch] reorg.c janitor patch 2: handle DEBUG_INSN

2012-11-25 Thread Steven Bosscher
Hi, stop_search_p will reach the default case on DEBUG_INSN, and the default case is gcc_unreachable(). I suppose this means nobody is using DWARF3+ on a dbr_sched target, it can't possibly ever have worked. Eric? Anyway, also obvious: Don't stop on DEBUG_INSNs. Ciao! Steven * reorg.c

Re: Reduce complette unrolling peeling limits

2012-11-25 Thread Dominique Dhumieres
... I believe I posted a patch? Yes: http://gcc.gnu.org/ml/gcc-patches/2012-11/msg01799.html I have found another fall out: I have some avatars of the polyhedron tests where the REAL(8) have been replaced with REAL(10). Some of them are now Should I open a new PR for that? Cheers, Dominique

Re: Reduce complette unrolling peeling limits

2012-11-25 Thread Dominique Dhumieres
My mailer has eaten a line in my previous mail. One should read: I have found another fall out: I have some avatars of the polyhedron tests where the REAL(8) have been replaced with REAL(10). Some of them are now ~50% slower with the new value of max-completely-peeled-insns. Should I open a new

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

2012-11-25 Thread Richard Biener
On Thu, Nov 15, 2012 at 5:46 PM, Eric Botcazou ebotca...@adacore.com wrote: But UNKNOWN_LOCATION is effectively wrong as well. If other optimizations move the statements above the inserted instruction, then the new instruction ends up inheriting whatever location happens to be in the previous

Re: [patch] Remove DECL_RESTRICTED_P

2012-11-25 Thread Richard Biener
On Fri, Nov 16, 2012 at 6:15 PM, Eric Botcazou ebotca...@adacore.com wrote: Hi, that's not really a bugfix, but I don't see the point in keeping this flag, which was already dead in 4.7, one more release. Tested on x86_64-suse-linux, OK for the mainline? Ok. Thanks, Richard. 2012-11-16

Re: VEC re-write [patch 01/25]

2012-11-25 Thread Richard Biener
On Sun, Nov 18, 2012 at 6:12 PM, Diego Novillo dnovi...@google.com wrote: On Sun, Nov 18, 2012 at 12:05 PM, Hans-Peter Nilsson h...@bitrange.com wrote: On Sat, 17 Nov 2012, Diego Novillo wrote: I have now committed all 25 parts of this patch as rev 193595. Please CC me on any problems that

Re: *ping* [patch, fortran] PR 30146, errors for INTENT(OUT) and INTENT(INOUT) for DO loop variables

2012-11-25 Thread Thomas Koenig
Am 17.11.2012 10:09, schrieb Thomas Koenig: I wrote: Attached is the new version of the patch, regression-tested. http://gcc.gnu.org/ml/gcc-patches/2012-11/msg00836.html Thanks for the review! OK for trunk? Ping? Ping**2? Thomas

Re: [PATCH] Fix VRP MULT_EXPR handling (PR tree-optimization/54471)

2012-11-25 Thread Richard Biener
On Wed, Nov 21, 2012 at 9:38 PM, Jakub Jelinek ja...@redhat.com wrote: Hi! If a type has 2 * HWI precision, sizem1 is maximum double_int (all ones) and thus size = sizem1 + double_int_one overflows into 0. If either min0 or min1 is also zero, we might wrongly canonicalize the range into a

Re: [PATCH]: Fix compiler segfault failure in cd_dce pass

2012-11-25 Thread Richard Biener
On Wed, Nov 21, 2012 at 10:35 PM, Xinliang David Li davi...@google.com wrote: In compiling one of the very large C++ source, the compiler hit a segfault in cddce -- the ssa_name of a vuse operand has a null def stmt. The def stmt was a PHI node, and later got removed by the phicprop pass (in

Re: [tsan] Don't instrument clobber stmts, minor cleanups

2012-11-25 Thread Richard Biener
On Fri, Nov 23, 2012 at 2:39 PM, Jakub Jelinek ja...@redhat.com wrote: Hi! var ={v} {CLOBBER}; stmts shouldn't be tsan instrumented, those aren't stores, just markups that var's scope ends. Additionally this patch removes the IMHO unneeded TODO_update_address_taken (discussed earlier

[patch] reorg.c janitor patch 3: remove rare_destination()

2012-11-25 Thread Steven Bosscher
Hello, reorg.c:rare_destination() tries to determine whether a given insn is a likely destination of a branch. To make this determination, it walks the insns chain from insn and stops at barriers, labels, return insns, or if more than 10 jumps have been followed. The function is supposed to

Re: *ping* [patch, fortran] PR 30146, errors for INTENT(OUT) and INTENT(INOUT) for DO loop variables

2012-11-25 Thread Steven Bosscher
On Sun, Nov 25, 2012 at 2:11 PM, Thomas Koenig wrote: Ping**2? This is OK. Ciao! Steven

Re: VEC re-write [patch 01/25]

2012-11-25 Thread Diego Novillo
On Sun, Nov 25, 2012 at 8:11 AM, Richard Biener richard.guent...@gmail.com wrote: Doesn't that mean that a non-GCC host compiler might fail to build 4.8 during stage1? Yes. Fixed last week. vec needed to become a pure POD. So sad. Diego.

[patch i386} Fix PR 55171 - [4.7/4.8 Regression] incorrect virtual thunk on mingw

2012-11-25 Thread Kai Tietz
Hi, the following patch adjusts used registers for thiscall-calling convention thunks, so that there aren't register-collisions. Issue was that thiscall-convention has different register-usage as regparam, but it wasn't handled. ChangeLog 2012-11-25 Kai Tietz PR target/55171

Re: [RFA 1/8] validate_failures.py: add TestResult ordinal

2012-11-25 Thread Diego Novillo
On Sat, Nov 24, 2012 at 5:41 PM, Doug Evans d...@google.com wrote: 2012-11-24 Doug Evans d...@google.com * testsuite-management/validate_failures.py: Record ordinal with TestResult. OK. Diego.

Re: [RFA 2/8] validate_failures.py: use target_alias

2012-11-25 Thread Diego Novillo
On Sat, Nov 24, 2012 at 5:43 PM, Doug Evans d...@google.com wrote: 2012-11-24 Doug Evans d...@google.com * testsuite-management/validate_failures.py: Use target_alias instead of target. Minor whitespace changes. OK. Diego.

[patch libgcc]: Fix PR target/55445 Always defined __SEH__ when build from trunk

2012-11-25 Thread Kai Tietz
Hi, the issue here is that the predefined macro __SEH__ does just indicate that SEH-infrastructure is present. It doesn't mean that SEH is used as exception-mechansim. Therefore the checks in libgcc's (and as followup in libstdc++'s) eh exception-mechansim for SEH were not regarding that there

Re: [RFA 3/8] validate_failures.py: pass options.results for clean build case

2012-11-25 Thread Diego Novillo
On Sat, Nov 24, 2012 at 5:47 PM, Doug Evans d...@google.com wrote: Hi. This third patch passes options.results to GetSumFiles when fetching the results for the clean build. It is useful in my use cases, but I'm not sure it's useful for upstream. [An alternative is to add another option to

[patch libstdc++]: Fix PR target/55445 Always defined __SEH__ when build from trunk

2012-11-25 Thread Kai Tietz
Hi, this patch fixes used exception-mechanism for SEH-enabled targets, which are requesting for SjLj-exception-mechanism. See also patch for libgcc. ChangeLog 2012-11-25 Kai Tietz PR target/55445 * libsupc++/eh_personaltity.cc (__SEH__): Additional check for not being

Re: [RFA 5/8] validate_failures.py: make options a global

2012-11-25 Thread Diego Novillo
On Sat, Nov 24, 2012 at 5:52 PM, Doug Evans d...@google.com wrote: Hi. This fifth patch makes options a global variable. As validate_failures.py becomes more complex, passing it around everywhere becomes cumbersome with no real gain. Ok to check in? 2012-11-24 Doug Evans d...@google.com

Re: [RFA 7/8] validate_failures.py: New options --manifest_subdir, --manifest_name

2012-11-25 Thread Diego Novillo
On Sat, Nov 24, 2012 at 5:58 PM, Doug Evans d...@google.com wrote: Hi. This seventh patch adds new options --manifest_subdir, --manifest_name. Useful when using validate_failures.py with a different tool, instead of gcc. Ok to check in? 2012-11-24 Doug Evans d...@google.com *

Re: [RFA 8/8] validate_failures.py: New directives @include, @remove.

2012-11-25 Thread Diego Novillo
On Sat, Nov 24, 2012 at 6:02 PM, Doug Evans d...@google.com wrote: 2012-11-24 Doug Evans d...@google.com * testsuite-management/validate_failures.py: Add support for @include, @remove directives in manifest files. OK. Thanks for all the cleanups and features! Diego.

Re: [patch] reorg.c janitor patch 3: remove rare_destination()

2012-11-25 Thread Richard Sandiford
Steven Bosscher stevenb@gmail.com writes: Hello, reorg.c:rare_destination() tries to determine whether a given insn is a likely destination of a branch. To make this determination, it walks the insns chain from insn and stops at barriers, labels, return insns, or if more than 10 jumps

Re: [patch] reorg.c janitor patch 3: remove rare_destination()

2012-11-25 Thread Steven Bosscher
On Sun, Nov 25, 2012 at 6:13 PM, Richard Sandiford wrote: Steven Bosscher writes: Hello, reorg.c:rare_destination() tries to determine whether a given insn is a likely destination of a branch. To make this determination, it walks the insns chain from insn and stops at barriers, labels,

Re: [Patch, Fortran, OOP] PR 54881: [4.8 Regression] [OOP] ICE in fold_convert_loc, at fold-const.c:2016

2012-11-25 Thread Mikael Morin
Le 11/10/2012 23:49, Janus Weil a écrit : Hi all, here is an OOP patch for the above PR, which has two disconnected parts: 1) It fixes a problem with ASSOCIATED, when it is fed a CLASS-valued function as argument (i.e. the ICE in the bug title). This is the trans-intrinsic part of the patch.

[Patch, libquadmath, committed] PR 55462 - rounding fix

2012-11-25 Thread Tobias Burnus
Committed as Rev. 193796. I forgot to change the condition after changing how the rounding mode is detected. Tobias Index: libquadmath/ChangeLog === --- libquadmath/ChangeLog (Revision 193792) +++ libquadmath/ChangeLog

Re: [PATCH] Section anchors and thread-local storage

2012-11-25 Thread Jakub Jelinek
On Sun, Nov 25, 2012 at 02:02:05PM -0500, David Edelsohn wrote: I have been working to enable native thread-local storage on AIX. One problem I encountered is the AIX assembler has difficulty with the anchor symbol for TLS CSECTs. While the section anchors machinery uses a separate pool for

[PATCH, i386]: Fix handling of TARGET_SSE_TYPELESS_STORES in unaligned move RTXes

2012-11-25 Thread Uros Bizjak
Hello! When unaligned moves were split to unaligned store/loads, mode calculation didn't get updated. Attached patch fixes this oversight. 2012-11-25 Uros Bizjak ubiz...@gmail.com * config/i386/sse.md (sse_loadussemodesuffixavxsizesuffix): Do not depend on

Re: Fix twolf -funroll-loops -O3 miscompilation (a semi-latent web.c bug)

2012-11-25 Thread Steven Bosscher
On Sat, Nov 24, 2012 at 2:09 AM, Steven Bosscher wrote: On Fri, Nov 23, 2012 at 11:45 PM, Steven Bosscher wrote: Removing the note is easier but it may hurt optimization later on: CSE2 puts the note back in but this introduces a pass ordering dependency. Perhaps that's not a big problem, but

Re: [patch] reorg.c janitor patch 2: handle DEBUG_INSN

2012-11-25 Thread Hans-Peter Nilsson
On Sun, 25 Nov 2012, Steven Bosscher wrote: stop_search_p will reach the default case on DEBUG_INSN, and the default case is gcc_unreachable(). I suppose this means nobody is using DWARF3+ on a dbr_sched target, it can't possibly ever have worked. Eric? There must be something else

[PATCH, i386]: Add -mtune=generic to gcc.target/i386/sw-1.c

2012-11-25 Thread Uros Bizjak
Hello! Otherwise, we can generate real memcpy call for certain targets (corei7) that interferes with shrink-wrapping. 2012-11-25 Uros Bizjak ubiz...@gmail.com * gcc.target/i386/sw-1.c (dg-options): Add -mtune=generic. Tested on x86_64-linux-gnu, committed to mainline SVN. Uros.

Re: [patch] reorg.c janitor patch 2: handle DEBUG_INSN

2012-11-25 Thread Jakub Jelinek
On Sun, Nov 25, 2012 at 12:59:48PM +0100, Steven Bosscher wrote: stop_search_p will reach the default case on DEBUG_INSN, and the default case is gcc_unreachable(). I suppose this means nobody is using DWARF3+ on a dbr_sched target, it can't possibly ever have worked. Eric? Isn't dbr

Re: [patch] reorg.c janitor patch 2: handle DEBUG_INSN

2012-11-25 Thread Steven Bosscher
On Sun, Nov 25, 2012 at 11:09 PM, Jakub Jelinek ja...@redhat.com wrote: On Sun, Nov 25, 2012 at 12:59:48PM +0100, Steven Bosscher wrote: stop_search_p will reach the default case on DEBUG_INSN, and the default case is gcc_unreachable(). I suppose this means nobody is using DWARF3+ on a

Re: Fix twolf -funroll-loops -O3 miscompilation (a semi-latent web.c bug)

2012-11-25 Thread Steven Bosscher
On Sun, Nov 25, 2012 at 9:44 PM, Steven Bosscher wrote: On Sat, Nov 24, 2012 at 2:09 AM, Steven Bosscher wrote: On Fri, Nov 23, 2012 at 11:45 PM, Steven Bosscher wrote: Removing the note is easier but it may hurt optimization later on: CSE2 puts the note back in but this introduces a pass

Re: Rewrite lto-symtab to work on symbol table

2012-11-25 Thread H.J. Lu
On Sun, Nov 25, 2012 at 9:55 AM, H.J. Lu hjl.to...@gmail.com wrote: On Mon, Sep 24, 2012 at 9:10 AM, H.J. Lu hjl.to...@gmail.com wrote: On Mon, Sep 24, 2012 at 8:50 AM, Martin Jambor mjam...@suse.cz wrote: Hi, On Tue, Sep 18, 2012 at 03:35:45PM +0200, Jan Hubicka wrote: Hi, this patch

Re: [PATCH] Section anchors and thread-local storage

2012-11-25 Thread David Edelsohn
On Sun, Nov 25, 2012 at 3:07 PM, Jakub Jelinek ja...@redhat.com wrote: Do you want to handle TLS_MODEL_EMULATED that way? If not, the test should be DECL_THREAD_LOCAL_P (decl) (I believe you want that), if yes, perhaps DECL_TLS_MODEL (decl) != TLS_MODEL_NONE. Thanks for pointing this out. I