Re: [PATCH v2] ARM: Use different linker path for hardfloat ABI

2012-05-23 Thread Andreas Jaeger
On 05/01/2012 10:52 AM, Richard Earnshaw wrote: On 30/04/12 22:47, Michael Hope wrote: On 1 May 2012 03:24, Richard Earnshawrearn...@arm.com wrote: On 27/04/12 00:27, Michael Hope wrote: On 27 April 2012 08:20, Carlos O'Donellcar...@systemhalted.org wrote: On Mon, Apr 23, 2012 at 5:36 PM,

Re: [PATCH v2] ARM: Use different linker path for hardfloat ABI

2012-05-23 Thread Richard Earnshaw
On 23/05/12 08:12, Andreas Jaeger wrote: On 05/01/2012 10:52 AM, Richard Earnshaw wrote: On 30/04/12 22:47, Michael Hope wrote: On 1 May 2012 03:24, Richard Earnshawrearn...@arm.com wrote: On 27/04/12 00:27, Michael Hope wrote: On 27 April 2012 08:20, Carlos O'Donellcar...@systemhalted.org

Re: [PATCH v2] ARM: Use different linker path for hardfloat ABI

2012-05-23 Thread Andreas Jaeger
On Wednesday, May 23, 2012 09:56:31 Richard Earnshaw wrote: [...] This is a behaviour change. It would need RM approval for a release branch. R. There was agreement by all pushing for the change to use it. So, let's ask the release managers about their opinion, Andreas -- Andreas

Re: [PATCH v2] ARM: Use different linker path for hardfloat ABI

2012-05-23 Thread Richard Guenther
On Wed, 23 May 2012, Andreas Jaeger wrote: On Wednesday, May 23, 2012 09:56:31 Richard Earnshaw wrote: [...] This is a behaviour change. It would need RM approval for a release branch. R. There was agreement by all pushing for the change to use it. So, let's ask the release

Re: [PATCH 2/2] Better system header location detection for built-in macro tokens

2012-05-23 Thread Dodji Seketeli
Jason Merrill ja...@redhat.com writes: What if the built-in macro appears in a macro defined in a system header but used in user code? This would resolve the location all the way to the user code, and warn. I think we only want to step out until we reach a non-built-in macro, not all the

Fix ipa-ref memory corruption

2012-05-23 Thread Jan Hubicka
Hi, the testcase bellow manages to corrupt IPA-REF reference lists. The problem is that C++ FE creates new symtab node while another is in construction via DECL_ASSEMBLER_NAME hook. Bootstrapped/regtested/comitted x86_64-linux. Honza Index: ChangeLog

[PR49888, VTA] don't keep VALUEs bound to modified MEMs

2012-05-23 Thread Alexandre Oliva
Nothing in var-tracking was prepared to drop MEMs from bindings upon writes that might modify the MEM contents. That was correct, back when it was variables (rather than values) that were bound to MEMs, and we wanted the binding to remain even if the value stored in the variables changed. VTA

[libgo] Fix IRIX bootstrap failure

2012-05-23 Thread Rainer Orth
The current 4.7 branch fails to build on IRIX 6.5: /vol/gcc/src/hg/gcc-4.7-branch/local/libgo/runtime/go-caller.c:51:1: error: conflicting types for '__go_file_line' In file included from /vol/gcc/src/hg/gcc-4.7-branch/local/libgo/runtime/go-caller.c:11:0:

Re: [PR49888, VTA] don't keep VALUEs bound to modified MEMs

