libgo patch committed: Always use $(AM_LDFLAGS)

2011-04-06 Thread Ian Lance Taylor
This patch to the libgo Makefile ensures that we always use $(AM_LDFLAGS) when linking libgo.la. This is required so that we pass -fsplit-stack to the link command, which tells the gcc driver to wrap pthread_create. Without this, most Go tests failed when not using gold. Bootstrapped and ran Go

Remove obsolete ??? comment in profile.c

2011-04-06 Thread Eric Botcazou
This is location_t curr_location = single_succ_edge (bb)-goto_locus; /* ??? The FILE/LINE API is inconsistent for these cases. */ output_location (LOCATION_FILE (curr_location), LOCATION_LINE (curr_location), offset, bb);

Re: [patch, ARM] Make 128 bits the default vector size for NEON

2011-04-06 Thread Ira Rosen
On 5 April 2011 15:30, Ramana Radhakrishnan ramana.radhakrish...@linaro.org wrote: On 31/03/11 12:39, Ira Rosen wrote: Hi, This patch changes NEON's default vector size from 64 to 128 bits. The patch doesn't touch mvectorize-with-neon-quad, but removes the uses of

[patch] Preserve return statements at -O0

2011-04-06 Thread Eric Botcazou
Hi, this patchlet ensures that return statements are preserved at -O0 so that you can put a breakpoint on them in the debugger. Bootstrapped/regtested on x86_64-suse-linux, OK for the mainline? 2011-04-06 Eric Botcazou ebotca...@adacore.com * gimple-low.c (lower_gimple_return):

Re: [PATCH] fix PR bootstrap/48471

2011-04-06 Thread Tristan Gingold
On Apr 6, 2011, at 3:06 AM, Nathan Froyd wrote: My changes that moved bits of final.c to dbxout.c caused compilation failures on targets that *didn't* enable DBX_DEBUGGING_INFO, since some now-static symbols could be unused. To fix this, the patch below moves all the new static variables

Re: typo in gcc manual

2011-04-06 Thread Jonathan Wakely
On 6 April 2011 04:15, ali hagigat wrote: 3.15 Specifying subprocesses and the switches to pass to them *[spec_name]: If this results in an empty string then the spec will be deleted. (Or, if the spec did not exist, then nothing will happened.)

[PATCH][1/2] Merge call stmt cost handling from pretty-IPA

