[AARCH64] AArch64 backport to 4.7

2012-09-12 Thread Sofiane Naci
[AARCH64] AArch64 backport to 4.7. This series of patches implements a back port of the AArch64 backend to gcc-4.7. This patch series is not intended to be applied directly to the gcc-4_7-branch branch. We have pushed a new branch into SVN to host this back port, located at:

[SH] Correct address cost estimations

2012-09-12 Thread Oleg Endo
Hello, This corrects the address cost estimations for SH. Tested on rev 191161 with make -k check RUNTESTFLAGS=--target_board=sh-sim \{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb} and no new failures. With this applied CSiBE shows a total size decrease of 4668 bytes for '-O2

Re: Obsolete picochip-* in 4.7.2+

2012-09-12 Thread Richard Guenther
On Tue, Sep 11, 2012 at 5:21 PM, Jakub Jelinek ja...@redhat.com wrote: Hi! As discussed on IRC, the picochip-* port doesn't have an active maintainer anymore, this patch adds it to deprecated ports for 4.7.2+ so that it can be removed in GCC 4.8 unless somebody steps up to maintain it. Ok

Re: [PATCH,mmix] convert to constraints.md

2012-09-12 Thread Nathan Froyd
- Original Message - Nathan, again thanks. There are a few minor tweaks compared to your version: Thanks for fixing this up! - Keeping old layout of mmix_reg_or_8bit_operand. That looked like a spurious change and I prefer the ior construct to the if_then_else. ISTR without

Re: [C++ Patch] Remove uses of ATTRIBUTE_UNUSED in the function parameters

2012-09-12 Thread Richard Guenther
On Tue, Sep 11, 2012 at 5:37 PM, Jakub Jelinek ja...@redhat.com wrote: On Tue, Sep 11, 2012 at 05:29:12PM +0200, Paolo Carlini wrote: PS: slightly interesting, in a couple of cases - write_unnamed_type_name, wrap_cleanups_r - the parameters were actually used. Just a general comment, often

Re: Backtrace library [3/3]

2012-09-12 Thread Florian Weimer
On 09/12/2012 12:55 AM, Ian Lance Taylor wrote: I have finished the initial implementation of the backtrace library I proposed at http://gcc.gnu.org/ml/gcc/2012-08/msg00317.html . I've separated the work into three patches. These patches only implement the backtrace library itself; actual use

Re: [PATCH] Combine location with block using block_locations

2012-09-12 Thread Richard Guenther
On Tue, Sep 11, 2012 at 5:32 PM, Michael Matz m...@suse.de wrote: Hi, On Tue, 11 Sep 2012, Dehao Chen wrote: Looks like we have two choices: 1. Stream out block info, and use LTO_SET_PREVAIL for TREE_CHAIN(t) This will actually not work correctly in some cases. The problem is, if the

Re: [PATCH] Combine location with block using block_locations

2012-09-12 Thread Richard Guenther
On Wed, Sep 12, 2012 at 7:06 AM, Dehao Chen de...@google.com wrote: Now I think we are facing a more complex problem. The data structure we use to store the location_adhoc_data are file-static in linemap.c in libcpp. These data structures are not guarded by GTY(()). Meanwhile, as we have

[v3, build] Clear hardware capabilities on libstdc++.so with Sun as