2012-05-23 Thread Richard Guenther
On Wed, May 23, 2012 at 12:13 PM, Jakub Jelinek ja...@redhat.com wrote: On Wed, May 23, 2012 at 06:27:21AM -0300, Alexandre Oliva wrote: +static int +drop_overlapping_mem_locs (void **slot, void *data) +{ +  struct overlapping_mems *coms = (struct overlapping_mems *)data; +  dataflow_set

Re: [C++ Patch] PR 29185

2012-05-23 Thread Paolo Carlini
On 05/23/2012 04:54 AM, Gabriel Dos Reis wrote: On Tue, May 22, 2012 at 8:25 PM, Paolo Carlinipaolo.carl...@oracle.com wrote: Hi, some years ago Martin lamented that we weren't consistently warning about deleting member arrays vs arrays. A fix seems simple and passes bootstrap and testing on

Re: [C++ Patch] PR 44516

2012-05-23 Thread Paolo Carlini
On 05/23/2012 06:30 AM, Jason Merrill wrote: On 05/22/2012 05:18 PM, Paolo Carlini wrote: Uhhm, I have an out of the blue idea, so please excuse me if for some obvious reason doesn't make sense: don't we have a global variable saying where we are in the compiler pipeline? I'm not sure, but

Re: [C++ Patch] PR 29185

2012-05-23 Thread Paolo Carlini
On 05/23/2012 06:34 AM, Jason Merrill wrote: On 05/22/2012 09:25 PM, Paolo Carlini wrote: some years ago Martin lamented that we weren't consistently warning about deleting member arrays vs arrays. I wonder why we look at the code of exp at all. Surely deleting any expression with array

[PATCH] Fix memory leak in inline_merge_summary

2012-05-23 Thread Martin Jambor
Hi, the vector operand_map is not freed in inline_merge_summary, this patch fixes it. It looks fairly obvious and I also have talked about the problem on IRC with Honza yesterday so I will commit it after bootstrap and testing on x86_64-linux. I suppose I should then test and commit it to the

Re: [PATCH] Fix memory leak in inline_merge_summary

2012-05-23 Thread Richard Guenther
On Wed, May 23, 2012 at 1:02 PM, Martin Jambor mjam...@suse.cz wrote: Hi, the vector operand_map is not freed in inline_merge_summary, this patch fixes it.  It looks fairly obvious and I also have talked about the problem on IRC with Honza yesterday so I will commit it after bootstrap and

Re: [PATCH] Hoist adjacent pointer loads

2012-05-23 Thread Richard Guenther
On Tue, 22 May 2012, William J. Schmidt wrote: Here's a revision of the hoist-adjacent-loads patch. Besides hopefully addressing all your comments, I added a gate of at least -O2 for this transformation. Let me know if you prefer a different minimum opt level. I'm still running SPEC

Re: [C++ Patch] PR 44516

2012-05-23 Thread Paolo Carlini
On 05/23/2012 12:30 PM, Paolo Carlini wrote: On 05/23/2012 06:30 AM, Jason Merrill wrote: On 05/22/2012 05:18 PM, Paolo Carlini wrote: Uhhm, I have an out of the blue idea, so please excuse me if for some obvious reason doesn't make sense: don't we have a global variable saying where we are in

[PATCH][4/n] referenced-vars TLC

2012-05-23 Thread Richard Guenther
This finally switches us to not record global vars in referenced-vars. For this to work I had to re-engineer how we handle global var removal from local-decls in remove_unused_locals. Incidentially that code already had some sort of a bitmap (for some weird reason even), thus I borrowed that and

Re: [PATCH, 4.6] Fix PR53170: missing target c++11 selector

2012-05-23 Thread Paolo Carlini
On 05/23/2012 05:39 AM, Michael Hope wrote: On 21/05/12 21:14, Paolo Carlini wrote: On 05/21/2012 01:45 AM, Michael Hope wrote: The testsuite for PR52796 uses the 'target c++11' selector which doesn't exist in 4.6. This patch backports the selector, clearing the 'ERROR:

Re: [C++ Patch] PR 29185

2012-05-23 Thread Jason Merrill
OK. We shouldn't need to worry about null type, since templates are handled at the top of the function. Jason

Re: [C++ Patch] PR 29185

2012-05-23 Thread Gabriel Dos Reis
On Wed, May 23, 2012 at 5:24 AM, Paolo Carlini paolo.carl...@oracle.com wrote: On 05/23/2012 04:54 AM, Gabriel Dos Reis wrote: On Tue, May 22, 2012 at 8:25 PM, Paolo Carlinipaolo.carl...@oracle.com  wrote: Hi, some years ago Martin lamented that we weren't consistently warning about

Re: [PATCH v2] ARM: Use different linker path for hardfloat ABI

2012-05-23 Thread Mike Frysinger
On Wednesday 23 May 2012 04:17:51 Richard Guenther wrote: On Wed, 23 May 2012, Andreas Jaeger wrote: On Wednesday, May 23, 2012 09:56:31 Richard Earnshaw wrote: [...] This is a behaviour change. It would need RM approval for a release branch. R. There was agreement by all

Re: [PATCH][4/n] referenced-vars TLC

2012-05-23 Thread H.J. Lu
On Wed, May 23, 2012 at 5:00 AM, Richard Guenther rguent...@suse.de wrote: This finally switches us to not record global vars in referenced-vars. For this to work I had to re-engineer how we handle global var removal from local-decls in remove_unused_locals.  Incidentially that code already

[Patch, fortran] PR 53456 Improve time resolution on targets without gettimeofday

2012-05-23 Thread Janne Blomqvist
Hi, some targets such as VXWorks don't provide gettimeofday but do provide clock_gettime. The attached patch allows such targets to get better resolution for the DATE_AND_TIME (up to the 1 millisecond limit of the API) intrinsic than the 1 second resolution provided by the current fallback of

[Patch,AVR]: Fix PR53448: ignored aligned(2)

2012-05-23 Thread Georg-Johann Lay
Sometimes, even on AVR there is the need to align data. However, alignments of 2 are ignored at the moment because of #define ASM_OUTPUT_ALIGN(STREAM, POWER) \ do { \ if ((POWER) 1)\

Re: [Patch,AVR]: Fix PR53448: ignored aligned(2)

2012-05-23 Thread Denis Chertykov
2012/5/23 Georg-Johann Lay a...@gjlay.de: Sometimes, even on AVR there is the need to align data. However, alignments of 2 are ignored at the moment because of #define ASM_OUTPUT_ALIGN(STREAM, POWER)                 \  do {                                                  \    if ((POWER)

Re: [RFA] PowerPC e5500 and e6500 cores support

2012-05-23 Thread Edmar
David, Michael, Thanks for the feedback. If you don't object, I will relay the message to the designers. Meanwhile I have to work with the cards I have, so... I will break the patch in three parts: - One that includes the very basic, scheduling etc. - One for the Altivec builtins, which I will

Re: [Patch, fortran] PR 53456 Improve time resolution on targets without gettimeofday

2012-05-23 Thread Tobias Burnus
Hi Janne, On 05/23/2012 04:44 PM, Janne Blomqvist wrote: some targets such as VXWorks don't provide gettimeofday but do provide clock_gettime. The attached patch allows such targets to get better resolution for the DATE_AND_TIME (up to the 1 millisecond limit of the API) intrinsic than the 1

[PATCH, rs6000] Fix insertion of nop[s] to force dependent load into new dispatch group.

2012-05-23 Thread Pat Haugen
The following patch fixes existing code that tried to prevent load-hit-store (LHS) from being in the same dispatch group. The main problem was use of the wrong dependency list in is_costly_group(), but I also added code to verify the memory refs overlap and to emit group ending nops for those

Re: [Patch, fortran] PR 53456 Improve time resolution on targets without gettimeofday

2012-05-23 Thread Janne Blomqvist
On Wed, May 23, 2012 at 6:36 PM, Tobias Burnus bur...@net-b.de wrote: Hi Janne, On 05/23/2012 04:44 PM, Janne Blomqvist wrote: some targets such as VXWorks don't provide gettimeofday but do provide clock_gettime. The attached patch allows such targets to get better resolution for the

Re: [Ada] Remove call to expand_decl

2012-05-23 Thread Eric Botcazou
2005-11-14 Thomas Quinot qui...@adacore.com Olivier Hainque hain...@adacore.com Eric Botcazou ebotca...@adacore.com ... (create_var_decl): call expand_decl for CONST_DECLs, to set MODE, ALIGN SIZE and SIZE_UNIT which we need for later back-annotations. I

Re: fix cross build

2012-05-23 Thread Nathan Sidwell
On 05/22/12 15:12, Richard Guenther wrote: But I wonder why CONSTRUCTORs do not inherit TREE_SIDE_EFFECTS properly ... the attached patch fixes the ICE and causes no regressions on i686-pc-linux-gnu. ok? nathan 2012-05-23 Nathan Sidwell nat...@acm.org * tree.c (build_constructor):

Re: [RFA] PowerPC e5500 and e6500 cores support

2012-05-23 Thread David Edelsohn
On Wed, May 23, 2012 at 10:18 AM, Edmar ed...@freescale.com wrote: David, Michael, Thanks for the feedback. If you don't object, I will relay the message to the designers. Meanwhile I have to work with the cards I have, so... I will break the patch in three parts: - One that includes the

[cxx-conversion] Convert vec.[ch] to C++ [2/3] (issue6236043)

2012-05-23 Thread Diego Novillo
Part 2 of the VEC C++ conversion. This patch implements the gengtype changes. I extended gengtype to understand templated types. These changes are not as ugly as I thought they would be. Gengtype has some hardwired knowledge of VEC_*, which I renamed to vec_t. I'm not even sure why gengtype

Re: [Patch, Fortran] PR51055 - accept non-spec-expr i in allocate(character(len=i)::s)

2012-05-23 Thread Tobias Burnus
*ping* On 20 May 2012 10:34, Tobias Burnus wrote: *ping* On Tue, 15 May 2012 12:26, Tobias Burnus wrote: A rather simple patch. Build and regtested on x86-64-linux. OK for the trunk? I think that is the last patch required for commonly used code. Remaining are issues with array

Re: [Patch, Fortran] PR51055 - accept non-spec-expr i in allocate(character(len=i)::s)

2012-05-23 Thread Steven Bosscher
On Wed, May 23, 2012 at 9:49 PM, Tobias Burnus bur...@net-b.de wrote: *ping* On 20 May 2012 10:34, Tobias Burnus wrote: *ping* On Tue, 15  May 2012 12:26, Tobias Burnus wrote: A rather simple patch. Build and regtested on x86-64-linux. OK for the trunk? Looks obvious to me :-) Ciao!

