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

2011-04-05 Thread Ira Rosen
On 6 April 2011 08:22, Ira Rosen wrote: > > Added a RejectNegative, changed "illegal" to "invalid" and prepared a > wwwdocs patch. > The testing finished successfully, except for the test in PR > target/48454 that fails with -mvectorize-with-neon-quad without the > patch as well. There is an unre

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

2011-04-05 Thread Kai Tietz
2011/4/6 Arnaud Charlet : >> Since then, the behavior you're relying on has been adopted by POSIX, and >> I think all or nearly all of the UNIX vendors have implemented it in their >> own versions of ln, so these days you may no longer care. > > To be more precise, we adopted this scheme at least 1

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

2011-04-05 Thread Arnaud Charlet
> Since then, the behavior you're relying on has been adopted by POSIX, and > I think all or nearly all of the UNIX vendors have implemented it in their > own versions of ln, so these days you may no longer care. To be more precise, we adopted this scheme at least 12 years ago on a wide range of O

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

2011-04-05 Thread Ira Rosen
On 5 April 2011 15:30, Ramana Radhakrishnan 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 TARGET_NEON_VECTORIZE_QUAD. >> Following J

Remove obsolete ??? comment in profile.c

2011-04-05 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);

libgo patch committed: Always use $(AM_LDFLAGS)

2011-04-05 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

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

2011-04-05 Thread Russ Allbery
Arnaud Charlet writes: > That's wrong. The above is > $(LN_S) /full/path/to/a_file a_dir > which is quite different from > $(LN_S) a_file a_dir > and will work equally well with ln or ln -s or cp -p. This is not historically portable for ln -s. Passing in a directory as the second argument

Re: [PATCH, ARM] PR47855 Compute attr "length" for some thumb2 insns

2011-04-05 Thread Carrot Wei
Committed as r172017. thanks Carrot On Tue, Apr 5, 2011 at 9:32 PM, Ramana Radhakrishnan wrote: > >> >> The patch was tested on arm qemu without regression, OK for install? > > Ok if no regressions. > > cheers > Ramana >

Separate C-family hooks better from general hooks

2011-04-05 Thread Joseph S. Myers
This patch separates the C-family hooks targetcm out of target.def into a separate c-family/c-target.def, with associated c-target.h and c-target-def.h. This is logically cleaner, in that declarations and definitions applicable only to C-family front ends (of targetcm and the associated initialize

[wwdocs] add libstdc++ docs for 4.6.0

2011-04-05 Thread Benjamin Kosnik
Hi Gerald! I've gone ahead and generated libstdc++ documentation for the 4.6.0 release. Please find enclosed a patch for the main gcc documentation page (http://gcc.gnu.org/onlinedocs) that adds these links. Of course, these links are predicated on your moving: ~bkoz/gcc-4.6.0/libstdc++ into

[PATCH] fix PR bootstrap/48471

2011-04-05 Thread Nathan Froyd
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 and two macros that use some of said variables below an #

Re: [wwwdocs] Change color scheme for links

2011-04-05 Thread Gerald Pfeifer
On Tue, 5 Apr 2011, Mike Stump wrote: > So, I don't know which patch did it, but that new orange color on the > title on the main page is awful. Please, pick any other color. So, you don't like our Dutch contributors? :-) Point taken, and addressed thusly. Note this also makes the color for v

[wwwdocs] Reduce the size of link texts on the main page

2011-04-05 Thread Gerald Pfeifer
No textual changes, just shorter links (since our main page really has quite many). Gerald Index: index.html === RCS file: /cvs/gcc/wwwdocs/htdocs/index.html,v retrieving revision 1.788 diff -u -r1.788 index.html --- index.html 6 Ap

[wwwdocs] obsolete gnu.css [4/4]

2011-04-05 Thread Gerald Pfeifer
And gone it is! Gerald Index: style.mhtml === RCS file: /cvs/gcc/wwwdocs/htdocs/style.mhtml,v retrieving revision 1.110 diff -u -r1.110 style.mhtml --- style.mhtml 4 Apr 2011 23:56:04 - 1.110 +++ style.mhtml 4 Apr 2011 23:5

[wwwdocs] Further condense our main page vertically

2011-04-05 Thread Gerald Pfeifer
Applied. (The gcc.css patch is with the -w option of diff.) Gerald Index: index.html === RCS file: /cvs/gcc/wwwdocs/htdocs/index.html,v retrieving revision 1.787 diff -u -w -r1.787 index.html --- index.html 5 Apr 2011 14:18:50 -000

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

2011-04-05 Thread Alexandre Oliva
On Apr 5, 2011, Jakub Jelinek wrote: > or DF would have resetted those debug_insns referencing it > (or replaced the pseudos in there with a debug temporary set before the last > real use), otherwise debug insns would change the liveness of the pseudo. Ah, right, that was the bit I was missing!