2012-09-12 Thread Rainer Orth
Since the use of rdrand was introduced in src/c++11/random.cc, all execution tests involving libstdc++.so.6 fail on Solaris 10 and 11/x86 with a sufficiently recent native assembler that supports rdrand: either Solaris 10/x86 patch 119961-11 or Solaris 11.1 builds (haven't checked which one). The

Re: [Fortran, Patch] PR 54225 - Fix ice-on-invalid-code with * in array refs

2012-09-12 Thread Mikael Morin
On 11/09/2012 07:54, Tobias Burnus wrote: This patch fixes a GCC 4.7/4.8 regression for invalid code. Build and regtested on x86-64-linux. OK for the trunk and 4.7? Yes. PR 53306 is also fixed by your patch according to Dominique, so don't forget to include its testcase and to add the PR

Re: [C++ Patch] Remove uses of ATTRIBUTE_UNUSED in the function parameters

2012-09-12 Thread Paolo Carlini
On 09/12/2012 10:57 AM, Richard Guenther wrote: And note that we have ARG_UNUSED for parameters - to cope with older compilers not handling attributes here too well (I run into this when using gcc 3.3 as host compiler). Ah, thanks, I didn't know that (both the problem and the solution): it

Re: [PATCH,mmix] convert to constraints.md

2012-09-12 Thread Hans-Peter Nilsson
On Wed, 12 Sep 2012, Nathan Froyd wrote: - Original Message - Nathan, again thanks. There are a few minor tweaks compared to your version: Thanks for fixing this up! - Keeping old layout of mmix_reg_or_8bit_operand. That looked like a spurious change and I prefer the ior

[PATCH] Fix PR54553

2012-09-12 Thread Richard Guenther
This makes sure -finline is saved across optimize attribute changes (-O0 changes it). Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. Richard. 2012-09-12 Richard Guenther rguent...@suse.de PR middle-end/54553 * common.opt (finline): Mark with

Re: [PATCH] Add option for dumping to stderr (issue6190057)

2012-09-12 Thread Richard Guenther
On Wed, Sep 12, 2012 at 12:30 PM, Richard Guenther richard.guent...@gmail.com wrote: On Wed, Sep 12, 2012 at 10:12 AM, Sharad Singhai sing...@google.com wrote: Thanks for your comments. Please see my responses inline. On Tue, Sep 11, 2012 at 1:16 PM, Xinliang David Li davi...@google.com

Unreviewed bootstrap patch

2012-09-12 Thread Rainer Orth
This patch Fix Solaris 9/x86 bootstrap http://gcc.gnu.org/ml/gcc-patches/2012-08/msg01494.html has remained unreviewed for 3 weeks. It is necessary to fix Solaris 9/x86 bootstrap after the cxx-conversion merge. Rainer --

Re: [v3, build] Clear hardware capabilities on libstdc++.so with Sun as

2012-09-12 Thread Paolo Bonzini
Il 12/09/2012 11:26, Rainer Orth ha scritto: Since the use of rdrand was introduced in src/c++11/random.cc, all execution tests involving libstdc++.so.6 fail on Solaris 10 and 11/x86 with a sufficiently recent native assembler that supports rdrand: either Solaris 10/x86 patch 119961-11 or

Re: shrink-wrapping duplicates BBs across partitions.

2012-09-12 Thread Christian Bruel
The problem stems from tree-ssa-tail-merge that breaks bb-count, The CFG looks like 2 / \ /6 5 (0) | | 3 - |/ \ | | 7 (1) 8 - | / 4 (1) (in parenthesis the bb-count from gcov) 2 / \ /6 / | | 3 -- | / | | 5 (0) 8 -- |

Re: Unreviewed bootstrap patch

2012-09-12 Thread Richard Guenther
On Wed, 12 Sep 2012, Rainer Orth wrote: This patch Fix Solaris 9/x86 bootstrap http://gcc.gnu.org/ml/gcc-patches/2012-08/msg01494.html has remained unreviewed for 3 weeks. It is necessary to fix Solaris 9/x86 bootstrap after the cxx-conversion merge. Ok. Thanks, Richard.

Re: [PATCH] Combine location with block using block_locations

2012-09-12 Thread Michael Matz
Hi, On Wed, 12 Sep 2012, Richard Guenther wrote: This will actually not work correctly in some cases. The problem is, if the prevailing decl is already part of another chain (say in another block_var list) you would break the current chain. Hence block vars need special handling in

Re: Change double_int calls to new interface.

2012-09-12 Thread Mark Kettenis
Date: Tue, 11 Sep 2012 17:03:39 -0700 From: Ian Lance Taylor i...@google.com On Tue, Sep 11, 2012 at 3:12 PM, Lawrence Crowl cr...@googlers.com wrote: On 9/11/12, Andreas Schwab sch...@linux-m68k.org wrote: Mark Kettenis mark.kette...@xs4all.nl writes: In file included from

Re: [PATCH] Combine location with block using block_locations

2012-09-12 Thread Richard Guenther
On Wed, Sep 12, 2012 at 2:14 PM, Michael Matz m...@suse.de wrote: Hi, On Wed, 12 Sep 2012, Richard Guenther wrote: This will actually not work correctly in some cases. The problem is, if the prevailing decl is already part of another chain (say in another block_var list) you would break

Re: [PATCH] Combine location with block using block_locations

2012-09-12 Thread Michael Matz
Hi, On Wed, 12 Sep 2012, Richard Guenther wrote: It hides a bug. If we replace anything in BLOCK_VARS then the risk is that you generate an infinite chain in some BLOCK_VARS list and thus get infinite loops somewhere in the compiler. That's what I said for using SET_PREVAIL. But

Remove obsolete compatibility notes in vec.h

2012-09-12 Thread Diego Novillo
Committed to trunk. * vec.h: Remove compatibility notes for previous distinction between vectors of objects and vectors of pointers. diff --git a/gcc/vec.h b/gcc/vec.h index 88891d7..8858f6a 100644 --- a/gcc/vec.h +++ b/gcc/vec.h @@ -31,23 +31,6 @@ along with GCC; see the

[PATCH] Fix PR54489 - FRE needing AVAIL_OUT

2012-09-12 Thread Richard Guenther
This removes the need for FRE to compute AVAIL_OUT which can consume an unreasonable amount of memory for testcases like int foo (int a) { int b = 0; #define X if (a) b = b + 1; #define XX X X X X X X X X X X #define XXX XX XX XX XX XX XX XX XX XX XX #define XXX XXX XXX XXX XXX XXX XXX

Re: [PATCH] Combine location with block using block_locations

2012-09-12 Thread Michael Matz
Hi, On Wed, 12 Sep 2012, Michael Matz wrote: Hm, but we shouldn't end up streaming any BLOCKs at this point (nor local TYPE_DECLs). Those are supposed to be in the local function sections only where no fixup for prevailing decls happens. That's true, something is fishy with the

Re: [PATCH] Combine location with block using block_locations

2012-09-12 Thread Martin Jambor
Hi, On Wed, Sep 12, 2012 at 04:17:45PM +0200, Michael Matz wrote: Hi, On Wed, 12 Sep 2012, Michael Matz wrote: Hm, but we shouldn't end up streaming any BLOCKs at this point (nor local TYPE_DECLs). Those are supposed to be in the local function sections only where no fixup for

Re: Backtrace library [1/3]

2012-09-12 Thread Ian Lance Taylor
On Tue, Sep 11, 2012 at 4:36 PM, Lawrence Crowl cr...@googlers.com wrote: On 9/11/12, Ian Lance Taylor i...@google.com wrote: This patch is the interface to and configury of libbacktrace. I've separated these out as the parts of libbacktrace that require the most review. The interface to

Re: [PATCH] Combine location with block using block_locations

2012-09-12 Thread Richard Guenther
On Wed, Sep 12, 2012 at 4:37 PM, Martin Jambor mjam...@suse.cz wrote: Hi, On Wed, Sep 12, 2012 at 04:17:45PM +0200, Michael Matz wrote: Hi, On Wed, 12 Sep 2012, Michael Matz wrote: Hm, but we shouldn't end up streaming any BLOCKs at this point (nor local TYPE_DECLs). Those are

Re: Backtrace library [3/3]

2012-09-12 Thread Ian Lance Taylor
On Tue, Sep 11, 2012 at 10:16 PM, Jakub Jelinek ja...@redhat.com wrote: On Tue, Sep 11, 2012 at 03:55:04PM -0700, Ian Lance Taylor wrote: +/* The DWARF abbreviations for a compilation unit. This structure + only exists while reading the compilation unit. Most DWARF readers + seem to a

Re: [PATCH] Fix PR54489 - FRE needing AVAIL_OUT

2012-09-12 Thread Steven Bosscher
On Wed, Sep 12, 2012 at 4:02 PM, Richard Guenther wrote: for a followup (and I bet sth else than PRE blows up at -O2 as well). Actually, the only thing that really blows up is that enemy of scalability, VRP. With -O2 -fno-tree-{pre,fre,vrp}, the slowest part of the compiler on this test case

Re: Backtrace library [3/3]

2012-09-12 Thread Ian Lance Taylor
On Wed, Sep 12, 2012 at 2:01 AM, Florian Weimer fwei...@redhat.com wrote: +backtrace_open (const char *filename, backtrace_error_callback error_callback, + void *data) +{ + int descriptor; + + descriptor = open (filename, O_RDONLY | O_CLOEXEC); + if (descriptor 0) +

Re: [PATCH] Combine location with block using block_locations

2012-09-12 Thread Martin Jambor
Hi, On Wed, Sep 12, 2012 at 04:47:11PM +0200, Richard Guenther wrote: On Wed, Sep 12, 2012 at 4:37 PM, Martin Jambor mjam...@suse.cz wrote: On Wed, Sep 12, 2012 at 04:17:45PM +0200, Michael Matz wrote: On Wed, 12 Sep 2012, Michael Matz wrote: Hm, but we shouldn't end up streaming any

Finish up PR rtl-optimization/44194

2012-09-12 Thread Eric Botcazou
This is the PR about the useless spilling to memory of structures that are returned in registers. It was essentially addressed last year by Easwaran with an enhancement of the RTL DSE pass, but Easwaran also noted that we still spill to memory in the simplest cases, e.g. gcc.dg/pr44194-1.c,

Re: [PATCH,i386] Enable prefetchw in processor alias table for AMD targets

2012-09-12 Thread Uros Bizjak
On Tue, Sep 11, 2012 at 11:03 AM, venkataramanan.ku...@amd.com wrote: Hi Maintainers, This patch enables prefetchw ISA in the processor alias table for targets amdfam10,barcelona and bdver1,2 and btver1,2. GCC regression test passes with the patch. Ok for trunk? Change log:

[AArch64] fix missing Dwarf call frame information in the epilogue

2012-09-12 Thread Yufeng Zhang
This patch fixes a bug in the AArch64 epilogue expander which failed to generate consistent information of REG_FRAME_RELATED_EXPR reg notes in some cases; for instance, given the following C source code: int bar (unsigned int); int foo (void) { return bar (0xcafe); } ---

Re: Backtrace library [1/3]

2012-09-12 Thread Joseph S. Myers
On Tue, 11 Sep 2012, Ian Lance Taylor wrote: The configury is fairly standard. Note that libbacktrace is built as both a host library (to link into the compilers) and as a target library (to link into libgo and possibly other libraries). Under what circumstances will the library be built for

Re: Backtrace library [1/3]

2012-09-12 Thread Toon Moene
On 09/12/2012 01:08 AM, Ian Lance Taylor wrote: The interface is somewhat constrained in that, on systems that support anonymous mmap, it does not call malloc. That makes it possible to do a symbolic backtrace from a signal handler. It would also make it possible to have a traceback of a

LTO partitioning reorg 4/n

2012-09-12 Thread Jan Hubicka
Hi, this patch unifies the code as promised in the previous patch. It also cleans up the APIs. We now have enum symbol_class specifying how the symbol should be handled and add_symbol_to_partition handling all the magic of what needs to be dragged into partition and what not. I also commonized

Re: [PATCH] Combine location with block using block_locations

2012-09-12 Thread Xinliang David Li
On Wed, Sep 12, 2012 at 2:13 AM, Richard Guenther richard.guent...@gmail.com wrote: On Wed, Sep 12, 2012 at 7:06 AM, Dehao Chen de...@google.com wrote: Now I think we are facing a more complex problem. The data structure we use to store the location_adhoc_data are file-static in linemap.c in

[PATCH] proposed fix for PRs target/54516 rtl-optimization/54540

2012-09-12 Thread Richard Earnshaw
Although the mersenne twister code is new, the build/runtime failures of PRs target/54516 and PR rtl-optimization/54540 can be tracked back to the addition of the variant add %0(=rk), %1(0), %2(rk) // size=2 to the thumb2 backend. This is causing reload to try to do clever things

Re: [Patch ARM] implement bswap16

2012-09-12 Thread Richard Earnshaw
On 11/09/12 16:04, Christophe Lyon wrote: On 11 September 2012 12:52, Richard Earnshaw rearn...@arm.com wrote: Try something like: short foo(int); short swaps (short x, int y) { int z = x; if (y) z = __builtin_bswap16(x); return foo (z); } If that's not enough, try

Re: [PATCH] proposed fix for PRs target/54516 rtl-optimization/54540

2012-09-12 Thread Ulrich Weigand
Richard Earnshaw wrote: It seems to me that find_dummy_reload is doing an unsafe thing by permitting the use of a register in fixed_regs[] as a reload register -- writing to fixed regs could have unspecified side effects on the machine, so they should only be used in the ways that earlier

Re: [PATCH] Combine location with block using block_locations

2012-09-12 Thread Dehao Chen
There are two parts that needs memory management: 1. The BLOCK structure. This is managed by GC. I originally thought that removing blocks from tree.gsbase would paralyze GC. This turned out not to be a concern because DECL_INITIAL will still mark those used tree nodes. This patch may decrease

Re: Finish up PR rtl-optimization/44194

2012-09-12 Thread H.J. Lu
On Wed, Sep 12, 2012 at 8:37 AM, Eric Botcazou ebotca...@adacore.com wrote: This is the PR about the useless spilling to memory of structures that are returned in registers. It was essentially addressed last year by Easwaran with an enhancement of the RTL DSE pass, but Easwaran also noted

Re: [PATCH] Combine location with block using block_locations

2012-09-12 Thread Dehao Chen
There is another bug in the patch (not covered by unittests, discovered through spec benchmarks). When we remove unused locals, we do not mark the block as used for debug stmt, but gimple-streamer-out will still stream out blocks for debug stmt. There can be 2 fixes: 1. ---

Re: Backtrace library [1/3]

2012-09-12 Thread Lawrence Crowl
On 9/12/12, Ian Lance Taylor i...@google.com wrote: On Sep 11, 2012 Lawrence Crowl cr...@googlers.com wrote: On 9/11/12, Ian Lance Taylor i...@google.com wrote: This patch is the interface to and configury of libbacktrace. I've separated these out as the parts of libbacktrace that

Re: Change double_int calls to new interface.

2012-09-12 Thread Lawrence Crowl
On 9/12/12, Mark Kettenis mark.kette...@xs4all.nl wrote: Date: Tue, 11 Sep 2012 17:03:39 -0700 From: Ian Lance Taylor i...@google.com On Tue, Sep 11, 2012 at 3:12 PM, Lawrence Crowl cr...@googlers.com wrote: On 9/11/12, Andreas Schwab sch...@linux-m68k.org wrote: Mark Kettenis

Loop stride optimization hint

2012-09-12 Thread Jan Hubicka
Hi, for Fortran one of common reason to inline is because array descriptor is known and defines loop stride. This patch makes ipa-inline-analysis to notice these cases. Bootstrapped/regtested x86_64-linux, will commit it tomorrow if there are no complains. Honza *

Re: PATCH: PR target/54445: TLS array lookup with negative constant is not combined into a single instruction

2012-09-12 Thread Uros Bizjak
On Wed, Sep 12, 2012 at 7:17 PM, H.J. Lu hjl.to...@gmail.com wrote: When x86-64 TLS support was added by: http://gcc.gnu.org/ml/gcc-patches/2002-10/msg01262.html it didn't allow negative offset. Jakub, do you remember the reason for it? I tested this patch on Linux/x86-64 and used the new

[4.7 PATCH, i386]: Enable prefetchw for selected AMD targets

2012-09-12 Thread Uros Bizjak
Hello! Newer AMD processors does not enable 3dNOW anymore. However, prefetchw depends on TARGET_3DNOW, so it is not generated anymore. Following patch is a 4.7 version of mainline patch [1]. 2012-09-12 Uros Bizjak ubiz...@gmail.com * config/i386/i386.h (x86_prefetchw): New global

Re: [4.7 PATCH, i386]: Enable prefetchw for selected AMD targets

2012-09-12 Thread H.J. Lu
On Wed, Sep 12, 2012 at 11:20 AM, Uros Bizjak ubiz...@gmail.com wrote: Hello! Newer AMD processors does not enable 3dNOW anymore. However, prefetchw depends on TARGET_3DNOW, so it is not generated anymore. Following patch is a 4.7 version of mainline patch [1]. 2012-09-12 Uros Bizjak

Re: [AArch64] fix missing Dwarf call frame information in the epilogue

2012-09-12 Thread Richard Henderson
On 09/12/2012 09:10 AM, Yufeng Zhang wrote: aarch64_set_frame_expr (gen_rtx_SET (Pmode, stack_pointer_rtx, -gen_rtx_PLUS (Pmode, stack_pointer_rtx, +

Re: [4.7 PATCH, i386]: Enable prefetchw for selected AMD targets

2012-09-12 Thread Uros Bizjak
On Wed, Sep 12, 2012 at 8:30 PM, H.J. Lu hjl.to...@gmail.com wrote: Newer AMD processors does not enable 3dNOW anymore. However, prefetchw depends on TARGET_3DNOW, so it is not generated anymore. Following patch is a 4.7 version of mainline patch [1]. 2012-09-12 Uros Bizjak

Re: Backtrace library [1/3]

2012-09-12 Thread Ian Lance Taylor
On Wed, Sep 12, 2012 at 9:23 AM, Joseph S. Myers jos...@codesourcery.com wrote: On Tue, 11 Sep 2012, Ian Lance Taylor wrote: The configury is fairly standard. Note that libbacktrace is built as both a host library (to link into the compilers) and as a target library (to link into libgo and

Re: Backtrace library [1/3]

2012-09-12 Thread Ian Lance Taylor
On Wed, Sep 12, 2012 at 10:31 AM, Lawrence Crowl cr...@googlers.com wrote: How about typing it as a pointer to an incomplete struct? extern void *backtrace_create_state (... becomes, e.g., struct backtrace_state; extern backtrace_state *backtrace_create_state (... Yeah, that is probably

[PATCH, i386]: Change x86_prefetch_sse to unsigned char

2012-09-12 Thread Uros Bizjak
Hello! This will match fake bool type that is used throughout gcc sources. We can't use bool there, since the header is used in libgcc which doesn't include system.h 2012-09-12 Uros Bizjak ubiz...@gmail.com * config/i386/i386.c (x86_prefetch_sse): Change to unsigned char. *

Re: Loop stride optimization hint

2012-09-12 Thread Tobias Burnus
Jan Hubicka wrote: for Fortran one of common reason to inline is because array descriptor is known and defines loop stride. This patch makes ipa-inline-analysis to notice these cases. Thanks for your Fortran inlining work. + t(int s, void **p) + { + int i; + for (i;i1;i+=s) +

Re: [PATCH] Combine location with block using block_locations

2012-09-12 Thread Dehao Chen
Attached is the memory consumption report for a very large source file. Looks like this patch actually reduced the memory consumption by 2%. Dehao On Thu, Sep 13, 2012 at 1:18 AM, Xinliang David Li davi...@google.com wrote: On Wed, Sep 12, 2012 at 10:05 AM, Dehao Chen de...@google.com wrote:

[PATCH] Fix PR gcov-profile/54487 (profiledbootstrap intermittent failures) (issue6496113)

2012-09-12 Thread Teresa Johnson
This fixes PR gcov-profile/54487 where the gcda files were not locked by the profile-use read, enabling writes by other instrumented compiles to change the profile in the middle of the profile use read. The GCOV_LOCKED macro was not set because it was guarded by HOST_HAS_F_SETLKW, which was never

Re: [PATCH] Fix PR gcov-profile/54487 (profiledbootstrap intermittent failures) (issue6496113)

2012-09-12 Thread Jakub Jelinek
On Wed, Sep 12, 2012 at 01:45:12PM -0700, Teresa Johnson wrote: This fixes PR gcov-profile/54487 where the gcda files were not locked by the profile-use read, enabling writes by other instrumented compiles to change the profile in the middle of the profile use read. The GCOV_LOCKED macro was

Re: [PATCH] Combine location with block using block_locations

2012-09-12 Thread Xinliang David Li
For the largest bucket (size==80), the size reduction is 20%. Not bad. David On Wed, Sep 12, 2012 at 1:44 PM, Dehao Chen de...@google.com wrote: Attached is the memory consumption report for a very large source file. Looks like this patch actually reduced the memory consumption by 2%. Dehao

Re: [PATCH] Prevent cselib substitution of FP, SP, SFP

2012-09-12 Thread Carrot Wei
Hi Jakub The same problem also affects gcc4.6, http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54398. Could this be ported to 4.6 branch? thanks Carrot On Mon, Feb 13, 2012 at 11:54 AM, Jakub Jelinek ja...@redhat.com wrote: On Wed, Jan 04, 2012 at 05:21:38PM +, Marcus Shawcroft wrote: Alias

Re: Backtrace library [1/3]

2012-09-12 Thread Joseph S. Myers
On Wed, 12 Sep 2012, Ian Lance Taylor wrote: On Wed, Sep 12, 2012 at 9:23 AM, Joseph S. Myers jos...@codesourcery.com wrote: On Tue, 11 Sep 2012, Ian Lance Taylor wrote: The configury is fairly standard. Note that libbacktrace is built as both a host library (to link into the

Re: [PATCH] Fix PR gcov-profile/54487 (profiledbootstrap intermittent failures) (issue6496113)

2012-09-12 Thread Teresa Johnson
On Wed, Sep 12, 2012 at 1:54 PM, Jakub Jelinek ja...@redhat.com wrote: On Wed, Sep 12, 2012 at 01:45:12PM -0700, Teresa Johnson wrote: This fixes PR gcov-profile/54487 where the gcda files were not locked by the profile-use read, enabling writes by other instrumented compiles to change the

Re: shrink-wrapping duplicates BBs across partitions.

2012-09-12 Thread Andrew Pinski
On Wed, Sep 12, 2012 at 4:47 AM, Christian Bruel christian.br...@st.com wrote: The problem stems from tree-ssa-tail-merge that breaks bb-count, The CFG looks like 2 / \ /6 5 (0) | | 3 - |/ \ | | 7 (1) 8 - | / 4 (1) (in parenthesis the bb-count

Loop stride inline hint

2012-09-12 Thread Jan Hubicka
Hi, this patch makes inliner to realize that it is good idea to inline when loop stride becomes constant. This is mostly to help fortran testcases where it is important to inline to get array descriptors. Bootstrapped/regtested x86_64-linux, comitted. Honza * ipa-inline-analysis.c

Re: Loop stride optimization hint

2012-09-12 Thread Martin Jambor
Hi, On Wed, Sep 12, 2012 at 07:57:16PM +0200, Jan Hubicka wrote: Hi, for Fortran one of common reason to inline is because array descriptor is known and defines loop stride. This patch makes ipa-inline-analysis to notice these cases. Bootstrapped/regtested x86_64-linux, will commit it

Re: [PATCH] limited C++ parsing support for gengtype

2012-09-12 Thread Gabriel Dos Reis
On Wed, Sep 12, 2012 at 4:54 PM, Aaron Gray aaronngray.li...@gmail.com wrote: On 11 September 2012 23:45, Gabriel Dos Reis g...@integrable-solutions.net wrote: On Tue, Sep 11, 2012 at 3:41 PM, Diego Novillo dnovi...@google.com wrote: @@ -778,6 +791,7 @@ type (options_p *optsp, bool nested)

Re: Loop stride optimization hint

2012-09-12 Thread Jan Hubicka
! set_hint_predicate (inline_summary (node)-loop_iterations, loop_iterations); ! set_hint_predicate (inline_summary (node)-loop_stride, loop_stride); scev_finalize (); } inline_summary (node)-self_time = time; Well, I know i's not that important but

[PING^2] C++ conversion - pull in cstdlib

2012-09-12 Thread Oleg Endo
Hello, On Sun, 2012-09-02 at 01:19 +0200, Oleg Endo wrote: On Sat, 2012-09-01 at 18:25 +0200, Oleg Endo wrote: On Sat, 2012-09-01 at 16:17 +, Joseph S. Myers wrote: On Sat, 1 Sep 2012, Oleg Endo wrote: Ping! This allows one to include e.g. algorithm in GCC source files.

Re: [patch] Finish double_int conversion.

2012-09-12 Thread Eric Botcazou
Index: gcc/config/sparc/sparc.c === --- gcc/config/sparc/sparc.c (revision 191083) +++ gcc/config/sparc/sparc.c (working copy) @@ -10113,33 +10113,27 @@ sparc_fold_builtin (tree fndecl, int n_a TREE_CODE (arg1) ==

Re: Loop stride optimization hint

2012-09-12 Thread Jan Hubicka
loop iterations:(op0[ref offset: 192] changed || op0[ref offset: 256] changed || op0[ref offset: 224] changed) (op0[ref offset: 96] changed || op0[ref offset: 160] changed || op0[ref offset: 128] changed) Looking at the testcase, loop stride is (op0[ref offset: 384] changed)

Re: [SH] Correct address cost estimations

2012-09-12 Thread Kaz Kojima
Oleg Endo oleg.e...@t-online.de wrote: This corrects the address cost estimations for SH. Tested on rev 191161 with make -k check RUNTESTFLAGS=--target_board=sh-sim \{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb} and no new failures. With this applied CSiBE shows a total

Re: [patch] Fix PR rtl-optimization/54290

2012-09-12 Thread Eric Botcazou
Bootstrapped/regtested on x86_64-suse-linux. Does that look plausible? Do we want to fix this on release branches as well? 2012-09-02 Eric Botcazou ebotca...@adacore.com PR rtl-optimization/54290 * reload1.c (choose_reload_regs): Also take into account secondary MEMs

Re: Scheduler: Allow breaking dependencies by modifying patterns

2012-09-12 Thread Maxim Kuvyrkov
On 12/09/2012, at 4:34 AM, Vladimir Makarov wrote: On 08/03/2012 08:05 AM, Bernd Schmidt wrote: ... Ok, thanks. The changes are pretty straightforward. Only just a few comments. ... Thanks for the patch, Bernd. Sorry for the delay with the review. I thought that Maxim writes his

Re: [google/gcc-4_7] Fix problem with asm spec for -gsplit-dwarf

2012-09-12 Thread Sterling Augustine
On Wed, Sep 12, 2012 at 4:18 PM, Cary Coutant ccout...@google.com wrote: 2012-09-12 Cary Coutant ccout...@google.com gcc/ * gcc.c (replace_extension_spec_func): Restrict search for extension to last component of path. Index: gcc/gcc.c

Re: [google/gcc-4_7] Fix problem with asm spec for -gsplit-dwarf

2012-09-12 Thread Cary Coutant
2012-09-12 Cary Coutant ccout...@google.com gcc/ * gcc.c (replace_extension_spec_func): Restrict search for extension to last component of path. This is OK for Google 4.7. Thanks, committed as r191234. -cary

Re: [C PATCH] Avoid ICEs due to save_expr instead of c_save_expr (PR c/54428)

2012-09-12 Thread H.J. Lu
On Fri, Aug 31, 2012 at 5:36 AM, Jakub Jelinek ja...@redhat.com wrote: Hi! This is another case of the issue that save_expr shouldn't be called when parsing C (except for in_late_binary_op), but c_save_expr must be called instead. I wonder if we shouldn't add a langhook, which would do if

Re: Backtrace library [1/3]

2012-09-12 Thread Ian Lance Taylor
On Wed, Sep 12, 2012 at 2:04 PM, Joseph S. Myers jos...@codesourcery.com wrote: On Wed, 12 Sep 2012, Ian Lance Taylor wrote: On Wed, Sep 12, 2012 at 9:23 AM, Joseph S. Myers jos...@codesourcery.com wrote: On Tue, 11 Sep 2012, Ian Lance Taylor wrote: The configury is fairly standard.

Re: [PATCH] Fix PR gcov-profile/54487 (profiledbootstrap intermittent failures) (issue6496113)

2012-09-12 Thread Teresa Johnson
On Wed, Sep 12, 2012 at 2:12 PM, Teresa Johnson tejohn...@google.com wrote: On Wed, Sep 12, 2012 at 1:54 PM, Jakub Jelinek ja...@redhat.com wrote: On Wed, Sep 12, 2012 at 01:45:12PM -0700, Teresa Johnson wrote: This fixes PR gcov-profile/54487 where the gcda files were not locked by the

Re: [PATCH] Fix PR gcov-profile/54487 (profiledbootstrap intermittent failures) (issue6496113)

2012-09-12 Thread Jakub Jelinek
On Wed, Sep 12, 2012 at 10:06:02PM -0700, Teresa Johnson wrote: Patch committed. I have the 4_7 backport ready. It is basically the same patch. Also tested with bootstap and profiledbootstrap. Ok for gcc/4_7? Yes, thanks. 2012-09-12 Teresa Johnson tejohn...@google.com Backport