Re: [C++ Patch] for c++/51214

2012-05-23 Thread Fabien Chêne
2012/5/7 Jason Merrill ja...@redhat.com: On 05/06/2012 04:06 PM, Fabien Chêne wrote: +      if (late_enum_values) +          VEC_safe_push (tree, gc, late_enum_values, decl); I would think you could walk the TYPE_VALUES list directly, rather than copy it into a temporary VEC. Indeed, let's

[Patch, fortran] PR 53456 More CPU timing fallbacks

2012-05-23 Thread Janne Blomqvist
Hi, the attached patch allows the use of clock_gettime() with CLOCK_PROCESS_CPUTIME_ID or CLOCK_THREAD_CPUTIME_ID if the target doesn't have getrusage() or times(). Such a target is apparently VxWorks 6.something, see

Re: [PATCH v2] ARM: Use different linker path for hardfloat ABI

2012-05-23 Thread Michael Hope
On 24 May 2012 02:16, Mike Frysinger vap...@gentoo.org wrote: On Wednesday 23 May 2012 04:17:51 Richard Guenther wrote: On Wed, 23 May 2012, Andreas Jaeger wrote: On Wednesday, May 23, 2012 09:56:31 Richard Earnshaw wrote: [...] This is a behaviour change.  It would need RM approval for