Re: [PATCH] Fix -m32 -g -O0 debug info (PR debug/48466)

2011-04-05 Thread Jakub Jelinek
On Wed, Apr 06, 2011 at 12:43:11AM +0200, Jakub Jelinek wrote: > On the pr48466.c testcase below on i?86 -m32 -g -O0 we generate wrong debug > info for some of the variables in main. This was caused by PR36977 fix, > crtl->stack_realign_tried is true, but fde->drap_reg is INVALID_REGNUM, > but fra

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

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

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

2011-04-05 Thread Eric Botcazou
> I believe this patch will work for any platform and I will check it in > once I can test it. I have been having bootstrap problems due to other > issues so I haven't been able to test it yet on all my platforms (some > of which will issue the note and others of which will not). > > 2011-04-05 S

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

2011-04-05 Thread Eric Botcazou
> 2011-04-05 Eric Botcazou > > * gcc.dg/torture/pr47917.c: Add -D_XOPEN_SOURCE=500 to dg-options for > Solaris 8 as well. I somehow missed that the same change is needed on Solaris 9... Adjusted. -- Eric Botcazou

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

2011-04-05 Thread Steve Ellcey
On Wed, 2011-04-06 at 00:17 +0200, Eric Botcazou wrote: > > As an increasing number of targets will start using the Enum facility, it > > would be good to be able to just to accept such a note on all targets > > without requiring it. But if that isn't possible, then this patch seems > > appropriat

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

