Re: [google/4.6] For -gfission, remove address table entry when removing location list entry (issue5900045)

2012-03-23 Thread Sterling Augustine
On Fri, Mar 23, 2012 at 3:40 PM, Cary Coutant wrote: > > 2012-03-23   Cary Coutant   > >        * dwarf2out.c (resolve_addr): Remove address table entry for symbol >        when removing location list entry. > > > Index: dwarf2out.c > ===

[google/4.6] For -gfission, remove address table entry when removing location list entry (issue5900045)

2012-03-23 Thread Cary Coutant
2012-03-23 Cary Coutant * dwarf2out.c (resolve_addr): Remove address table entry for symbol when removing location list entry. Index: dwarf2out.c === --- dwarf2out.c (revision 185594) +++ dwarf2out.c (working cop

Re: Propagate profile counts after switch case expansion (issue5896043)

2012-03-23 Thread Andi Kleen
> Do you mean use the weights to decide the shape of the binary tree Yes. > (similar to COST_TABLE heuristic)? COST_TABLE should die I hope. > I am planning to send a separate patch for that. Great. -Andi

Re: Propagate profile counts after switch case expansion (issue5896043)

2012-03-23 Thread Easwaran Raman
On Fri, Mar 23, 2012 at 3:29 PM, Andi Kleen wrote: > Easwaran Raman writes: > >> Some more background on this patch: Right now, while the execution >> counts of different case labels of a switch statement are obtained >> during profile collection, they are not propagated to RTL. Instead, >> count

Re: Propagate profile counts after switch case expansion (issue5896043)

2012-03-23 Thread Andi Kleen
Easwaran Raman writes: > Some more background on this patch: Right now, while the execution > counts of different case labels of a switch statement are obtained > during profile collection, they are not propagated to RTL. Instead, > counts are regenerated at the RTL level using static heuristics

[pph] Fix bindings for using declarations (issue5900043)

2012-03-23 Thread Diego Novillo
Fix bindings for using declarations. When an identifier is declared in a using declaration, we were registering the USING_DECL object in the corresponding binding. This was wrong, and at the time I hacked around it by simply ignoring USING_DECLs when setting bindings. That was wrong too. What w

Re: Propagate profile counts after switch case expansion (issue5896043)

2012-03-23 Thread Easwaran Raman
Some more background on this patch: Right now, while the execution counts of different case labels of a switch statement are obtained during profile collection, they are not propagated to RTL. Instead, counts are regenerated at the RTL level using static heuristics that tend to weigh branches equal

Re: [PATCH] AIX malloc.h fixincludes

2012-03-23 Thread Bruce Korb
On Fri, Mar 23, 2012 at 11:47 AM, David Edelsohn wrote: > Some early releases of AIX 6 include a malloc.h header with an > incorrect use of "builtin" keyword: > > #ifdef __cplusplus > extern "C" { >   extern "builtin" char *__alloca (size_t); > #  define alloca __alloca > } > #endif /* def __cplus

[PATCH] AIX malloc.h fixincludes

2012-03-23 Thread David Edelsohn
Some early releases of AIX 6 include a malloc.h header with an incorrect use of "builtin" keyword: #ifdef __cplusplus extern "C" { extern "builtin" char *__alloca (size_t); # define alloca __alloca } #endif /* def __cplusplus */ GCC is not too happy about this. Now that GCC mainline and 4.7

libgo patch committed: Fix errno handling in syscall.Syscall

2012-03-23 Thread Ian Lance Taylor
This patch to libgo fixes the handling of errno in syscall.Syscall and friends. Previously it would return a non-zero errno value even when the system call succeeded. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline and 4.7 branch. Ian diff -r b626c77ab40e l

Propagate profile counts after switch case expansion (issue5896043)

2012-03-23 Thread Easwaran Raman
This patch propagates execution count of thee case labels of a switch-case statement after its expansion. Bootstraps and all tests pass. OK for trunk? 2012-03-23 Easwaran Raman * cfgbuild.c (non_zero_profile_counts): New function. (compute_outgoing_frequencies): If at least on

AW: [PATCH] eh_personality.cc: unwinding on ARM

2012-03-23 Thread EXTERNAL Waechtler Peter (Fa. TCP, CM-AI/PJ-CF31)
> -Ursprüngliche Nachricht- > Von: Andrew Stubbs [mailto:a...@codesourcery.com] > Gesendet: Montag, 19. März 2012 17:12 > An: EXTERNAL Waechtler Peter (Fa. TCP, CM-AI/PJ-CF31) > Cc: gcc-patches@gcc.gnu.org; libstd...@gcc.gnu.org; > p...@codesourcery.com; pwaecht...@mac.com; d...@false.org >

Re: [v3] remove duplicate cxxflags

2012-03-23 Thread Benjamin De Kosnik
... here's the patch.2012-03-22 Benjamin Kosnik * scripts/testsuite_flags.in (--cxxflags): Let CXXFLAGS set -g -O2. * testsuite/lib/libstdc++.exp: Same. diff --git a/libstdc++-v3/scripts/testsuite_flags.in b/libstdc++-v3/scripts/testsuite_flags.in index 457adaf..f77784b 100755 --- a/libstd

[v3] remove duplicate cxxflags

2012-03-23 Thread Benjamin De Kosnik
Removes duplicates on command line when running test suite. CXXFLAG info was being quoted twice. tested x86/linux -benjamin

[C11-atomic] new test: limit precomputing values across acquire barriers

2012-03-23 Thread Aldy Hernandez
This is a similar test to the previous acquire test. Here we are incorrectly caching 'x' and failing to reload it after the __ATOMIC_ACQUIRE. + i = x + y; + + if (__atomic_load_n (&flag, __ATOMIC_ACQUIRE)) +{ + /* x here should not be reused from above. */ + k = x; +} Note

Re: [C11-atomic] test invalid hoist across and acquire load

2012-03-23 Thread Aldy Hernandez
After much pondering and talking with both you and Torvald, it has been determined that the test at hand is technically allowed to hoist the value of x/y because the standard guarantees that the code below is data race free: if (__atomic_load_n (&flag1, __ATOMIC_ACQUIRE))

Re: [C11-atomic] test invalid hoist across and acquire load

2012-03-23 Thread Andrew MacLeod
On 03/23/2012 10:39 AM, Aldy Hernandez wrote: On 03/21/12 12:54, Andrew MacLeod wrote: On 03/21/2012 01:35 PM, Aldy Hernandez wrote: In the test below, we cannot cache either [x] or [y] neither before the load of flag1 nor the load of flag2. This is because the corresponding store/release can f

Re: [C11-atomic] test invalid hoist across and acquire load

2012-03-23 Thread Aldy Hernandez
On 03/21/12 12:54, Andrew MacLeod wrote: On 03/21/2012 01:35 PM, Aldy Hernandez wrote: In the test below, we cannot cache either [x] or [y] neither before the load of flag1 nor the load of flag2. This is because the corresponding store/release can flush a different value of x or y: + if (__atom

[PATCH] PR bootstrap/52623 Disable libquadmath on AIX

2012-03-23 Thread David Edelsohn
The build process of libquadmath sometimes encounters problems on AIX due to multilib and LD_LIBRARY_PATH interfering with GCC's own library dependencies. Libquadmath is not used on AIX, so this patch adds it to noconfigdirs. Thanks, David * configure.ac: Add libquadmath to noconfigdirs

[Patch, libfortran] Use calloc instead of malloc+memset

2012-03-23 Thread Janne Blomqvist
Hi, in a few places in libgfortran we use malloc (or rather, our own xmalloc-like wrapper, get_mem) followed by a memset to zero the memory. The attached patch replaces this usage with calloc(). Regtested on x86_64-unknown-linux-gnu, unless somebody objects I'll commit it as obvious in a few days

Re: [PATCH] Preserve loops from tree to RTL loop optimizers

2012-03-23 Thread Richard Guenther
On Wed, 21 Mar 2012, Richard Sandiford wrote: > Richard Guenther writes: > > This patch makes us preserve loop structures from the start of tree > > loop optimizers to the end of RTL loop optimizers. It uses a new > > property, PROP_loops to indicate we want to preserve them and > > massages loo

[PATCH] Remove strict-alignment checks in SRA

2012-03-23 Thread Martin Jambor
Hi, since we now should be able to expand misaligned MEM_REFs properly and both SRA and IPA-SRA now tag the memory accesses with the appropriate alignment information, we should now be able to get rid off the SRA disabling in the face of potential strict-alignment expansion problems. The patch be

Re: [Ping][PATCH, libstdc++-v3] Enable to cross-test libstdc++ on simulator

2012-03-23 Thread Paolo Carlini
On 03/23/2012 02:02 PM, Paolo Bonzini wrote: It was approved by Mike Stump who obviously confused the *.in with *.am, or libstdc++-v3 with gcc. It happens. :) Good, good, thanks Paolo. The messages on the mailing list aren't fully threaded and I didn't see immediately the approval. And of cou

Re: [Ping][PATCH, libstdc++-v3] Enable to cross-test libstdc++ on simulator

2012-03-23 Thread Paolo Bonzini
Il 23/03/2012 13:40, Paolo Carlini ha scritto: > Hi, > > On 03/07/2012 06:22 AM, Terry Guo wrote: >> Hello, >> >> Can anybody please review and approve the following simple patch? Thanks >> very much. >> >> http://gcc.gnu.org/ml/libstdc++/2011-08/msg00063.html > apparently somebody, somewhere, app

Re: [Ping][PATCH, libstdc++-v3] Enable to cross-test libstdc++ on simulator

2012-03-23 Thread Paolo Carlini
Hi, On 03/07/2012 06:22 AM, Terry Guo wrote: Hello, Can anybody please review and approve the following simple patch? Thanks very much. http://gcc.gnu.org/ml/libstdc++/2011-08/msg00063.html apparently somebody, somewhere, approved this patch, because I see: 2012-03-13 Terry Guo * test

Re: [PATCH] Replace a SRA FIXME with an assert

2012-03-23 Thread Martin Jambor
On Wed, Mar 21, 2012 at 08:46:49AM +0100, Richard Guenther wrote: > On Tue, 20 Mar 2012, Martin Jambor wrote: > > > Hi, > > > > On Tue, Mar 20, 2012 at 04:08:31PM +0100, Richard Guenther wrote: > > > On Tue, 20 Mar 2012, Martin Jambor wrote: > > > > > > > Hi, > > > > > > > > this patch which re

[PATCH] Fix PR52678

2012-03-23 Thread Richard Guenther
This patch fixes PR52678 - re-computing the evolution of a loop PHI node after updating it during prologue loop peeling is fragile (and I remember several cases in the past where we ICEd here and adjusted the code). So, instead of re-computing the evolution this patch saves the evolution part at

Re: RFA: Document addition of Epiphany support

2012-03-23 Thread Rainer Orth
Joern Rennecke writes: > 2012-03-23 Joern Rennecke > > * htdocs/gcc-4.7/changes.html: Document addition of Epiphany support. > > Index: htdocs/gcc-4.7/changes.html > === > RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-4.7/changes.ht

RFA: Document addition of Epiphany support

2012-03-23 Thread Joern Rennecke
2012-03-23 Joern Rennecke * htdocs/gcc-4.7/changes.html: Document addition of Epiphany support. Index: htdocs/gcc-4.7/changes.html === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-4.7/changes.html,v retrieving revision 1.102 di

Re: remove wrong code in immed_double_const

2012-03-23 Thread Richard Sandiford
Mike Stump writes: >> Sorry, with this bit, I meant that the current svn code is correct >> for GET_MODE_BITSIZE (op_mode) == HOST_BITS_PER_WIDE_INT * 2. >> In that case, hv < 0 can just mean that we have a uint128_t >> (or whatever) whose high bit happens to be set. (To be clear, I was using uin

Re: [PATCH, ARM] Don't force vget_lane returning a 64-bit result to transfer to core registers

2012-03-23 Thread Julian Brown
On Wed, 21 Mar 2012 11:20:13 + Richard Earnshaw wrote: > Semantically the neon intrinsic vgetq_lane_[su]64 returns a 64 bit > sub-object of a 128-bit vector; there's no real need for the intrinsic > to map onto a specific machine instruction. > > Indeed, if force a particular instruction tha

Re: [PATCH] Update sparc-linux-gnu baselines

2012-03-23 Thread Jonathan Wakely
On 23 March 2012 07:47, David Miller wrote: > > I verified that this matches what other Linux platforms have added > recently, and I was careful not to accidently include the TLS symbols. > > In fact, this baseline file now matches powerpc-linux-gnu's precisely. > > Ok for mainline and the 4.7 bran

Re: [PATCH] Fix PRs 52080, 52097 and 48124, rewrite bitfield expansion, enable the C++ memory model wrt bitfields everywhere

2012-03-23 Thread Richard Guenther
On Thu, 22 Mar 2012, Richard Guenther wrote: > On Thu, 22 Mar 2012, Eric Botcazou wrote: > > > > bitregion_start == 11 looks bogus. The representative is starting at > > > > > > DECL_FIELD_BIT_OFFSET (repr) > > > = size_binop (BIT_AND_EXPR, > > > DECL_FIELD_BIT_OFFSET (fi

[Patch, fortran] Remove unused GFC_MAX_LINE macro

2012-03-23 Thread Janne Blomqvist
Hi, the maximum line length is nowadays decided dynamically at runtime as specified by the chosen standard version, or overridden by the -fline-length option. The patch below removes the GFC_MAX_LINE macro which is no longer used. Committed to trunk as obvious. Index: gfortran.h

[PATCH] Testcase for PR52638

2012-03-23 Thread Richard Guenther
Committed. Richard.

[PATCH] Fix PR52682

2012-03-23 Thread Richard Guenther
Committed as obvious. Richard. 2012-03-23 William Bader c-family/ * c-lex.c (c_lex_with_flags): Avoid declarations after stmts. Index: gcc/c-family/c-lex.c === *** gcc/c-family/c-lex.c(revision 185724) -

[PATCH] Update sparc-linux-gnu baselines

2012-03-23 Thread David Miller
I verified that this matches what other Linux platforms have added recently, and I was careful not to accidently include the TLS symbols. In fact, this baseline file now matches powerpc-linux-gnu's precisely. Ok for mainline and the 4.7 branch? Thanks. 2012-03-23 David S. Miller *