Re: [PATCH, 4.6] Fix PR53170: missing target c++11 selector

2012-05-23 Thread Michael Hope
On 24 May 2012 00:27, Paolo Carlini paolo.carl...@oracle.com wrote: On 05/23/2012 05:39 AM, Michael Hope wrote: On 21/05/12 21:14, Paolo Carlini wrote: On 05/21/2012 01:45 AM, Michael Hope wrote: The testsuite for PR52796 uses the 'target c++11' selector which doesn't exist in 4.6. This

Re: [cxx-conversion] Convert vec.[ch] to C++ [1/3] (issue6233044)

2012-05-23 Thread Lawrence Crowl
On 5/23/12, Diego Novillo dnovi...@google.com wrote: OK for cxx-conversion branch? LGTM. -- Lawrence Crowl

Re: [cxx-conversion] Convert vec.[ch] to C++ [2/3] (issue6236043)

2012-05-23 Thread Lawrence Crowl
On 5/23/12, Diego Novillo dnovi...@google.com wrote: Part 2 of the VEC C++ conversion. This patch implements the gengtype changes. LGTM. -- Lawrence Crowl

Re: [cxx-conversion] Convert vec.[ch] to C++ [3/3] (issue6236044)

2012-05-23 Thread Lawrence Crowl
On 5/23/12, Diego Novillo dnovi...@google.com wrote: Part 3 of the VEC C++ conversion. This patch implements all the client code changes needed by the API changes made by the first patch. LGTM. -- Lawrence Crowl

Re: Turn check macros into functions. (issue6188088)