2011-04-05 Thread Eric Botcazou
> 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 > with _XOPEN_SOURCE defined as 500. The following patch does > this. Likewise on Solaris 8 (It's deprecated, I know, but I have access to a quite fa

[PATCH] Fix -m32 -g -O0 debug info (PR debug/48466)

2011-04-05 Thread Jakub Jelinek
Hi! On the pr48466.c testcase below on i?86 -m32 -g -O0 we generate wrong debug info for some of the variables in main. This was caused by PR36977 fix, crtl->stack_realign_tried is true, but fde->drap_reg is INVALID_REGNUM, but frame_pointer_rtx is being eliminated to hard_frame_pointer_regnum ra

Re: Scheduler cleanups, 1/N

2011-04-05 Thread Alexandre Oliva
On Apr 5, 2011, Bernd Schmidt wrote: > On 04/05/2011 11:05 PM, Alexandre Oliva wrote: >> While debugging the -fcompare-debug regression that turned out to be >> independently reported as 48403, I got slightly annoyed that >> rank_for_schedule would walk a sequence of debug insns over and over, >

Re: [wwwdocs] Change color scheme for links

2011-04-05 Thread Mike Stump
On Apr 3, 2011, at 2:45 PM, Gerald Pfeifer wrote: > This removes the violet coloring for visited links that I had always > not quite enjoyed (but implemented to establish sync with www.gnu.org). So, I don't know which patch did it, but that new orange color on the title on the main page is awful.

Re: Scheduler cleanups, 5/5

2011-04-05 Thread Bernd Schmidt
On 04/06/2011 12:15 AM, Steve Ellcey wrote: > I finally got a good bootstrap on IA64 HP-UX. I used this patch, Jeff's > patch for PR 48444 (already checked in) and Nathan's patch for PR 48471. > So yes, I'd like to see this patch checked in too. Done. Bernd

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

2011-04-05 Thread Eric Botcazou
> As an increasing number of targets will start using the Enum facility, it > would be good to be able to just to accept such a note on all targets > without requiring it. But if that isn't possible, then this patch seems > appropriate (and similar changes to the test will be needed for other > ta

Re: Scheduler cleanups, 5/5

2011-04-05 Thread Steve Ellcey
On Tue, 2011-04-05 at 23:06 +0200, Eric Botcazou wrote: > Yes, possibly, and there is still the other issue that Jeff is working on. > > Given my results and that your patch from this thread apparently only > restores > the old behaviour, I'd install this patch. I finally got a good bootstrap

Re: [PATCH 3/6] Allow jumps in epilogues

2011-04-05 Thread Bernd Schmidt
On 03/31/2011 11:28 PM, Richard Henderson wrote: >> 003 - Store dw_cfi_refs in VECs rather than linked lists. Looks >> larger than it is due to reindentation > > Like 001, this one looks like it's totally independent of and of > the other changes, and a good cleanup. Please go ahead and te

Re: [PATCH, PR43920, 9/9] Cross-jumping - Allow both directions.

2011-04-05 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 04/01/11 08:56, Tom de Vries wrote: > Reposting, with ChangeLog. > PR target/43920 > * cfgcleanup.c (try_crossjump_to_edge): Add dir parameter. Pass dir to > flow_find_cross_jump. Swap variables to implement backward replacement.

Re: [PATCH, PR43920, 8/9] Cross-jumping - Extend search scope - test case.

2011-04-05 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 04/01/11 09:18, 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. >> >>> 2011-04-01 Tom de Vries >>> >>> PR target/43920 >>>

Re: [PATCH, PR43920, 7/9] Cross-jumping - Extend search scope.

2011-04-05 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 04/01/11 08:55, Tom de Vries wrote: > Reposting, with ChangeLog. > > PR target/43920 > * cfgcleanup.c (walk_to_nondebug_insn): New function. > (flow_find_cross_jump): Use walk_to_nondebug_insn. Recalculate bb1 and > bb2. >

Fix bootstrap comparison failures with checking disabled

2011-04-05 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Ugh. I feel like an idiot. bkoz mentioned in IRC that he was having bootstrapping failures several days ago. Unfortunately, we didn't exchange much info other than the target (x86-64) and since I wasn't having any problems I didn't follow up furth

libgo patch committed: Add external timeout to gotest script

2011-04-05 Thread Ian Lance Taylor
This patch to the gotest test script adds an external timeout, which will kill a test if the internal timeout fails due to some problem in the runtime. This patch is imperfect in that it will leave a sleep command around after the test is complete. The sleep command will eventually terminate and

Re: [PATCH] Fix LTO bootstrap on i686-linux (problem with two Ldebug_info0 labels; PR bootstrap/48148)

2011-04-05 Thread Jakub Jelinek
On Tue, Apr 05, 2011 at 02:06:14PM -0700, Nathan Froyd wrote: > On Tue, Apr 05, 2011 at 04:19:39PM +0200, Jakub Jelinek wrote: > > + for (i = 0; i < VEC_length (deferred_locations, > > deferred_locations_list); i++) > > +{ > > + add_location_or_const_value_attribute ( > > + VEC_index (

Re: Scheduler cleanups, 1/N

2011-04-05 Thread Bernd Schmidt
On 04/05/2011 11:05 PM, Alexandre Oliva wrote: > While debugging the -fcompare-debug regression that turned out to be > independently reported as 48403, I got slightly annoyed that > rank_for_schedule would walk a sequence of debug insns over and over, > once per compare. I figured it might be use

Re: Scheduler cleanups, 5/5

2011-04-05 Thread Eric Botcazou
> > After having done another round of testing, let me recap: > > 1. yesterday's pristine tree yields the bootstrap comparison failure on > > the IA-64/Linux machine, > > 2. yesterday's pristine tree + your 2 patches to haifa-sched.c > > successfully bootstraps on the IA-64/Linux machine. > > S

Re: [PATCH] Fix LTO bootstrap on i686-linux (problem with two Ldebug_info0 labels; PR bootstrap/48148)

2011-04-05 Thread Nathan Froyd
On Tue, Apr 05, 2011 at 04:19:39PM +0200, Jakub Jelinek wrote: > + for (i = 0; i < VEC_length (deferred_locations, deferred_locations_list); > i++) > +{ > + add_location_or_const_value_attribute ( > + VEC_index (deferred_locations, deferred_locations_list, i)->die, > + VEC_index

Re: Scheduler cleanups, 1/N

2011-04-05 Thread Alexandre Oliva
On Apr 4, 2011, Bernd Schmidt wrote: > There were a few places where last_scheduled_insn wasn't just examined > on its own, but code wanted to walk backwards and forwards from it. This > patch adapts them. I've also included Steven's patch from the bugzilla. While debugging the -fcompare-debug

Re: Scheduler cleanups, 5/5

2011-04-05 Thread Steve Ellcey
On Tue, 2011-04-05 at 22:31 +0200, Bernd Schmidt wrote: > On 04/05/2011 07:48 PM, Eric Botcazou wrote: > >> Looking into it. I ran into PR48441, I assume you were using the patch > >> from that as well? > > > > It's yesterday's tree + your 2 patches to haifa-sched.c, so without > > Steven's > >

Re: Scheduler cleanups, 5/5

2011-04-05 Thread Bernd Schmidt
On 04/05/2011 07:48 PM, Eric Botcazou wrote: >> Looking into it. I ran into PR48441, I assume you were using the patch >> from that as well? > > It's yesterday's tree + your 2 patches to haifa-sched.c, so without Steven's > patch... at least it was supposed to be, but I screwed up, sorry about th

Re: Scheduler cleanups, 5/5

2011-04-05 Thread Steve Ellcey
On Tue, 2011-04-05 at 20:18 +0200, Eric Botcazou wrote: > > What are the two patches to haifa-sched.c? I have one patch to > > schedule_block from > > http://gcc.gnu.org/ml/gcc-patches/2011-04/msg00271.html > > but that patch (alone or with the Jeff Law patch) isn't working for me > > on IA64 Linu

[pph] Fix release compilation failure

2011-04-05 Thread Lawrence Crowl
The current state of PPH uses cp_lexer_dump_tokens and cp_lexer_debug_tokens even in non-checking mode, so always generate them. We will probably reconsider at a future time. Index: gcc/cp/ChangeLog.pph 2011-04-05 Lawrence Crowl * parser.c (cp_lexer_dump_tokens): Remove #ifdef ENABLE_

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

2011-04-05 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 German team of translators. The file is available at: http://translationproject.org/latest/gcc/de.po (This file, 'gcc-4.6.0.de.po', has just

Re: [libgo, build] Avoid echo -n in gotest

2011-04-05 Thread Ian Lance Taylor
Andreas Schwab writes: > Rainer Orth writes: > >> diff --git a/libgo/testsuite/gotest b/libgo/testsuite/gotest >> --- a/libgo/testsuite/gotest >> +++ b/libgo/testsuite/gotest >> @@ -231,7 +231,7 @@ mkdir _test >> >> case "x$gofiles" in >> x) >> -gofiles=$(echo -n $(ls *_test.go 2>/dev/nu

Re: [PATCH] cleanup gcse.c:canon_modify_mem_list

2011-04-05 Thread Nathan Froyd
On Tue, Apr 05, 2011 at 07:18:16PM +, Joseph S. Myers wrote: > On Tue, 5 Apr 2011, Nathan Froyd wrote: > > Certainly vec.h never uses spaces; I thought this was simply The Way > > Things Were. > > I also had the impression that for certain special macros such as VEC, > GTY, _, N_, G_ - macros

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

2011-04-05 Thread Nathan Froyd
On Tue, Apr 05, 2011 at 12:29:45PM -0700, H.J. Lu wrote: > On Tue, Apr 5, 2011 at 11:28 AM, Nathan Froyd > wrote: > >> This broke compilation on AUTO_INC_DEC targets.  Currently putting > >> together a fix and testing via cross to powerpc-eabispe. > > > > ...and here's the patch I'm going to inst

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

2011-04-05 Thread H.J. Lu
On Tue, Apr 5, 2011 at 11:28 AM, Nathan Froyd wrote: > On Tue, Apr 05, 2011 at 11:22:56AM -0700, Nathan Froyd wrote: >> On Tue, Apr 05, 2011 at 09:59:43AM -0400, Nathan Froyd wrote: >> > On Mon, Apr 04, 2011 at 02:49:54PM -0400, Nathan Froyd wrote: >> > > This patch does just what $SUBJECT suggest

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

2011-04-05 Thread H.J. Lu
On Tue, Apr 5, 2011 at 11:28 AM, Nathan Froyd wrote: > On Tue, Apr 05, 2011 at 11:22:56AM -0700, Nathan Froyd wrote: >> On Tue, Apr 05, 2011 at 09:59:43AM -0400, Nathan Froyd wrote: >> > On Mon, Apr 04, 2011 at 02:49:54PM -0400, Nathan Froyd wrote: >> > > This patch does just what $SUBJECT suggest

Re: [PATCH] cleanup gcse.c:canon_modify_mem_list

2011-04-05 Thread Joseph S. Myers
On Tue, 5 Apr 2011, Nathan Froyd wrote: > On Tue, Apr 05, 2011 at 01:22:39PM +0100, Richard Earnshaw wrote: > > On Tue, 2011-04-05 at 04:44 -0700, Nathan Froyd wrote: > > > > nit; You're missing some whitespace here (after the VEC). > > > > > > This doesn't seem to be a hard-and-fast policy; all

Re: [REQUEST]Update config.sub from upstream

2011-04-05 Thread Ralf Wildenhues
[ from the binutils list ] * Ralf Wildenhues wrote on Tue, Apr 05, 2011 at 08:03:37AM CEST: > I will sync the copy in GCC tonight. FYI, I've synced config.sub from the config project (already done in src this morning). Cheers, Ralf > 2011-04-05 Ralf Wildenhues > > * config.sub: Sync f

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

2011-04-05 Thread Nathan Froyd
On Tue, Apr 05, 2011 at 11:22:56AM -0700, Nathan Froyd wrote: > On Tue, Apr 05, 2011 at 09:59:43AM -0400, Nathan Froyd wrote: > > On Mon, Apr 04, 2011 at 02:49:54PM -0400, Nathan Froyd wrote: > > > This patch does just what $SUBJECT suggests. > > > > v2, now with obstacks! > > This broke compilat

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

2011-04-05 Thread Nathan Froyd
On Tue, Apr 05, 2011 at 09:59:43AM -0400, Nathan Froyd wrote: > On Mon, Apr 04, 2011 at 02:49:54PM -0400, Nathan Froyd wrote: > > This patch does just what $SUBJECT suggests. > > v2, now with obstacks! This broke compilation on AUTO_INC_DEC targets. Currently putting together a fix and testing v

Re: [lto, testsuite] Don't use visibility on targets that don't support it (PR lto/47334)

2011-04-05 Thread Rainer Orth
Mike Stump writes: > On Apr 5, 2011, at 1:56 AM, Rainer Orth wrote: >> * lto.c (promote_var): Only set VISIBILITY_HIDDEN if >> HAVE_GAS_HIDDEN. > > Oh, at a minimum, if TARGET_ASM_ASSEMBLE_VISIBILITY is set, doing this stuff > I think is useful? No, this won't work. E.g. on Solaris w

Re: Scheduler cleanups, 5/5

2011-04-05 Thread Eric Botcazou
> What are the two patches to haifa-sched.c? I have one patch to > schedule_block from > http://gcc.gnu.org/ml/gcc-patches/2011-04/msg00271.html > but that patch (alone or with the Jeff Law patch) isn't working for me > on IA64 Linux. Is there a second haifa-sched.c patch I should also > have? N

Re: [lto, testsuite] Don't use visibility on targets that don't support it (PR lto/47334)

2011-04-05 Thread Rainer Orth
Mike Stump writes: > On Apr 5, 2011, at 1:56 AM, Rainer Orth wrote: >> * lto.c (promote_var): Only set VISIBILITY_HIDDEN if >> HAVE_GAS_HIDDEN. > > This looks wrong, there are more things that have visibility than those > things that use GAS and have .hidden. Darwin I think is one of them

Re: [PATCH] cleanup gcse.c:canon_modify_mem_list

2011-04-05 Thread Mike Stump
On Apr 5, 2011, at 5:55 AM, Richard Earnshaw wrote: > IMO, rototils are generally pointless. As a counter point, I like polish and style, in addition to beauty and flexibility. I'd rather see more style cleanups, more polish cleanups and more beautiful cleanups. I'd like to see more, not less

Re: [PATCH] use use_reg in several backends

2011-04-05 Thread Eric Botcazou
> * config/sparc/sparc.c (sparc_legitimize_tls_address): Likewise. Fine with me, thanks. -- Eric Botcazou

Re: Scheduler cleanups, 5/5

2011-04-05 Thread Steve Ellcey
On Tue, 2011-04-05 at 19:48 +0200, Eric Botcazou wrote: > > Looking into it. I ran into PR48441, I assume you were using the patch > > from that as well? > > It's yesterday's tree + your 2 patches to haifa-sched.c, so without Steven's > patch... at least it was supposed to be, but I screwed up, s

Re: [PATCH] don't use TYPE_ARG_TYPES in type_generic or sentinel attribute handling

2011-04-05 Thread Eric Botcazou
> gcc/ada/ > * gcc-interface/utils.c (handle_sentinel_attribute): Don't use > TYPE_ARG_TYPES > (handle_type_generic_attribute): Likewise. This code is very likely a duplicate of the one in c-common.c so Joseph's approval counts for it as well. -- Eric Botcazou

[PATCH] use use_reg in several backends

2011-04-05 Thread Nathan Froyd
Several places in the compiler could be usefully modified to use use_reg instead of hand-building EXPR_LISTs. This patch makes such changes. Since this patch implements equivalent functionality with an existing function call, I believe it counts as obvious. Tested by building cc1 for appropriate

Re: Scheduler cleanups, 5/5

2011-04-05 Thread Eric Botcazou
> Looking into it. I ran into PR48441, I assume you were using the patch > from that as well? It's yesterday's tree + your 2 patches to haifa-sched.c, so without Steven's patch... at least it was supposed to be, but I screwed up, sorry about that. After having done another round of testing, let

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

2011-04-05 Thread Jakub Jelinek
On Tue, Apr 05, 2011 at 02:24:55PM -0300, Alexandre Oliva wrote: > On Apr 4, 2011, Jakub Jelinek wrote: > > > Alternatively, perhaps all we care about is either i3, or NEXT_INSN of > > the last debug_insn propagate_for_debug changed in an interesting way. > > Ideally, we'd like to identify all

[Patch, Fortran] PR 18918 - Fix/Add multi-image support to UCOBOUND

2011-04-05 Thread Tobias Burnus
This patch adds multi-image support to UCOBOUND. In the -fcoarray=single case, the last dimension is just "LCOARRAY (coarray, dim=corank)". However, if there are multiple images, one has for corank-1 coarrays: "lcobound(coarray) + num_images() -1" and for multi-rank coarrays for the last dimens

Re: [lto, testsuite] Don't use visibility on targets that don't support it (PR lto/47334)

2011-04-05 Thread Mike Stump
On Apr 5, 2011, at 1:56 AM, Rainer Orth wrote: > * lto.c (promote_var): Only set VISIBILITY_HIDDEN if > HAVE_GAS_HIDDEN. Oh, at a minimum, if TARGET_ASM_ASSEMBLE_VISIBILITY is set, doing this stuff I think is useful?

Re: [lto, testsuite] Don't use visibility on targets that don't support it (PR lto/47334)

2011-04-05 Thread Mike Stump
On Apr 5, 2011, at 1:56 AM, Rainer Orth wrote: > * lto.c (promote_var): Only set VISIBILITY_HIDDEN if > HAVE_GAS_HIDDEN. This looks wrong, there are more things that have visibility than those things that use GAS and have .hidden. Darwin I think is one of them. ? cygming.h seems to be

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

2011-04-05 Thread Alexandre Oliva
On Apr 4, 2011, Jakub Jelinek wrote: > Alternatively, perhaps all we care about is either i3, or NEXT_INSN of > the last debug_insn propagate_for_debug changed in an interesting way. Ideally, we'd like to identify all debug users of i2dest (or whatever DEF we're removing) and propagate into the

Re: Scheduler cleanups, 5/5

2011-04-05 Thread Steve Ellcey
On Tue, 2011-04-05 at 09:41 -0700, Steve Ellcey wrote: > Hm, I just tried a bootstrap build on IA64 HP-UX using the haifa-sched.c > patch and r171988 and I got this error during stage 2: > > > /proj/opensrc_nobackup/sje/reg/src/trunk/gcc/genautomata.c: In function > 'create_ > automata': > /pro

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

2011-04-05 Thread Charles Wilson
On 4/5/2011 12:50 PM, Arnaud Charlet wrote: >> This: >> $(LN_S) $(fsrcpfx)ada/$(f) $(RTSDIR) >> fits that non-portable pattern > > That's wrong. The above is > > $(LN_S) /full/path/to/a_file a_dir > > which is quite different from > > $(LN_S) a_file a_dir > > and will work equally well wi

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

2011-04-05 Thread Arnaud Charlet
> According to 'info Autoconf --index LN_S' as quoted by Ralf, it is not > *portable* to do > $(LN_S) a_file a_dir > because it behaves differently depending on whether LN_S is "ln -s", > "ln" (or, as in this case, "cp -p"). That is certainly true. > This: > $(LN_S) $(fsrcpfx)ada/$(f)

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

2011-04-05 Thread Charles Wilson
On 4/5/2011 12:11 PM, Arnaud Charlet wrote: >> As a side issue, it ALSO seems to me that Ada's Mafile.in is doing it >> wrong, given Ralf's statements above: >> >> # Copy target independent sources >> $(foreach f,$(ADA_INCLUDE_SRCS) $(LIBGNAT_SRCS), \ >> $(LN_S) $(fsrcpfx)ada

Re: Scheduler cleanups, 5/5

2011-04-05 Thread Steve Ellcey
On Tue, 2011-04-05 at 17:54 +0200, Bernd Schmidt wrote: > On 04/05/2011 04:32 PM, Eric Botcazou wrote: > >> The patch below fixes the testcase in the PR. I'll test > >> tonight/tomorrow, probably on mips64-elf. Ok if that passes? > > > > I get back the comparison failure with it on IA-64/Linux: >

Re: [PATCH] make gcse.c:modify_mem_list hold VECs instead of INSN_LISTs

2011-04-05 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 04/05/11 07:20, Nathan Froyd wrote: > As promised, this patch turns modify_mem_list into an array of VECs, > similar to the changes done to canon_modify_mem_list. Since I'm in the > area, I took the liberty of tweaking the VEC declarations related

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

2011-04-05 Thread Nathan Froyd
On Tue, Apr 05, 2011 at 05:55:33PM +0200, Michael Matz wrote: > I have a preference in having just one DECL_RTL field for conceptual > reasons: > > Most DECLs are actually objects (there are some prominent exceptions, but > those always would be better described with something like NAMED_ENTITY,

Re: [PATCH] cleanup gcse.c:canon_modify_mem_list

2011-04-05 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 04/05/11 05:44, Nathan Froyd wrote: > On Mon, Apr 04, 2011 at 12:01:09PM -0600, Jeff Law wrote: >>> + canon_modify_mem_list = GCNEWVEC (VEC(modify_pair,heap) *, >>> + last_basic_block); >> nit; You're missing some whit

Re: Scheduler cleanups, 5/5

2011-04-05 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 04/05/11 10:10, Bernd Schmidt wrote: > On 04/05/2011 06:08 PM, Jeff Law wrote: >> On 04/05/11 09:54, Bernd Schmidt wrote: >>> On 04/05/2011 04:32 PM, Eric Botcazou wrote: > The patch below fixes the testcase in the PR. I'll test > tonight/to

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

2011-04-05 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 04/05/11 08:51, Nathan Froyd wrote: > On Tue, Apr 05, 2011 at 04:42:27PM +0200, Steven Bosscher wrote: >> On Tue, Apr 5, 2011 at 3:59 PM, Nathan Froyd >> wrote: >>> v2, now with obstacks! >> >> @findex LOG_LINKS >> @item LOG_LINKS (@var{i}) >> A l

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

2011-04-05 Thread Arnaud Charlet
> As a side issue, it ALSO seems to me that Ada's Mafile.in is doing it > wrong, given Ralf's statements above: > > # Copy target independent sources > $(foreach f,$(ADA_INCLUDE_SRCS) $(LIBGNAT_SRCS), \ > $(LN_S) $(fsrcpfx)ada/$(f) $(RTSDIR) ;) true > > But as this isn't s

Re: Scheduler cleanups, 5/5

2011-04-05 Thread Bernd Schmidt
On 04/05/2011 06:08 PM, Jeff Law wrote: > On 04/05/11 09:54, Bernd Schmidt wrote: >> On 04/05/2011 04:32 PM, Eric Botcazou wrote: The patch below fixes the testcase in the PR. I'll test tonight/tomorrow, probably on mips64-elf. Ok if that passes? >>> >>> I get back the comparison failure

Re: Scheduler cleanups, 5/5

2011-04-05 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 04/05/11 09:54, Bernd Schmidt wrote: > On 04/05/2011 04:32 PM, Eric Botcazou wrote: >>> The patch below fixes the testcase in the PR. I'll test >>> tonight/tomorrow, probably on mips64-elf. Ok if that passes? >> >> I get back the comparison failure

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

2011-04-05 Thread Charles Wilson
On 4/5/2011 10:21 AM, Ralf Wildenhues wrote: > * Charles Wilson wrote on Tue, Apr 05, 2011 at 04:15:02PM CEST: >> On 4/5/2011 9:21 AM, Arnaud Charlet wrote: I'm one of the MSys "devs" (if you want to call the sporadic process of updating MSys "development"). I'll take a look at implement

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

2011-04-05 Thread Michael Matz
Hi, On Mon, 4 Apr 2011, Nathan Froyd wrote: > 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 SWITC

Re: Scheduler cleanups, 5/5

2011-04-05 Thread Bernd Schmidt
On 04/05/2011 04:32 PM, Eric Botcazou wrote: >> The patch below fixes the testcase in the PR. I'll test >> tonight/tomorrow, probably on mips64-elf. Ok if that passes? > > I get back the comparison failure with it on IA-64/Linux: Looking into it. I ran into PR48441, I assume you were using the pa

Re: [PATCH, PR48461] define emit_use_return_register_into_block only if HAVE_return.

2011-04-05 Thread Richard Guenther
On Tue, Apr 5, 2011 at 5:50 PM, Tom de Vries wrote: > Hi, > > I introduced the function emit_use_return_register_into_block for > PR43920, but should have guarded it with HAVE_return. > The patch fixes this. > > Tested by doing x86_64 non-bootstrap c/fortran build. > > OK for trunk? Ok. Thanks,

[PATCH, PR48461] define emit_use_return_register_into_block only if HAVE_return.

2011-04-05 Thread Tom de Vries
Hi, I introduced the function emit_use_return_register_into_block for PR43920, but should have guarded it with HAVE_return. The patch fixes this. Tested by doing x86_64 non-bootstrap c/fortran build. OK for trunk? Thanks, - Tom 2011-04-05 Tom de Vries PR middle-end/48461 * function.c (emi

Re: [PATCH, ARM] Avoid element-order-dependent operations for quad-word vectors in big-endian mode for NEON

2011-04-05 Thread Julian Brown
On Wed, 9 Feb 2011 12:11:35 + Julian Brown wrote: > On Wed, 12 Jan 2011 17:38:22 + > Julian Brown wrote: > > > This version of the patch tweaks target-supports.exp to say that > > various operations are not available in big-endian mode (removing > > some of the FAILs from the previous v

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

2011-04-05 Thread Steven Bosscher
On Tue, Apr 5, 2011 at 4:51 PM, Nathan Froyd wrote: > On Tue, Apr 05, 2011 at 04:42:27PM +0200, Steven Bosscher wrote: >> On Tue, Apr 5, 2011 at 3:59 PM, Nathan Froyd >> wrote: >> > v2, now with obstacks! >> >> @findex LOG_LINKS >> @item LOG_LINKS (@var{i}) >> A list (chain of @code{insn_list} e

[wwwdocs] Fix two blunders of mine

2011-04-05 Thread Gerald Pfeifer
That'll teach me sneaking in some wwwdocs work around lunch break. Fixed thusly. Gerald Index: index.html === RCS file: /cvs/gcc/wwwdocs/htdocs/index.html,v retrieving revision 1.785 retrieving revision 1.787 diff -u -r1.785 -r1.787

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

2011-04-05 Thread Eric Botcazou
> Alternatively, perhaps all we care about is either i3, or NEXT_INSN of the > last debug_insn propagate_for_debug changed in an interesting way. Thus > propagate_for_debug could return the last DEBUG_INSN it changed, and caller > decide either that no updating of last_modified_debug_insn is neede

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

2011-04-05 Thread Nathan Froyd
On Tue, Apr 05, 2011 at 04:42:27PM +0200, Steven Bosscher wrote: > On Tue, Apr 5, 2011 at 3:59 PM, Nathan Froyd wrote: > > v2, now with obstacks! > > @findex LOG_LINKS > @item LOG_LINKS (@var{i}) > A list (chain of @code{insn_list} expressions) giving information about > dependencies between inst

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

2011-04-05 Thread Steven Bosscher
On Tue, Apr 5, 2011 at 3:59 PM, Nathan Froyd wrote: > On Mon, Apr 04, 2011 at 02:49:54PM -0400, Nathan Froyd wrote: >> This patch does just what $SUBJECT suggests. > > v2, now with obstacks! Still one TODO in doc/rtl.texi: @findex LOG_LINKS @item LOG_LINKS (@var{i}) A list (chain of @code{insn_l

Re: [PATCH] cleanup gcse.c:canon_modify_mem_list

2011-04-05 Thread Richard Earnshaw
On Tue, 2011-04-05 at 07:07 -0600, Jeff Law wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > On 04/05/11 06:55, Richard Earnshaw wrote: > > > > On Tue, 2011-04-05 at 05:30 -0700, Nathan Froyd wrote: > >> On Tue, Apr 05, 2011 at 01:22:39PM +0100, Richard Earnshaw wrote: > >>> On Tue,

Re: [libgo, build] Avoid echo -n in gotest

2011-04-05 Thread Andreas Schwab
Rainer Orth writes: > diff --git a/libgo/testsuite/gotest b/libgo/testsuite/gotest > --- a/libgo/testsuite/gotest > +++ b/libgo/testsuite/gotest > @@ -231,7 +231,7 @@ mkdir _test > > case "x$gofiles" in > x) > - gofiles=$(echo -n $(ls *_test.go 2>/dev/null)) > + gofiles=$(printf "%s"

Fix accidental reversion

2011-04-05 Thread Eric Botcazou
In June of last year I installed the change: 2010-06-06 Eric Botcazou * stor-layout.c (self_referential_size): Set UNKNOWN_LOCATION on the newly built CALL_EXPR. but it was accidentally reverted by: 2010-07-13 Nathan Froyd [...] (expand_parallel_call): Call build_

Re: Scheduler cleanups, 5/5

2011-04-05 Thread Eric Botcazou
> The patch below fixes the testcase in the PR. I'll test > tonight/tomorrow, probably on mips64-elf. Ok if that passes? I get back the comparison failure with it on IA-64/Linux: Comparing stages 2 and 3 warning: gcc/cc1-checksum.o differs warning: gcc/cc1plus-checksum.o differs Bootstrap compari

[libgo, build] Avoid echo -n in gotest

2011-04-05 Thread Rainer Orth
Inspecting the Solaris 8 libgo.sum file, I was reminded that echo -n isn't portable. The autoconf manual agrees and suggests printf as a replacement. The following patch implements this. Bootstrapped on i386-pc-solaris2.8 with gas. Rainer 2011-04-05 Rainer Orth * testsuite

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

2011-04-05 Thread Ralf Wildenhues
* Charles Wilson wrote on Tue, Apr 05, 2011 at 04:15:02PM CEST: > On 4/5/2011 9:21 AM, Arnaud Charlet wrote: > >> I'm one of the MSys "devs" (if you want to call the sporadic process of > >> updating MSys "development"). I'll take a look at implementing > >>ln -s file dir > >> as synonymous wi

Re: [PATCH] use build_function_type_list for float128 functions

2011-04-05 Thread Tobias Burnus
On 04/05/2011 03:12 PM, Nathan Froyd wrote: When the real16/float128 bits were added to the Fortran FE, some uses of build_function_type snuck in. This patch changes them to use build_function_type_list. Thanks! The patch also corrects a small bug: the types for frexp and scalbn had their ar

[PATCH] Fix LTO bootstrap on i686-linux (problem with two Ldebug_info0 labels; PR bootstrap/48148)

2011-04-05 Thread Jakub Jelinek
Hi! i686-linux LTO bootstrap currently fails, because in one partition we emit .Ldebug_info0 label twice. The problem is that resolve_addr for call_site support attempts to force_decl_die external function decls, and at least with LTO that in turn can attempt to create new type DIEs, in this case

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

2011-04-05 Thread Charles Wilson
On 4/5/2011 9:21 AM, Arnaud Charlet wrote: >> I'm one of the MSys "devs" (if you want to call the sporadic process of >> updating MSys "development"). I'll take a look at implementing >> ln -s file dir >> as synonymous with >> ln -s file dir/basename-of-file >> in the next few days. > >

  1   2   >