[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: svn://gcc.gnu.

[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 -m4

Re: Obsolete picochip-* in 4.7.2+

2012-09-12 Thread Richard Guenther
On Tue, Sep 11, 2012 at 5:21 PM, Jakub Jelinek 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 for trun

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 wit

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 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 an argume

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 o

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 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 pr

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 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 removed the block

[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

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 seem

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 pref

[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 PR middle-end/54553 * common.opt (finline): Mark with Optimization. Index: gcc/c

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 wrote: > On Wed, Sep 12, 2012 at 10:12 AM, Sharad Singhai wrote: >> Thanks for your comments. Please see my responses inline. >> >> On Tue, Sep 11, 2012 at 1:16 PM, Xinliang David Li >> wrote: >>> Can you resend your patch in text form (also ne

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 S

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, Ri

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 handl

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 > > On Tue, Sep 11, 2012 at 3:12 PM, Lawrence Crowl wrote: > > On 9/11/12, Andreas Schwab wrote: > >> Mark Kettenis writes: > >>> In file included from ../../../src/gcc/gcc/mcf.c:47:0: > >>> ../../../src/gcc/gcc/mcf.c: In functio

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 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_PREV

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 fil

[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 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 th

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

Re: Backtrace library [1/3]

2012-09-12 Thread Ian Lance Taylor
On Tue, Sep 11, 2012 at 4:36 PM, Lawrence Crowl wrote: > On 9/11/12, Ian Lance Taylor 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 libbacktrace is in >> the file

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 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 sup

Re: Backtrace library [3/3]

2012-09-12 Thread Ian Lance Taylor
On Tue, Sep 11, 2012 at 10:16 PM, Jakub Jelinek 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 hash table t

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 ar

Re: Backtrace library [3/3]

2012-09-12 Thread Ian Lance Taylor
On Wed, Sep 12, 2012 at 2:01 AM, Florian Weimer 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 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 BLO

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, bec

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, 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: > > 2012-09-11 Venkataraman

[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); } --- CUT

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 f

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 segm

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 so

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 wrote: > On Wed, Sep 12, 2012 at 7:06 AM, Dehao Chen 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

[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 thin

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 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

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 the

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

2012-09-12 Thread H.J. Lu
On Sun, Sep 2, 2012 at 7:20 AM, H.J. Lu wrote: > Hi, > > 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 GCC to

Re: [PATCH] Combine location with block using block_locations

2012-09-12 Thread Xinliang David Li
On Wed, Sep 12, 2012 at 10:05 AM, Dehao Chen wrote: > 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

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 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 that we still > spi

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. --- a/gcc/gimple-streamer-

Re: Backtrace library [1/3]

2012-09-12 Thread Lawrence Crowl
On 9/12/12, Ian Lance Taylor wrote: > On Sep 11, 2012 Lawrence Crowl wrote: > > On 9/11/12, Ian Lance Taylor 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 t

Re: Change double_int calls to new interface.

2012-09-12 Thread Lawrence Crowl
On 9/12/12, Mark Kettenis wrote: >> Date: Tue, 11 Sep 2012 17:03:39 -0700 >> From: Ian Lance Taylor >> >> On Tue, Sep 11, 2012 at 3:12 PM, Lawrence Crowl >> wrote: >> > On 9/11/12, Andreas Schwab wrote: >> >> Mark Kettenis writes: >> >>> In file included from ../../../src/gcc/gcc/mcf.c:47:0: >

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 * ipa-inline-analysis.

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 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 GCC to

[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 * config/i386/i386.h (x86_prefetchw): New global variable. (TAR

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 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 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 >> >> * config/i

Re: Backtrace library [1/3]

2012-09-12 Thread Ian Lance Taylor
On Wed, Sep 12, 2012 at 9:23 AM, Joseph S. Myers 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 possibly other li

Re: Backtrace library [1/3]

2012-09-12 Thread Ian Lance Taylor
On Wed, Sep 12, 2012 at 10:31 AM, Lawrence Crowl 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 the way to

[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 * config/i386/i386.c (x86_prefetch_sse): Change to unsigned char. * config/i386/i386.h

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;i<1;i+=s) + p

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 wrote: > On Wed, Sep 12, 2012 at 10:05 AM, Dehao Chen wrote: >> There are two parts that needs me

[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 s

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 wa

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 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 > > On Thu,

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 wrote: > > On Wed, Jan 04, 2012 at 05:21:38PM +, Marcus Shawcroft wrote: > > Alias analysis by

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 > 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 t

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 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 profile in th

Re: shrink-wrapping duplicates BBs across partitions.

2012-09-12 Thread Andrew Pinski
On Wed, Sep 12, 2012 at 4:47 AM, Christian Bruel 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 from

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 (dump_

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

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 wrote: > On 11 September 2012 23:45, Gabriel Dos Reis > wrote: >> On Tue, Sep 11, 2012 at 3:41 PM, Diego Novillo wrote: >> @@ -778,6 +791,7 @@ type (options_p *optsp, bool nested) return resolve_typedef (s, &lexer_line);

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 no

[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. in GCC so

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 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 size decrease of

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 > > PR rtl-optimization/54290 > * reload1.c (choose_reload_regs): Also take into account secondary MEMs > to remove

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 wri

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

2012-09-12 Thread Cary Coutant
2012-09-12 Cary Coutant gcc/ * gcc.c (replace_extension_spec_func): Restrict search for extension to last component of path. Index: gcc/gcc.c === --- gcc/gcc.c (revision 191233) +++ gcc/gcc.c (working copy) @@

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 wrote: > 2012-09-12 Cary Coutant > > 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 >> >> 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 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 (in_late_

Re: Backtrace library [1/3]

2012-09-12 Thread Ian Lance Taylor
On Wed, Sep 12, 2012 at 2:04 PM, Joseph S. Myers wrote: > On Wed, 12 Sep 2012, Ian Lance Taylor wrote: > >> On Wed, Sep 12, 2012 at 9:23 AM, Joseph S. Myers >> wrote: >> > On Tue, 11 Sep 2012, Ian Lance Taylor wrote: >> > >> >> The configury is fairly standard. Note that libbacktrace is built as

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 wrote: > On Wed, Sep 12, 2012 at 1:54 PM, Jakub Jelinek 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

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 > > Backport from mainline. >