2012-05-23 Thread Lawrence Crowl
On 5/21/12, Mike Stump mikest...@comcast.net wrote: On May 18, 2012, at 7:48 PM, Lawrence Crowl wrote: On 5/17/12, Mike Stump mikest...@comcast.net wrote: On May 17, 2012, at 2:41 PM, Lawrence Crowl wrote: Reusing the compiler for this seems like the only way to go. But, we did look at using

Re: Turn check macros into functions. (issue6188088)

2012-05-23 Thread Lawrence Crowl
On 5/21/12, Tom Tromey tro...@redhat.com wrote: Alexander == Alexander Monakov amona...@ispras.ru writes: Alexander Hm, isn't GDB's 'set unwindonsignal on' enough to fix it? Alexander It's useful to have that in your .gdbinit anyway, because the Alexander same issue arises when calling

[PATCH] Add powerpc64-linux configuration options

2012-05-23 Thread Michael Meissner
On powerpc64-linux systems that run on IBM servers, the 32-bit software emulation library is not built with the Red Hat and SUSE distributions, but the FSF sources still list it as a multilib. This patch adds a configuration option (--disable-ppc64-swfloat) to disable building this library.

Re: [PATCH] Add powerpc64-linux configuration options

2012-05-23 Thread Joseph S. Myers
On Wed, 23 May 2012, Michael Meissner wrote: An alternative would be for the powerpc64-linux case, should we just delete the software floating emulation multilib and stop using the -mstrict-align, since Linux only runs in big endian mode. The software emulation multilib would be built for

Re: Turn check macros into functions. (issue6188088)

2012-05-23 Thread Mike Stump
On May 23, 2012, at 3:36 PM, Lawrence Crowl wrote: For variables that are not optimized out of memory, I think this can work. But if you need to go for registers, Also trivial... Just one has to generate the context correctly. One has to save off the registers and then in the context

Re: [PATCH 2/2] Better system header location detection for built-in macro tokens

2012-05-23 Thread Jason Merrill
On 05/23/2012 04:52 AM, Dodji Seketeli wrote: I tried to do what the C FE seems to do, which is to consider that the default location (the global input_location variable) is on the LHS of the assignment (on the usi variable), rather than on the token that comes from DEC32_MAX. That makes

Re: [PATCH][4/n] referenced-vars TLC

2012-05-23 Thread H.J. Lu
On Wed, May 23, 2012 at 7:25 AM, H.J. Lu hjl.to...@gmail.com wrote: On Wed, May 23, 2012 at 5:00 AM, Richard Guenther rguent...@suse.de wrote: This finally switches us to not record global vars in referenced-vars. For this to work I had to re-engineer how we handle global var removal from

Re: [PATCH] Fix VRP handling of undefined state

2012-05-23 Thread H.J. Lu
On Fri, Jul 29, 2011 at 1:46 AM, Richard Guenther rguent...@suse.de wrote: I noticed that for binary expressions VRP contains the same bugs that CCP once did (it treats UNDEFINED * 0 as UNDEFINED).  Then I noticed we never hit this bug because we never end up with any range being UNDEFINED -

Re: [PATCH v2] ARM: Use different linker path for hardfloat ABI

2012-05-23 Thread Mike Frysinger
On Wednesday 23 May 2012 17:11:53 Michael Hope wrote: On 24 May 2012 02:16, Mike Frysinger wrote: On Wednesday 23 May 2012 04:17:51 Richard Guenther wrote: On Wed, 23 May 2012, Andreas Jaeger wrote: On Wednesday, May 23, 2012 09:56:31 Richard Earnshaw wrote: [...] This is a

Re: [PATCH] Hoist adjacent pointer loads

2012-05-23 Thread William J. Schmidt
On Wed, 2012-05-23 at 13:25 +0200, Richard Guenther wrote: On Tue, 22 May 2012, William J. Schmidt wrote: Here's a revision of the hoist-adjacent-loads patch. Besides hopefully addressing all your comments, I added a gate of at least -O2 for this transformation. Let me know if you

Re: fix install-no-fixedincludes mishaps

2012-05-23 Thread Ian Lance Taylor
On Wed, May 16, 2012 at 7:46 AM, Olivier Hainque hain...@adacore.com wrote:  2012-05-16  Olivier Hainque  hain...@adacore.com        libgcc/        * Makefile.in (install-unwind_h): Rename into ...        (install-unwind_h-forbuild): New target.        (all): Use it instead of the former