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

2011-04-04 Thread Kai Tietz
Hello, this patch fixes some issues in libjava for windows targets. It addresses for now only 32-bit target, but for JNICALL definition I introduced already logic for 64-bit windows target too as preparation. One nit remains, but I am not quite sure how to approach it. In ltmain.sh it is necessar

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

2011-04-04 Thread Paolo Bonzini
>>> Index: cgraphbuild.c >>> === >>> --- cgraphbuild.c.orig  2011-04-03 11:28:45.0 +0200 >>> +++ cgraphbuild.c       2011-04-03 11:31:21.0 +0200 >>> @@ -53,6 +53,12 @@ record_reference (tree *tp, int *walk_su >>>   tree

Re: [PATCH, Fortran] Correct declaration of frexp and friends

2011-04-04 Thread Tobias Burnus
Duncan Sands wrong: Ping? Pong. It helps to send Fortran patches also to fortran@ ... On 30/03/11 16:43, Duncan Sands wrote: While working on the dragonegg plugin I noticed that the Fortran front-end declares frexp with the parameters the wrong way round. Instead of double frexp(double x, i

Go patch committed: Use backend interface for labels and goto

2011-04-04 Thread Ian Lance Taylor
This patch to the Go compiler uses the backend interface for labels and goto statements. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian 2011-04-04 Ian Lance Taylor * go-gcc.c (class Blabel): Define. (Gcc_backend::make_expression):

Re: [PATCH] (partially) fix lto bootstrap (PR bootstrap/48148)

2011-04-04 Thread H.J. Lu
On Fri, Apr 1, 2011 at 2:18 PM, Jakub Jelinek wrote: > Hi! > > richi's lto-cgraph.c change actually didn't do anything, because > DECL_ABSTRACT_ORIGIN is still NULL there, see PR for details. > This patch instead doesn't try to force dies for DECL_ABSTRACT_ORIGIN > DECL_EXTERNAL decls, as dwarf2ou

Re: RFC: PATCH: Remove 26 element limit in vector

2011-04-04 Thread H.J. Lu
On Thu, Mar 31, 2011 at 5:05 AM, Kenneth Zadeck wrote: > we hit this limit trying to write the explicit semantics for a > vec_interleave_evenv32qi. > > ;;(define_insn "vec_interleave_evenv32qi" > ;;  [(set (match_operand:V32QI 0 "register_operand" "=r") > ;;    (vec_select:V32QI > ;;      (vec_con

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

2011-04-04 Thread H.J. Lu
On Mon, Apr 4, 2011 at 3:00 PM, H.J. Lu wrote: > On Sun, Apr 3, 2011 at 3:14 AM, Michael Matz wrote: >> Hi, >> >> On Thu, 31 Mar 2011, Richard Guenther wrote: >> >>> > In the meanwhile, is the below version okay? >>> >>> If it bootstraps & tests ok then yes.  The java parts look obvious. >> >> So

Re: Improve jump threading #1 of N

2011-04-04 Thread H.J. Lu
On Mon, Mar 28, 2011 at 11:33 AM, Jeff Law wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > On 03/26/11 11:43, Richard Guenther wrote: >> >>> Looks good to me apart from not using gsi_start_nondebug_bb in >>> thread_around_empty_block. > Ah.  I wasn't aware of gsi_start_nondebug_bb.  I

Re: [libgo] Account for mmap ignoring addr arg without MAP_FIXED (PR go/48240)

2011-04-04 Thread Ian Lance Taylor
Rainer Orth writes: > 2011-04-02 Rainer Orth > > PR go/48240 > * configure.ac: Check for mincore. > * configure: Regenerate. > * config.h.in: Regenerate. > * runtime/mem.c: Include unistd.h. > (addrspace_free): New function. > (runtime_SysMap): Retry 6

Re: [patch] make default linker --hash-style configurable option

2011-04-04 Thread Joseph S. Myers
On Tue, 5 Apr 2011, Matthias Klose wrote: > On 04.04.2011 20:17, Paul Pluzhnikov wrote: > > Greetings, > > > > Several Linux distributions (e.g. Fedora) carry local patches that turn > > on --hash-style=gnu for all links. > > > > Attached is a proposed patch (originally by Satoru Takabayashi) th

[wwwdocs] Remove manual formatting from the navigation style

2011-04-04 Thread Gerald Pfeifer
Probably a relict dating back a decade ago, these days ... really has no place, and our current CSS has the same effect anyway. Committed. Gerald Index: style.mhtml === RCS file: /cvs/gcc/wwwdocs/htdocs/style.mhtml,v retrieving revi

Split up config/i386/linux.h etc.

2011-04-04 Thread Joseph S. Myers
Following up on the changes I made in to separate config/linux.h into parts for GNU userspace (gnu-user.h) and parts for the Linux kernel (linux.h), this patch makes similar changes to config/i386/linux.h and config/i386/linux64.h. As with

Re: [libgo] Don't try to use uninitialized semaphores (PR go/48222)

2011-04-04 Thread Ian Lance Taylor
Rainer Orth writes: > 2011-04-03 Rainer Orth > > PR go/48222 > * runtime/malloc.goc (runtime_mallocinit): Call > runtime_Mprof_Init, runtime_initfintab. > * runtime/cpuprof.c (runtime_cpuprofinit): New function. > * runtime/runtime.h (runtime_cpuprofinit): Declare

[wwwdocs] obsolete gnu.css [3/4] and revamp navigation style

2011-04-04 Thread Gerald Pfeifer
Not perfect yet, but I now like it a LOT better than what we had just two days ago. And http://www.gnu.org/software/gcc/ starts to look somewhat reasonable again, too. Committed. Gerald Index: gcc.css === RCS file: /cvs/gcc/wwwdoc

Go patch committed: Use backend interface for return statements

2011-04-04 Thread Ian Lance Taylor
This patch uses the new backend interface for return statements. To simplify this, the frontend now always creates result parameters, even if they have no names. Lowering a return statement now assigns the values to the result parameters. This permits a return statement to just copy the values t

[google] Enable lightweight debug checks (issue4344067)

2011-04-04 Thread Paul Pluzhnikov
This patch adds lightweight debug checks (if enabled by macros). To be applied only to google/main branch. Tested by buildit --run-tests. 2011-04-04 Paul Pluzhnikov * libstdc++-v3/include/ext/vstring.h(revision 171950): Enable debug checks when __google_stl_debug_string

Re: Scheduler cleanups, 5/5

2011-04-04 Thread Bernd Schmidt
On 03/24/2011 02:19 PM, Bernd Schmidt wrote: > We can currently select an insn to be scheduled, only to find out that > it's not actually valid at the current time, either due to state > conflicts or being an asm with something else already scheduled in the > same cycle. Not only is this pointless,

Re: [patch] make default linker --hash-style configurable option

2011-04-04 Thread Matthias Klose
On 04.04.2011 20:17, Paul Pluzhnikov wrote: > Greetings, > > Several Linux distributions (e.g. Fedora) carry local patches that turn > on --hash-style=gnu for all links. > > Attached is a proposed patch (originally by Satoru Takabayashi) that makes > default hash style a configure option. > > Te

Re: Random cleanups [2/4]: canonicalize ctor values

2011-04-04 Thread H.J. Lu
On Sun, Apr 3, 2011 at 3:14 AM, Michael Matz wrote: > Hi, > > On Thu, 31 Mar 2011, Richard Guenther wrote: > >> > In the meanwhile, is the below version okay? >> >> If it bootstraps & tests ok then yes.  The java parts look obvious. > > So, we indeed can't remove the other calls to > canonicalize_

New Spanish PO file for 'gcc' (version 4.6.0)

2011-04-04 Thread Translation Project Robot
Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'gcc' has been submitted by the Spanish team of translators. The file is available at: http://translationproject.org/latest/gcc/es.po (This file, 'gcc-4.6.0.es.po', has just

[RFC] Remove kludge in commit_edge_insertions

2011-04-04 Thread Eric Botcazou
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 responsible for making sure that control flow is valid at all times. */ if (current_ir_ty

Re: [cxx-mem-model] C++ atomic tests (synchronized and relaxed)

2011-04-04 Thread Jason Merrill
On 04/04/2011 01:11 PM, Aldy Hernandez wrote: 1. Synchronized atomic load/stores: atomic_int atomi; long double j; Thread 1: j = 13.0; atomi.store(1); Thread 2: atomi.load(); As I understand it, the load/stores have acquire/release semantics, so the store to must happen before the store to .

Convert legacy ColdFire options to .opt aliases

2011-04-04 Thread Joseph S. Myers
This patch begins simplifying m68k option handling, in preparation for using the .opt Enum facility to process enumerated options, avoiding global state in option handlers and sharing option hanlders with the driver, by converting the legacy -m options for particular ColdFire CPUs into .opt Alias e

[google] Build libstdc++ exception throwing code with frame pointers (issue4348050)

2011-04-04 Thread Paul Pluzhnikov
This patch builds code which throws exceptions with -fno-omit-frame-pointer, so frame-based unwinder can unwind through it. To be applied only to google/main branch. Tested by buildit bootstrap, verified in the build log that -fno-omit-frame-pointer is applied to functexcept, eh_throw and eh_term

Re: [patch, testsuite, ia64] patch for gcc.dg/mtune.c

2011-04-04 Thread Steve Ellcey
On Sat, 2011-04-02 at 00:25 +, Joseph S. Myers wrote: > On Fri, 1 Apr 2011, Steve Ellcey wrote: > > > Looking into why this doesn't happen on other systems, it looks like > > the difference is that IA64 uses -mtune to set an enum variable and > > most (all?) the other platforms use it to set a

[patch, fortran, committed] Fix regression PR 48412

2011-04-04 Thread Thomas Koenig
Hello world, I have committed the attached patch to trunk as obvious after regression-testing. The problem was that the assignment statements had been added in the wrong order, which was fixed by reversing the order of the loops looking for common function calls in cfe_expr_0. Many thanks to

ARM generate constants for non load scheduling arch patch

2011-04-04 Thread sa...@hederstierna.com
Hi! I was recently cleaning my old sources attic, and I found this old GCC-patch for ARM that never was contributed. First I was thinking to just delete it, but then I thought that maybe someone can make use it, or be inspired to do something with it. The patch is 5-6 years old so I really thi

[patch][cprop.c] Simplify find_used_regs, iterate cprop_insn

2011-04-04 Thread Steven Bosscher
Hi, This patch has one significant change: cprop_insn would not iterate after a successful copy propagation, assuming the uses found with find_used_regs remain the same. This is not true in two cases: * find_used_regs found multiple uses of the same register, but that register was replaced by try

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

2011-04-04 Thread Basile Starynkevitch
Hello All, When gengtype is used in plugin mode, it should also generate various #define ggc_alloc_* macro definitions. It seems that the 4.6.0 gengtype don't do that (and that is a bug). The attached patch, to trunk rev171950, fixes this issue. # gcc/ChangeLog entry 2011-04-04 Ba

Re: [PATCH] allocate combine.c:LOG_LINKS in an alloc_pool

2011-04-04 Thread Nathan Froyd
On Mon, Apr 04, 2011 at 09:01:20PM +0200, Steven Bosscher wrote: > On Mon, Apr 4, 2011 at 8:49 PM, Nathan Froyd wrote: > > This patch does just what $SUBJECT suggests.  Benefits: > > > > - Smaller data structures in combine; > > - Freeing LOG_LINKS becomes much easier (don't have to transfer > >  

Re: [PATCH, PR43920, 5/9] Cross-jumping - Add missing use of return register.

2011-04-04 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 04/01/11 08:49, Tom de Vries wrote: > Reposting, with ChangeLog. > PR target/43920 > * function.c (emit_use_return_register_into_block): New function. > (thread_prologue_and_epilogue_insns): Use > emit_use_return_register_int

Re: [PATCH, PR43920, 4/9] Cross-jumping - Don't count use or clobber.

2011-04-04 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 04/01/11 08:48, Tom de Vries wrote: > Reposting, with ChangeLog. > PR target/43920 > * cfgcleanup.c (flow_find_cross_jump): Don't count USE or CLOBBER as > insn. OK. jeff -BEGIN PGP SIGNATURE- Version: GnuPG v1.4.11 (GN

Re: [testsuite, fortran] Compile gfortran.dg/bessel_[67].f90 with -mieee

2011-04-04 Thread Rainer Orth
Tobias Burnus writes: >> I've also had a look at the other gfortran testsuite failures on Tru64 >> UNIX V5.1B and IRIX 6.5 that couldn't be fixed that easily. >> How to best report them and have them investigated? I suspect that >> there might be only a few common issues, so a PR per failing tes

Re: [PATCH] allocate combine.c:LOG_LINKS in an alloc_pool

2011-04-04 Thread Steven Bosscher
On Mon, Apr 4, 2011 at 8:49 PM, Nathan Froyd wrote: > This patch does just what $SUBJECT suggests.  Benefits: > > - Smaller data structures in combine; > - Freeing LOG_LINKS becomes much easier (don't have to transfer >  everything to the INSN_LIST free list); > > Potential downsides: > > - Less s

[PATCH] allocate combine.c:LOG_LINKS in an alloc_pool

2011-04-04 Thread Nathan Froyd
This patch does just what $SUBJECT suggests. Benefits: - Smaller data structures in combine; - Freeing LOG_LINKS becomes much easier (don't have to transfer everything to the INSN_LIST free list); Potential downsides: - Less sharing of INSN_LIST nodes might mean more cache thrashing. Bootstr

Re: [Patch, Fortran] PR18918 - UCOBOUND coarray draft patch

2011-04-04 Thread Tobias Burnus
Daniel Kraft wrote: On 04/03/11 17:03, Tobias Burnus wrote: For the support, the scalarizer had to be modified and the cobounds had to be saved in the descriptor (and in TYPE_LANG_SPECIFIC(node)). I'm all but an expert on this area (maybe you could ask Mikael for his opinion on the scalarizer

Re: [PATCH, PR43920, 3/9] Cleanup.

2011-04-04 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 04/01/11 09:15, Tom de Vries wrote: > On 04/01/2011 05:01 PM, Jakub Jelinek wrote: >> On Fri, Apr 01, 2011 at 04:56:10PM +0200, Tom de Vries wrote: >>> Reposting, with ChangeLog. >> >> Similarly one of your patches had >> * gcc/cfgcleanup.c (wh

Re: [patch] make default linker --hash-style configurable option

2011-04-04 Thread Ian Lance Taylor
Rainer Orth writes: > ppluzhni...@google.com (Paul Pluzhnikov) writes: > >> Several Linux distributions (e.g. Fedora) carry local patches that turn >> on --hash-style=gnu for all links. > > Shouldn't configure verify that the linker used actually understands > that option? No, I don't think so.

Re: patch to solve PR48380

2011-04-04 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 04/01/11 21:06, Vladimir Makarov wrote: > The following patch is to solve the PR. The reason for the problem is > described on > > http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48380 > > I also found some problem with function fix_reg_equiv_init (i

Re: [patch] make default linker --hash-style configurable option

2011-04-04 Thread Rainer Orth
ppluzhni...@google.com (Paul Pluzhnikov) writes: > Several Linux distributions (e.g. Fedora) carry local patches that turn > on --hash-style=gnu for all links. Shouldn't configure verify that the linker used actually understands that option? Rainer -- --

[build] Support multilib testing in libgo

2011-04-04 Thread Rainer Orth
make check in libgo currently tests the default multilib only, among others because automake has no support for multilib testing. It would have to be especially careful here since for a DejaGnu testsuite, this is handled internally to dg, while for other testsuites, one needs to run make check in

[patch] make default linker --hash-style configurable option

2011-04-04 Thread Paul Pluzhnikov
Greetings, Several Linux distributions (e.g. Fedora) carry local patches that turn on --hash-style=gnu for all links. Attached is a proposed patch (originally by Satoru Takabayashi) that makes default hash style a configure option. Tested by doing native bootstrap and verifying that no --hash-st

Re: [patch][cprop.c] Allow splitting of critical edges to expose implicit sets

2011-04-04 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 04/01/11 07:55, Steven Bosscher wrote: > Hi, > > With this patch, it is OK for find_implicit_sets to split an edge if > that makes it possible to record an implicit set on that edge. This is > possible and cheap now because CPROP runs in cfglayout

Re: [patch][cprop.c] Make assignments pairs instead of (set (dest) (src))

2011-04-04 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 04/04/11 11:57, Steven Bosscher wrote: > Hi, > > This is mostly a small memory and pointer-chasing cleanup. We always > know exactly what form an assignment takes so there is no need to > record assignments as SET-rtxen. > > Bootstrapped and teste

Re: Scheduler cleanups, 1/N

2011-04-04 Thread Bernd Schmidt
On 04/04/2011 06:54 PM, H.J. Lu wrote: > On Mon, Apr 4, 2011 at 9:34 AM, Vladimir Makarov wrote: >> On 04/04/2011 11:07 AM, Bernd Schmidt wrote: >>> Ok after retest? >>> >> Ok, thanks. >> >> > > I bootstrapped it on Linux/x86-64 and checked it in. That was slightly premature, but the tests did n

Re: [patch][cprop.c] Random cleanups to un-gcse cprop.c

2011-04-04 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 03/31/11 15:38, Steven Bosscher wrote: > Hi, > > This is a boring one. An almost-perfect %s/gcse/cprop/g except for > flag_gcse, and a few other random cleanups. Hardly noteworthy. This > goes in after the cprop_cleanup_2.diff patch as obvious unle

Re: [patch][cprop.c] Use DF_INSN_INFO_DEFS cache also for mark_oprs_set

2011-04-04 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 03/31/11 15:16, Steven Bosscher wrote: > Hi, > > This is more of the same, really: oprs_not_set_p() and mark_oprs_set() > can use the DF_INSN_INFO cache. > > Bootstrapped and tested on x86_64-unknown-linux-gnu. OK? OK. jeff -BEGIN PGP SIGNATUR

Re: [patch ada]: Fix MSG_WAITALL handling for windows native targets

2011-04-04 Thread Eric Botcazou
> 2011-04-04 Kai Tietz > >PR ada/4773 >* s-oscons-tmplt.c (MSG_WAITALL): Define it for native windows >targets to flag value. How adding an #ifdef without the corresponding #endif is supposed to work? Attached patch applied. 2011-04-04 Eric Botcazou PR ada/47

Re: [cxx-mem-model] bitfield tests

2011-04-04 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 04/02/11 01:56, Richard Guenther wrote: > But well, I guess the thing I don't like about the standard is that it makes > people that have started to be somewhat aware about threading issues > _less_ aware of them by providing some "false" safety to

Re: [PATCH] cleanup gcse.c:canon_modify_mem_list

2011-04-04 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 04/03/11 19:44, Nathan Froyd wrote: > The patch below converts gcse.c:canon_modify_mem_list to hold VECs > instead of EXPR_LIST rtxes. I am ambivalent about the use of VECs in > canon_modify_mem_list; they will waste some memory compared to the > l

[patch][cprop.c] Make assignments pairs instead of (set (dest) (src))

2011-04-04 Thread Steven Bosscher
Hi, This is mostly a small memory and pointer-chasing cleanup. We always know exactly what form an assignment takes so there is no need to record assignments as SET-rtxen. Bootstrapped and tested on x86_64-unknown-linux-gnu. OK? Ciao! Steven * cprop.c (struct expr): Split 'expr' field in

[libgo] Account for mmap ignoring addr arg without MAP_FIXED (PR go/48240)

2011-04-04 Thread Rainer Orth
As I had suspected, all the go and libgo testsuite failures on 64-bit Solaris 10 were caused by mmap ignoring the addr argument without MAP_FIXED. That's the same problem already solved in gcc/config/host-solaris.c. The following code is inspired from mmap_fixed() there. If this turns out to be

[wwwdocs] obsolete gnu.css [2/4]: standardize style for

2011-04-04 Thread Gerald Pfeifer
This is part 2 of 4 to remove the now obsolete ("upstream") gnu.css stylesheet, and it generally simplifies and consolidates things, including removal of an extraneous style for html and a redundant one for libstdc++. Installed. Gerald Index: gcc.css =

Re: libgo patch committed: Add timeout for tests

2011-04-04 Thread Rainer Orth
Mike Stump writes: >> Ok, I'll see how long they take on my 250 MHz R10k MIPS :-) > > I'd rather slow environments and slow machines setup a scaling factor with > which they can expand the time as needed. So, using a rs232 port at 110 to > download testcases, scaling_factor=1000 or so. That's

Re: libgo patch committed: Add timeout for tests

2011-04-04 Thread Rainer Orth
Mike Stump writes: >> Shouldn't the default match the DejaGnu default of 300, though? > > No... I like the idea that people trim it down to something sane. If > nothing else, as a way to discourage them from big stupid test cases. No argument from me, since I've often been bitten by slower te

[cxx-mem-model] C++ atomic tests (synchronized and relaxed)

2011-04-04 Thread Aldy Hernandez
[Jason, Benjamin: It'd be nice to get input from y'all, since you are more intimately involved with the standard. I want to make sure I'm not misunderstanding things.] Hopefully a non-controversial feature, but given my luck, I doubt it :). Here are two additional tests to verify our atomic c

Re: Scheduler cleanups, 1/N

2011-04-04 Thread H.J. Lu
On Mon, Apr 4, 2011 at 9:34 AM, Vladimir Makarov wrote: > On 04/04/2011 11:07 AM, Bernd Schmidt wrote: >> >> I eventually managed to reproduce it, and even figured out what I'd >> stupidly been doing wrong with my bootstraps which caused me not to see >> this. >> >> There were a few places where l

Avoid global state in rx_handle_option

2011-04-04 Thread Joseph S. Myers
This patch stops the RX handle_option hook from using global state. The -mcpu= handling is changed to use Enum, while most -mint-register= handling is deferred to rx_option_override, similarly to how such options are handled for other targets. Note that the -mcpu= change has the semantic effect t

[Patch] bfin: addsi3 alterinative for IREGs.

2011-04-04 Thread Henderson, Stuart
This patch adds an IREG alternative to the addsi3 insn. 2011-04-04 Stuart Henderson From Bernd Schmidt * config/bfin/bfin.md (addsi3): Add an alternative for IREGS. Thanks, Stu Index: gcc/config/bfin/bfin.md === --- gc

Re: Scheduler cleanups, 1/N

2011-04-04 Thread Vladimir Makarov
On 04/04/2011 11:07 AM, Bernd Schmidt wrote: I eventually managed to reproduce it, and even figured out what I'd stupidly been doing wrong with my bootstraps which caused me not to see this. There were a few places where last_scheduled_insn wasn't just examined on its own, but code wanted to wa

[libgo] Don't try to use uninitialized semaphores (PR go/48222)

2011-04-04 Thread Rainer Orth
After some time with DTrace, I found what caused the intermittend assertion failures in thread.c: /vol/gcc/src/hg/trunk/local/libgo/runtime/thread.c:63: libgo assertion failure /vol/gcc/src/hg/trunk/local/libgo/runtime/thread.c:40: libgo assertion failure It turned out that sem_wait or sem_post w

Re: [build, lto] Only accept -fuse-linker-plugin if linker supports -plugin (PR lto/46944)

2011-04-04 Thread Rainer Orth
Richard Guenther writes: > I think we should do the linker version checks which relate to linker-plugin > use on the plugin-linker instead. So if I specify a separate but known > buggy linker I don't want it to be used by default. Here's a patch that does this. I'm not at all happy with the pa

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

2011-04-04 Thread Tom de Vries
Hi Jeff, On 04/01/2011 04:54 PM, 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 replace insn b b

RE: [Patch] Bfin: Ensure rotrsi and rotlsi don't accept non-const INTVALS

2011-04-04 Thread Henderson, Stuart
Yep, I'm seeing this behaviour (getting the error using your patch). But I'm confused as to why the define_expand is being considered when the predicate doesn't match. Apologies if this is a dim question, I'm still learning. -Original Message- From: Bernd Schmidt [mailto:ber...@codesour

[build, libjava] Avoid jar -@ usage message

2011-04-04 Thread Rainer Orth
While configuring libjava on Solaris, I regularly get a jar usage message since /usr/bin/jar doesn't support -@: Illegal option: @ Usage: jar {ctxui}[vfm0Me] [jar-file] [manifest-file] [entry-point] [-C dir] files ... [...] The following patch fixes this, bootstrapped on i386-pc-solaris2.11, ins

Re: [PATCH] make LABEL_DECL has its own rtx field for its associated CODE_LABEL

2011-04-04 Thread Nathan Froyd
On Mon, Apr 04, 2011 at 05:52:00PM +0200, Steven Bosscher wrote: > Have you looked into maybe putting the CODE_LABEL for a LABEL_DECL in > an on-the-side structure (hash table, whatever)? It looks like it is > only used during expansion of SWITCH statements. I haven't, though it'd be easy enough o

[wwwdocs] obsolete gnu.css [1/4]: style.mthml

2011-04-04 Thread Gerald Pfeifer
This is part 1 of 4 to remove the now obsolete ("upstream") gnu.css stylesheet. For our installation documentation we can now make the switch to solely use our own, gcc.css, stylesheet. Done thusly, will become active over night. Gerald Index: style.mhtml ===

Re: [PATCH] make LABEL_DECL has its own rtx field for its associated CODE_LABEL

2011-04-04 Thread Steven Bosscher
Hi Nathan, Have you looked into maybe putting the CODE_LABEL for a LABEL_DECL in an on-the-side structure (hash table, whatever)? It looks like it is only used during expansion of SWITCH statements. Ciao! Steven

Re: [PATCH] cleanup gcse.c:canon_modify_mem_list

2011-04-04 Thread Nathan Froyd
On Mon, Apr 04, 2011 at 05:42:42PM +0200, Steven Bosscher wrote: > Nice cleanup. I can't approve it but it looks alright to me. I suppose > you're planning something similar for modify_mem_list? That's the plan, along with numerous other users of {INSN,EXPR}_LIST. -Nathan

[wwwdocs] Fix footer CSS for serving from http://www.gnu.org

2011-04-04 Thread Gerald Pfeifer
It turns out that the gnu.org webmasters while restructuring things actually broke CSS for the copy of our web pages hosted on www.gnu.org. This fixes this, and also reduced font size from 80% to 75%. Gerald Index: gcc.css === RCS f

Re: [PATCH] cleanup gcse.c:canon_modify_mem_list

2011-04-04 Thread Steven Bosscher
Hi Nathan, Nice cleanup. I can't approve it but it looks alright to me. I suppose you're planning something similar for modify_mem_list? Ciao! Steven

Re: Scheduler cleanups, 1/N

2011-04-04 Thread H.J. Lu
On Mon, Apr 4, 2011 at 8:07 AM, Bernd Schmidt wrote: > On 04/02/2011 02:55 AM, H.J. Lu wrote: >> On Thu, Mar 24, 2011 at 6:07 AM, Bernd Schmidt >> wrote: >>> I have a number of patches that will be necessary for a new target. Some >>> of these can be applied now as cleanups, so I'm submit them n

Re: Scheduler cleanups, 1/N

2011-04-04 Thread Bernd Schmidt
On 04/02/2011 02:55 AM, H.J. Lu wrote: > On Thu, Mar 24, 2011 at 6:07 AM, Bernd Schmidt > wrote: >> I have a number of patches that will be necessary for a new target. Some >> of these can be applied now as cleanups, so I'm submit them now. >> >> This changes the schedule_block main loop not to m

Re: [PATCH] gcc is broken on i386 FreeBSD due to binutil upgrade

2011-04-04 Thread Loren J. Rittle
I really thought I already approved this patch when Steve pinged me (off-list) in Feb. Approved. -Loren On Sat, Apr 2, 2011 at 6:41 PM, Gerald Pfeifer wrote: > On Mon, 21 Feb 2011, Richard Guenther wrote: >>> 2011-02-20  Steven G. Kargl   >>> >>>        * gcc/config.gcc:  Include i386/t-crtstuff

Fix optc-gen.awk handling of variables with both Variable and Var

2011-04-04 Thread Joseph S. Myers
In the course of testing another patch I noticed an "excess elements in struct initializer" warning building options.c for iq2000-elf. The problem turns out to be that if a .opt variable is specified both with a Variable entry, without an initializer, and with Var on individual options, then it ge

Re: [Patch,AVR]: Fix PR45263

2011-04-04 Thread Georg-Johann Lay
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]: Fix PR45263

Re: [cxx-mem-model] new branch, new project

2011-04-04 Thread Aldy Hernandez
On 04/04/11 08:30, Aldy Hernandez wrote: mind updating svn.html with a short note on this new branch? Whoops, sorry about that. Applied attached patch. Sorry, wrong patch. This is the correct one. Index: svn.html === RCS file

Re: [cxx-mem-model] new branch, new project

2011-04-04 Thread Aldy Hernandez
mind updating svn.html with a short note on this new branch? Whoops, sorry about that. Applied attached patch. Index: svn.html === RCS file: /cvs/gcc/wwwdocs/htdocs/svn.html,v retrieving revision 1.159 diff -u -r1.159 svn.html --

Re: [cxx-mem-model] bitfield tests

2011-04-04 Thread Aldy Hernandez
(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 or even CPUs without a byte store. As per 5, it doesn't

Re: [cxx-mem-model] bitfield tests

2011-04-04 Thread Aldy Hernandez
But well, I guess the thing I don't like about the standard is that it makes people that have started to be somewhat aware about threading issues _less_ aware of them by providing some "false" safety to them. It really smells like a standard designed for a very high-level language where people

Re: [patch] Remove call to tidy_fallthru_edge in tree-inline.c

2011-04-04 Thread Richard Guenther
On Mon, Apr 4, 2011 at 9:12 AM, Eric Botcazou wrote: > Hi, > > tidy_fallthru_edges contains RTL-specific bits: > >          if (! (s->flags & EDGE_COMPLEX) >              && s->dest == c >              && !find_reg_note (BB_END (b), REG_CROSSING_JUMP, NULL_RTX)) >            tidy_fallthru_edge (s)

Re: [PATCH, PR43920, 1/9] ARM specific part.

2011-04-04 Thread Rainer Orth
Richard Guenther writes: > On Sun, Apr 3, 2011 at 9:34 PM, Sergey Ostanevich > wrote: >> I would recommend to use 'nm -S a.out' that gives >> >> [...] >> 004004a4 0054 T main >> [...] >> >> then you can provide a name for the routine you want to test for the size. > > That a

Re: [wwwdocs] Add Subversion revisions to the timeline

2011-04-04 Thread Richard Guenther
On Mon, Apr 4, 2011 at 8:05 AM, Diego Novillo wrote: > On Mon, Apr 4, 2011 at 01:04, Gerald Pfeifer wrote: > >> Richi did not like it, you seem to be in favor, Michael neutral to in >> favor. >> >> I tried finding an alternate place to put this information, such as >> releases.html or svn.html, b

Re: [google] Add testcase for relative paths for FDO (issue4354042)

2011-04-04 Thread Diego Novillo
On Sat, Apr 2, 2011 at 08:37, Diego Novillo wrote: > On Fri, Apr 1, 2011 at 19:37, Martin Thuresson wrote: > >> 2011-04-01  Martin Thuresson   >> >>        PR gcov-profile/47793 >>        * gcc/testsuite/gcc.dg/pr47793.c: New test > > OK. Committed in rev r171935. Given that you'll keep sending

Re: [PATCH, PR43920, 1/9] ARM specific part.

2011-04-04 Thread Richard Guenther
On Sun, Apr 3, 2011 at 9:34 PM, Sergey Ostanevich wrote: > I would recommend to use 'nm -S a.out' that gives > > [...] > 004004a4 0054 T main > [...] > > then you can provide a name for the routine you want to test for the size. That also sounds reasonable. Is nm -S more port

[pph] Fix ICE with -fpph-hdr= (issue4316055)

2011-04-04 Thread Diego Novillo
We were not allocating memory for include_pph_mapping for -fpph-hdr= 2011-04-04 Diego Novillo * c-opts.c (add_pph_header_map): Create include_pph_mapping if it was NULL. diff --git a/gcc/c-family/c-opts.c b/gcc/c-family/c-opts.c index c774dca..551db04 100644 --- a/gcc/c-famil

Re: [2/2] Reducing the overhead of dwarf2 location tracking

2011-04-04 Thread Richard Sandiford
Richard Sandiford writes: > gcc/ > * dwarf2out.c (dw_loc_list_node): Add resolved_addr and replaced. > (cached_dw_loc_list_def): New structure. > (cached_dw_loc_list): New typedef. > (cached_dw_loc_list_table): New variable. > (cached_dw_loc_list_table_hash): New func

PR target/46329: Reject Neon structure constants

2011-04-04 Thread Richard Sandiford
This patch fixed PR target/46329, which is a problem that occurs when trying to reload: (set (reg:OI foo) (const_int 0)) There is no move alternative for moving constants directly into large numbers of VFPs, and it probably isn't going to be sensible to do that for all constants, even if prop

Re: Target header etc. cleanup patch

2011-04-04 Thread Joseph S. Myers
Here's a version of the patch updated for current trunk, since the pdp11 IRA_COVER_CLASSES removal has now been committed separately. OK to commit? 2011-04-04 Joseph Myers * config/alpha/alpha.c (struct machine_function): Use rtx, not struct rtx_def *. * config/bfin/

RE: [Patch] bfin: move loop invariants out of loop

2011-04-04 Thread Henderson, Stuart
Do you mean you plan to implement a generic solution in postreload? I have no objections to dropping this in that case. I'll carry on using the patch in our local 4.5 release (for what it's worth). -Original Message- From: Bernd Schmidt [mailto:ber...@codesourcery.com] Sent: 31 March 20

Re: [Patch, Fortran] PR18918 - UCOBOUND coarray draft patch

2011-04-04 Thread Daniel Kraft
Hi Tobias, On 04/03/11 17:03, Tobias Burnus wrote: For the support, the scalarizer had to be modified and the cobounds had to be saved in the descriptor (and in TYPE_LANG_SPECIFIC(node)). I'm all but an expert on this area (maybe you could ask Mikael for his opinion on the scalarizer changes)

Add a mode argument to LEGITIMATE_CONSTANT_P, and make it a hook

2011-04-04 Thread Richard Sandiford
Pretty much as the subject says. This is a prerequisite for fixing target/46329. I've used the mode argument instead of GET_MODE where it seems appropriate. In the end, the only GET_MODEs I kept were in explicit checks for VOIDmode. E.g.: /* double integer constants are ok. */ if (mode == V

Add a mode argument to cannot_force_const_mem

2011-04-04 Thread Richard Sandiford
I'm about to post a patch that adds a mode argument to LEGITIMATE_CONSTANT_P. Since LEGITIMATE_CONSTANT_P is closely associated with force_const_mem (by explicit calls on some targets), I thought I'd better add a mode argument there too. The mode argument will be used more after that L_C_P patch.

Re: Add ToLower .opt facility

2011-04-04 Thread Nick Clifton
Hi Joseph, This patch adds a ToLower .opt feature, for option arguments that are case-insensitive and should be converted to a canonical lowercase form. This is relevant to at least mips, mn10300 and rx. This patch makes rx use it, thereby causing an existing spec that tested only for a lowerc

Re: [wwwdocs] Remove unused styles from gnu.css

2011-04-04 Thread Gerald Pfeifer
On Sun, 3 Apr 2011, Gerald Pfeifer wrote: > And this is just more of the same which I missed in my first > round. Three's the charm. Also installed. Gerald Index: gnu.css === RCS file: /cvs/gcc/wwwdocs/htdocs/gnu.css,v retrieving r

[committed] Fix builds of pdp11

2011-04-04 Thread Richard Sandiford
Another trivial fix, this time for pdp11. Tested by building cc1 on that target. Committed as obvious. Richard gcc/ * config/pdp11/pdp11.h (IRA_COVER_CLASSES): Delete. Index: gcc/config/pdp11/pdp11.h === --- gcc/config/pd

[committed] Fix builds of vax-netbsdelf

2011-04-04 Thread Richard Sandiford
Trivial fix to make vax-netbsdelf build again. Tested by building cc1 on that target. Committed as obvious. Richard gcc/ * config/vax/vax.c: Include reload.h. Index: gcc/config/vax/vax.c === --- gcc/config/vax/vax.c

Re: [PATCH] Fix VTA updating in the combiner (PR debug/48343)

2011-04-04 Thread Jakub Jelinek
On Sun, Apr 03, 2011 at 11:39:22AM +0200, Eric Botcazou wrote: > > I think we need to update there in all cases. The reason we don't need to > > update beyond i3 resp. undobuf.other_insn is that DF guarantees us that > > there won't be debug insns referring to those pseudos afterwards, otherwise >

[patch ada]: Last fix for PR ada/47163 on windows native hosts

2011-04-04 Thread Kai Tietz
Hello, as described in PR, there is a failure to copy adainclude/adalib libraries in MSYS. This is caused by LN_S set to cp -p for MSYS. This option for cp is not suitable to copy folder to folder in MSYS, and the additional option -r is necessary. ChangeLog 2011-04-04 Kai Tietz PR ada

Re: [patch ada]: Fix MSG_WAITALL handling for windows native targets

2011-04-04 Thread Kai Tietz
2011/4/4 Thomas Quinot : > * Kai Tietz, 2011-04-04 : > >> So here is the patch without the part in g-socthi-mingw.adb. > > Thanks, OK for me. > > -- > Thomas Quinot, Ph.D. ** qui...@adacore.com ** Senior Software Engineer >               AdaCore -- Paris, France -- New York, USA > Committed at rev

  1   2   >