2011-04-06 Thread Richard Guenther
This merges the call stmt cost handling from pretty IPA together with dependent cleanups. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. Richard. 2011-04-05 Richard Guenther rguent...@suse.de PR tree-optimization/47663 * cgraph.h (struct

Re: [testsuite] Fix gcc.dg/torture/pr47917.c on IRIX 6.5

2011-04-06 Thread Rainer Orth
Eric Botcazou ebotca...@adacore.com writes: The new gcc.dg/torture/pr47917.c test currently fails the execution test on IRIX 6.5. To get a C99-conformant snprintf, one needs to include stdio.h with _XOPEN_SOURCE defined as 500. The following patch does this. Likewise on Solaris 8 (It's

[PATCH][2/2] Simplify call stmt handling in stmt costs

2011-04-06 Thread Richard Guenther
With the call stmt cost merge from pretty-IPA we can now simplify their handling in estimate_num_insns. Instead of fancy ways to look at types simply account for the actual arguments and returns. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. Richard. 2011-04-06

Re: [patch] Preserve return statements at -O0

2011-04-06 Thread Richard Guenther
On Wed, Apr 6, 2011 at 8:57 AM, Eric Botcazou ebotca...@adacore.com wrote: Hi, this patchlet ensures that return statements are preserved at -O0 so that you can put a breakpoint on them in the debugger. Bootstrapped/regtested on x86_64-suse-linux, OK for the mainline? Ok. Thanks, Richard.

Re: Separate C-family hooks better from general hooks

2011-04-06 Thread Diego Novillo
On Wed, Apr 6, 2011 at 03:27, Joseph S. Myers jos...@codesourcery.com wrote: 2011-04-05  Joseph Myers  jos...@codesourcery.com        * doc/tm.texi.in: Document C target hooks as separate from general        target hooks.        * doc/tm.texi: Regenerate.        * genhooks.c (struct

[Patch ARM] Committed fix for PR43920 testcase

2011-04-06 Thread Ramana Radhakrishnan
Hi, I've committed the following patch under the obvious rule for the testcase for PR43920. There seemed to have been an accidental duplication of the function in the patch committed. Tested in a cross compiler to arm-linux-gnueabi with just that test to armv7-a and made sure that it

Re: [PATCH][ARM] Tweak arm_class_likely_spilled_p, MODE_BASE_REG_CLASS for Thumb-2

2011-04-06 Thread Andrew Stubbs
On 30/03/11 16:13, Richard Earnshaw wrote: 2011-02-14 Andrew Stubbsa...@codesourcery.com Julian Brownjul...@codesourcery.com Mark Shinwellshinw...@codesourcery.com gcc/ * config/arm/arm.h (arm_class_likely_spilled_p): Check against LO_REGS

[PATCH] Merge early inliner cleanup from pretty-ipa

2011-04-06 Thread Richard Guenther
This merges the last patch from pretty-ipa, the early inliner cleanup. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. Richard. 2011-02-22 Richard Guenther rguent...@suse.de * ipa-inline.c (enum inlining_mode): Remove. (cgraph_flatten): Use some other

Remove old toplevel subdirectory support

2011-04-06 Thread Joseph S. Myers
This patch removes toplevel support for various subdirectories not present in the gcc or src repositories, as agreed in the thread including http://gcc.gnu.org/ml/gcc/2011-03/msg00501.html. Some other directories may in fact also be no longer useful to support building at toplevel, or may be

Re: Remove old toplevel subdirectory support

2011-04-06 Thread Paolo Bonzini
On 04/06/2011 02:07 PM, Joseph S. Myers wrote: This patch removes toplevel support for various subdirectories not present in the gcc or src repositories, as agreed in the thread includinghttp://gcc.gnu.org/ml/gcc/2011-03/msg00501.html. Some other directories may in fact also be no longer useful

RE: [Ping #2][Patch, testsuite]: Don't xfail sibcalls on AVR

2011-04-06 Thread Weddington, Eric
-Original Message- From: Georg-Johann Lay [mailto:a...@gjlay.de] Sent: Wednesday, April 06, 2011 4:55 AM To: Weddington, Eric Cc: gcc-patches@gcc.gnu.org; Anatoly Sokolov; Denis Chertykov Subject: Re: [Ping #2][Patch, testsuite]: Don't xfail sibcalls on AVR Target avr now

PowerPC64 non-delegitimized unspecs

2011-04-06 Thread Alan Modra
I've noticed a number of tests fail on mainline with complaints about non-delegitimized UNSPECs. The reason why rs6000_delegitimze_address isn't working is that our insn matching doesn't allow a stack slot in place of the REG. I could have expanded the condition to include stack slots, but it

Re: [patch libjava]: Do some fixes for windows native targets

2011-04-06 Thread Kai Tietz
2011/4/5 Tom Tromey tro...@redhat.com: Kai == Kai Tietz ktiet...@googlemail.com writes: Kai this patch fixes some issues in libjava for windows targets.  It Kai addresses for now only 32-bit target, but for JNICALL definition I Kai introduced already logic for 64-bit windows target too as

Re: [Ping #2][Patch, testsuite]: Don't xfail sibcalls on AVR

2011-04-06 Thread Georg-Johann Lay
Weddington, Eric schrieb: -Original Message- From: Georg-Johann Lay [mailto:a...@gjlay.de] To: Weddington, Eric Please commit, Johann. Thanks. I don't have write privilege, and I don't think I want write access at the moment. Oh! I didn't know. I thought you did. I signed a

Re: [Patch,AVR]: Fix PR45263

2011-04-06 Thread Georg-Johann Lay
Georg-Johann Lay schrieb: Weddington, Eric schrieb: -Original Message- From: Georg-Johann Lay [mailto:a...@gjlay.de] Sent: Friday, April 01, 2011 4:01 PM To: Georg-Johann Lay Cc: gcc-patches@gcc.gnu.org; Denis Chertykov; Anatoly Sokolov; Weddington, Eric Subject: Re: [Patch,AVR]:

Re: [patch, ARM] Make 128 bits the default vector size for NEON

2011-04-06 Thread Hans-Peter Nilsson
Date: Thu, 31 Mar 2011 13:39:05 +0200 From: Ira Rosen ira.ro...@linaro.org This patch changes NEON's default vector size from 64 to 128 bits. I'm wondering, are there NEON-specific measurements to support this change? A colleague of mine implemented support for 64- and 128-bit NEON for RAPP

[Patch][AVR]: Initial builtins support

2011-04-06 Thread Georg-Johann Lay
This patch adds builtin support for some RTL builtins to avr backend. One builtin implements loop for delay of specific number of ticks (under the assumption IRQs are off), others simply map to machine instructions like SEI, CLI, NOP, SLEEP, WDR, FMUL, FMAULS, FMULSU. In addition, builtin macros

Re: [patch libjava]: Do some fixes for windows native targets

2011-04-06 Thread Kai Tietz
2011/4/6 Kai Tietz ktiet...@googlemail.com: 2011/4/5 Tom Tromey tro...@redhat.com: Kai == Kai Tietz ktiet...@googlemail.com writes: Kai this patch fixes some issues in libjava for windows targets.  It Kai addresses for now only 32-bit target, but for JNICALL definition I Kai introduced

Re: PowerPC64 non-delegitimized unspecs

2011-04-06 Thread Alan Modra
On Wed, Apr 06, 2011 at 02:45:49PM +0200, Jakub Jelinek wrote: On Wed, Apr 06, 2011 at 10:06:11PM +0930, Alan Modra wrote: I've noticed a number of tests fail on mainline with complaints about non-delegitimized UNSPECs. The reason why rs6000_delegitimze_address isn't working is that our

[testsuite] Account for absolute path to ranlib on Darwin (PR testsuite/48480)

2011-04-06 Thread Rainer Orth
As reported in the PR, we can get ranlib messages with an absolute path to ranlib when running the boehm-gc testsuite on x86_64-apple-darwin10. This differs from what Iain had reported for powerpc-apple-darwin9. The following patch accounts for that, tested by myself in tclsh and by Dominique on

Re: [cxx-mem-model] bitfield tests

2011-04-06 Thread Michael Matz
Hi, On Mon, 4 Apr 2011, Aldy Hernandez wrote: (5) Do we agree that all such cpus use a byte-granular modification mask? Now, as of (0) I might agree to disregard the original Alpha, but as the embedded world moves to SMP I'm not sure we can disregard non-cache coherent NUMA setups

Go patch committed: Use backend interface for if statements

2011-04-06 Thread Ian Lance Taylor
This patch to the Go frontend uses the backend interface for if statements. I also renamed some of the temporary conversion functions to use shorter names. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian 2011-04-06 Ian Lance Taylor i...@google.com

Re: RFA: MN10300: Handle MDR register in movqi/movhi pattern.

2011-04-06 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 04/06/11 03:38, Nick Clifton wrote: Hi Jeff, Hi Alex, Hi Richard, The MN10300 gcc port currently does not build on the mainline because of: In file included from gcc/unwind-dw2-fde.c:35:0: gcc/unwind-pe.h: In function

Re: [PATCH 1/n, i386]: Merge SSE and AVX patterns using enable attribute.

2011-04-06 Thread Uros Bizjak
On Wed, Apr 6, 2011 at 1:12 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Nice patch. But I have two question: - What will be with future extensions? E.g. what if `avxXX` will appear, do you think that common parts with previous extentions should be shrinked in that way? The patch in fact

Re: [RFC] Remove kludge in commit_edge_insertions

2011-04-06 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 04/04/11 15:07, Eric Botcazou wrote: commit_edge_insertions contains this kludge: /* In the old rtl CFG API, it was OK to insert control flow on an edge, apparently? In cfglayout mode, this will *not* work, and the caller is

Rename LINUX_TARGET_OS_CPP_BUILTINS and LINUX_DYNAMIC_LINKER*

2011-04-06 Thread Joseph S. Myers
This patch, relative to a tree with http://gcc.gnu.org/ml/gcc-patches/2011-04/msg00276.html (pending review) applied, further cleans up the handling of GNU-userspace targets by renaming LINUX_TARGET_OS_CPP_BUILTINS (a macro also defined in headers such as kfreebsd-gnu.h) to

Re: PATCH: PR middle-end/48440: [4.7 Regression] FAIL: gcc.c-torture/compile/labels-3.c

2011-04-06 Thread H.J. Lu
On Tue, Apr 5, 2011 at 3:29 AM, Richard Guenther richard.guent...@gmail.com wrote: On Tue, Apr 5, 2011 at 8:44 AM, Paolo Bonzini bonz...@gnu.org wrote: Index: cgraphbuild.c === --- cgraphbuild.c.orig  2011-04-03 11:28:45.0

Re: [PATCH, ARM] Switch to EABI version 5 for RTEMS

2011-04-06 Thread Sebastian Huber
On 06/04/11 18:24, Ralf Corsepius wrote: On 04/06/2011 05:20 PM, Sebastian Huber wrote: there were several requests for ARM Cortex-M support on RTEMS recently. The first step towards this is a suitable ARM tool chain. I want to use this event to clean up the multilibs and switch to the

[PATCH 2/n, i386]: Merge SSE and AVX patterns using enable attribute.

2011-04-06 Thread Uros Bizjak
Hello! This patch continues merging of SSE and AVX patterns in Parallel floating point logical operations, Parallel single-precision floating point conversion operations and Parallel double-precision floating point conversion operations sections of sse.md. Additionally, it introduces handling of

Re: [PATCH, PR43920, 6/9] Cross-jumping - Use reg-notes.

2011-04-06 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 04/01/11 08:54, Tom de Vries wrote: On 03/31/2011 11:16 PM, Tom de Vries wrote: On 03/31/2011 08:52 PM, Jeff Law wrote: On 03/31/11 12:42, Tom de Vries wrote: Uses regnotes to analyze whether we can replace insn a by insn b, even if we cannot

Re: Fix powerpc rs6000_stack_info ICE

2011-04-06 Thread David Edelsohn
On Wed, Apr 6, 2011 at 8:18 AM, Alan Modra amo...@gmail.com wrote: The ENABLE_CHECKING tests I added at the end of rs6000_stack_info are now asserting on gcc.c-torture/execute/20041011-1.c and other testcases.  The cause is the late running DSE pass removing a set of LR, which means lr_save_p

Re: [RFC] Remove kludge in commit_edge_insertions

2011-04-06 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 04/06/11 12:01, Steven Bosscher wrote: On Wed, Apr 6, 2011 at 6:52 PM, Jeff Law l...@redhat.com wrote: What about when we have a PHI, which we eliminate by inserting insns on edges and those insns actually form a loop? You can see an example of

Re: [patch] Preserve return statements at -O0

2011-04-06 Thread Steven Bosscher
On Wed, Apr 6, 2011 at 8:57 AM, Eric Botcazou ebotca...@adacore.com wrote: this patchlet ensures that return statements are preserved at -O0 so that you can put a breakpoint on them in the debugger. Isn't one of the effects of this patch to generate debug info for the artificial label decl?

Re: [RFC] Remove kludge in commit_edge_insertions

2011-04-06 Thread Steven Bosscher
On Wed, Apr 6, 2011 at 6:52 PM, Jeff Law l...@redhat.com wrote: What about when we have a PHI, which we eliminate by inserting insns on edges and those insns actually form a loop?  You can see an example of this in PR48389. AFAIU the patch doesn't change behavior for the GIMPLE CFG. It only

[PATCH] Cleanup, use add_to_hard_reg_set instead of SET_HARD_REG_BIT loops.

2011-04-06 Thread Anatoly Sokolov
Hi. This patch converts loops of SET_HARD_REG_BIT in to add_to_hard_reg_set functions call. The patch has been bootstrapped on and regression tested on x86_64-unknown-linux-gnu for c. OK to install? * expr.c (expand_expr_real_1): Use add_to_hard_reg_set function instead

Re: [patch] Preserve return statements at -O0

2011-04-06 Thread Eric Botcazou
Isn't one of the effects of this patch to generate debug info for the artificial label decl? No, DECL_IGNORED_P is preserved on the label decl. When I first saw your patch, I thought FORCED_LABEL(), perhaps I'm not sure that it's any better than your solution... Have you tried that? No,

[Patch, Fortran, committed] Some coarray fixes (PR 18918 and 48477)

2011-04-06 Thread Tobias Burnus
I have committed the following three rather obvious patches (separately): 1. libgfortran/caf/mpi.c: I am now finally convinced that one indeed need to make use of MPI_Initialized's return value when calling MPI_Finalize. Thanks to all who pointed it out several times. 2.

Re: [RFC] Remove kludge in commit_edge_insertions

2011-04-06 Thread Steven Bosscher
On Wed, Apr 6, 2011 at 8:15 PM, Jeff Law l...@redhat.com wrote: AFAIU the patch doesn't change behavior for the GIMPLE CFG. It only affects RTL. But the elimination of the PHI results in creating RTL that is inserted on a CFG edge. Yes, but gimple_expand_cfg() calls

Re: [RFC] Remove kludge in commit_edge_insertions

2011-04-06 Thread Steven Bosscher
On Wed, Apr 6, 2011 at 8:37 PM, Steven Bosscher stevenb@gmail.com wrote: On Wed, Apr 6, 2011 at 8:15 PM, Jeff Law l...@redhat.com wrote: AFAIU the patch doesn't change behavior for the GIMPLE CFG. It only affects RTL. But the elimination of the PHI results in creating RTL that is

Re: [Patch][AVR]: Initial builtins support

2011-04-06 Thread Georg-Johann Lay
Added Anatoly and Eric to changelog entry Johann 2011-04-06 Anatoly Sokolov ae...@post.ru Eric Weddington eric.wedding...@atmel.com Georg-Johann Lay a...@gjlay.de * config/avr/avr.c: (insn-codes.h, optabs.h, langhooks.h): New Includes

Re: [RFC] Remove kludge in commit_edge_insertions

2011-04-06 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 04/06/11 12:37, Steven Bosscher wrote: On Wed, Apr 6, 2011 at 8:15 PM, Jeff Law l...@redhat.com wrote: AFAIU the patch doesn't change behavior for the GIMPLE CFG. It only affects RTL. But the elimination of the PHI results in creating RTL

[PATCH] Remove ASM_OUTPUT_BSS macro.

2011-04-06 Thread Anatoly Sokolov
Hi. No one back end does not use ASM_OUTPUT_BSS macro now, this patch remove it. The GCC have more flexible ASM_OUTPUT_ALIGNED_BSS macro. The patch has been bootstrapped on and regression tested on x86_64-unknown-linux-gnu for c. OK to install? * doc/tm.texi.in (ASM_OUTPUT_BSS):

Re: [RFC] Remove kludge in commit_edge_insertions

2011-04-06 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 04/06/11 12:39, Steven Bosscher wrote: On Wed, Apr 6, 2011 at 8:37 PM, Steven Bosscher stevenb@gmail.com wrote: On Wed, Apr 6, 2011 at 8:15 PM, Jeff Law l...@redhat.com wrote: AFAIU the patch doesn't change behavior for the GIMPLE CFG. It

[MIPS] Remove REG_OK_FOR_BASE_P and REG_OK_FOR_INDEX_P macros

2011-04-06 Thread Anatoly Sokolov
Hello. This patch remove unused REG_OK_FOR_BASE_P and REG_OK_FOR_INDEX_P macros from the MIPS back end. Bootstrapped and regression tested on mips64el-unknown-linux-gnu. OK to install? * config/mips/mips.h (REG_MODE_OK_FOR_BASE_P, REG_OK_FOR_INDEX_P): Remove macros.

Convert legacy m68k options to .opt aliases

2011-04-06 Thread Joseph S. Myers
Similar to http://gcc.gnu.org/ml/gcc-patches/2011-04/msg00265.html, this patch converts legacy m68k options for non-ColdFire CPUs into aliases for the corresponding -mcpu= options. (Note that -mcpu32 is an alias for -mcpu=68332 rather than -mcpu=cpu32, to match the old code in

Re: Cloog ISL - and linking of libisl

2011-04-06 Thread Jack Howarth
On Tue, Feb 15, 2011 at 08:07:43PM +0100, Tobias Burnus wrote: Dear Sebastian and Tobias, hello all, I think GCC should automatically link libisl (-lisl) if one configures GCC to use cloog-isl. That's the first patch at http://gcc.gnu.org/ml/gcc-patches/2010-11/msg01272.html , which

Re: [wwdocs] add libstdc++ docs for 4.6.0

2011-04-06 Thread Benjamin Kosnik
I ended up fiddling a bit with permissions and such, but everything should be in place now and I also committed the web patch on your behalf. Thanks! This pleases me a great deal. Testing the links, there is one thing does does not work, and where I was not sure how to address it: The

[PATCH 3/7] cgraph_node - cgraph_get_node with asserts

2011-04-06 Thread Martin Jambor
Hi, the patch below changes a number of calls to cgraph_node to calls to cgraph_get_node. These calls should never return NULL but because the callers do not immediately dereference the pointer they get, a NULL value would result in a segmentation fault at some later point, obscuring the real

[PATCH 2/7] cgraph_node - cgraph_get_node conversions accepting NULL results

2011-04-06 Thread Martin Jambor
Hi, this patch converts a number of calls to cgraph_node to calls to cgraph_get_node and provides means to deal with returned NULL value. These are essentially the places where lazy node creation was happening for no good reason. Bootstrapped and tested separately on x86_64-linux without any

[PATCH 1/7] Simple cgraph_node - cgraph_get_node conversions

2011-04-06 Thread Martin Jambor
Hi, this patch changes most of current calls to cgraph_node to calls to cgraph_get_node. The function should never return NULL in he contexts of the callers below and either the probability is so low that it does not warrant an assert (like cgraph_get_node (current_function_decl) or the result

[patch] Import iota and is_sorted from std to __gnu_cxx in C++0x mode. (issue4384043)

2011-04-06 Thread Jeffrey Yasskin
In C++0x mode, import iota and is_sorted from namespace std into namespace __gnu_cxx to avoid ambiguous call errors. copy_n would be a good candidate for this too, except it was standardized with a different return type. I think this is a candidate to backport to libstdc++-4.6.1. Tested:

Re: [patch] Import iota and is_sorted from std to __gnu_cxx in C++0x mode. (issue4384043)

2011-04-06 Thread Paolo Carlini
Hi, I think this is a candidate to backport to libstdc++-4.6.1. Ok mainline and 4_6-branch if testing passes. Thanks, Paolo

Re: PATCH [trunk] gengtype should generate ggc_alloc macros in plugin mode.

2011-04-06 Thread Laurynas Biveinis
Hello Basile - The patch is correct in general. But a few nitpicks: 2011-04-04 Basile Starynkevitch bas...@starynkevitch.net * gengtype.c (write_typed_alloc_def): Gets extra outf_p argument and use it. ...: New argument f. Use it instead of header_file.