Re: PATCH: PR other/48007: Unwind library doesn't work with UNITS_PER_WORD > sizeof (void *)

2011-03-21 Thread Ian Lance Taylor
"H.J. Lu" writes: > Here is the updated patch. It has This patch is OK if it bootstraps and passes tests. Thanks. Ian

Re: PATCH: PR other/48007: Unwind library doesn't work with UNITS_PER_WORD > sizeof (void *)

2011-03-21 Thread H.J. Lu
On Mon, Mar 21, 2011 at 2:55 PM, Ian Lance Taylor wrote: > On Sun, Mar 6, 2011 at 9:18 AM, H.J. Lu wrote: > >> We shouldn't save call frame hard registers as "void *".  This patch >> changes the unwind library to save call frame hard registers as >> _Unwind_Word.  OK for 4.7? > >> +       PR othe

[PATCH 2/2] refactor emit_*_{after,before}{,_setloc} using common functions

2011-03-21 Thread Nathan Froyd
This patch builds on the previous one to refactor the close cousins of the *_noloc family. I attempted to separate these out into separate patches, one dealing with *_setloc and the other dealing with the remainder, but I did not trust myself to do it correctly even with magit's help. * e

[PATCH 1/2] refactor emit_*_{after,before}_noloc using common functions

2011-03-21 Thread Nathan Froyd
This patch deals with the *_noloc family of functions. emit_pattern seemed like a good prefix, even though AIUI these functions can receive toplevel rtxes. * emit-rtl.c (emit_pattern_before_noloc): New function. (emit_insn_before_noloc, emit_jump_insn_before_noloc): Call it.

[PATCH 0/2] reduce cut-and-paste programming in emit-rtl.c

2011-03-21 Thread Nathan Froyd
The functions emit_*_{before,after}* for emitting insns into the insn stream are full of duplication. This patch series attempts to rationalize the code by providing common building blocks for the {,call_,jump_,debug_}insn variants that build the necessary rtxes through the magic of function point

Re: [patch, libgfortran] Bug 48030 - Implement read_x using fbuf_getc

2011-03-21 Thread Jerry DeLisle
Ping On 03/19/2011 09:46 PM, Jerry DeLisle wrote: Hi, The attached patch re-implements read_x using fbuf_getc. Self explanatory. Regression tested on x86-64 linux. No new test case needed. Ok for trunk? Regards, Jerry 2011-03-19 Jerry DeLisle PR libgfortran/48030 * io/read.c (read_x): Re

Re: invoke.texi: Fix minor typos

2011-03-21 Thread Gerald Pfeifer
On Mon, 21 Mar 2011, Nick Clifton wrote: > 2011-03-21 Nick Clifton > > * doc/invoke.texi (Overall Options): Move closing brace to end of > options list. > (Optimization Options): Add missing @gol. > (Directory Options): Likewise. > (i386 and x86-64 Options): Likewi

[PATCH, 4.7] Fix failure of two tests on 32-bit powerpc

2011-03-21 Thread Michael Meissner
I noticed that dg.gcc/torture/vector-1.c and dg.gcc/torture/vector-2.c were failing on 32-bit PowerPC Linux systems. The failure was due to the compiler complaining that by default the ABI on 32-bit does not allow passing vector items. I fixed the tests so that they pass -mabi=altivec on powerpc

[PATCH, 4.7] Fix PR 48226, Allow Iterator::vector vector on powerpc with VSX

2011-03-21 Thread Michael Meissner
This patch fixes a problem discovered by compiling the boost libraries with VSX enabled. In the middle of a template is the code: namespace boost { // ... template struct apply { // ... typedef typename Iterator::vec

Re: [PATCH][RFC] Add gimple_fold

2011-03-21 Thread Diego Novillo
On 03/18/2011 10:11 AM, Richard Guenther wrote: This tries to extend the previously posted CCP folding patch by introducing a generic interface for non-tree-building, GIMPLE SSA aware folding. The low-level interface for folding regular operations is /* Fold the expression composed by *CODEP,

Re: Patch to align local PowerPC AltiVec arrays

2011-03-21 Thread Wellander, Pat - Code Sourcery
On 3/21/2011 3:26 PM, Andrew Pinski wrote: On Mon, Mar 21, 2011 at 3:25 PM, Andrew Pinski wrote: On Mon, Mar 21, 2011 at 3:21 PM, Wellander, Pat - Code Sourcery wrote: The stack is aligned at 16 bytes with AltiVec (if not, this patch would not make sense). Also, all char arrays are aligned a

Re: Patch to align local PowerPC AltiVec arrays

2011-03-21 Thread Andrew Pinski
On Mon, Mar 21, 2011 at 3:25 PM, Andrew Pinski wrote: > On Mon, Mar 21, 2011 at 3:21 PM, Wellander, Pat - Code Sourcery > wrote: >> The stack is aligned at 16 bytes with AltiVec (if not, this patch would not >> make sense). >> Also, all char arrays are aligned at 16 bytes. >> This patch causes ot

Re: Patch to align local PowerPC AltiVec arrays

2011-03-21 Thread Andrew Pinski
On Mon, Mar 21, 2011 at 3:21 PM, Wellander, Pat - Code Sourcery wrote: > The stack is aligned at 16 bytes with AltiVec (if not, this patch would not > make sense). > Also, all char arrays are aligned at 16 bytes. > This patch causes other local stack arrays >= 16 bytes to be aligned at 16 > bytes

Re: Patch to align local PowerPC AltiVec arrays

2011-03-21 Thread Wellander, Pat - Code Sourcery
The stack is aligned at 16 bytes with AltiVec (if not, this patch would not make sense). Also, all char arrays are aligned at 16 bytes. This patch causes other local stack arrays >= 16 bytes to be aligned at 16 bytes in order to make more effective use of AltiVec instructions. On 3/21/2011 3:0

Re: [libgo] Support GOTESTFLAGS

2011-03-21 Thread Ian Lance Taylor
Rainer Orth writes: > 2011-03-20 Rainer Orth > > * Makefile.am (CHECK): Use $(GOTESTFLAGS). > * Makefile.in: Regenerate. Thanks. Committed. Ian

Re: Patch to align local PowerPC AltiVec arrays

2011-03-21 Thread Andrew Pinski
On Mon, Mar 21, 2011 at 2:54 PM, Wellander, Pat - Code Sourcery wrote: > This is my first patch submittal so constructive criticism is appreciated. > This patch causes PowerPC AltiVec local stack arrays > 16 bytes to be > aligned at 16 bytes. Wait I thought the alignment of the stack is 16 bytes

Re: PATCH: PR other/48007: Unwind library doesn't work with UNITS_PER_WORD > sizeof (void *)

2011-03-21 Thread Ian Lance Taylor
On Sun, Mar 6, 2011 at 9:18 AM, H.J. Lu wrote: > We shouldn't save call frame hard registers as "void *".  This patch > changes the unwind library to save call frame hard registers as > _Unwind_Word.  OK for 4.7? > +       PR other/48007 > +       * unwind-dw2.c (_Unwind_Context): Save call fram

Patch to align local PowerPC AltiVec arrays

2011-03-21 Thread Wellander, Pat - Code Sourcery
This is my first patch submittal so constructive criticism is appreciated. This patch causes PowerPC AltiVec local stack arrays > 16 bytes to be aligned at 16 bytes. 2011-01-27 Pat Wellander * config/rs6000/rs6000.h (LOCAL_ALIGNMENT): AltiVec local arrays >= 16 bytes are now aligned to

Re: Cleaning up expand optabs code

2011-03-21 Thread Richard Sandiford
Richard Henderson writes: >> * reload.c (find_reloads_address_1): Use insn_operand_matches. >> * reload1.c (gen_reload): Likewise. > > All the bits that just use insn_operand_matches are approved. > You can commit those first if you like to reduce the patch size. OK, thanks, here's what

Re: Prune empty cases from toplevel configure.ac

2011-03-21 Thread Ralf Wildenhues
* Joseph S. Myers wrote on Mon, Mar 21, 2011 at 11:41:23AM CET: > This patch, relative to a tree with my deprecated targets removal patch > applied, further > cleans up the toplevel configure.ac by removing empty cases from a case > state

[avr] fixup dwarf2 unwind info

2011-03-21 Thread Richard Henderson
This is a slightly tidied patch from sometime last month, which was buried in a thread about dwarf2 unwind info for gas. This fixes some totally incorrect unwind markings in the avr backend, and adds support in the dwarf2 middle-end for handling avr's unique post_dec (rather than pre_dec) push in

[PATCH, alpha]: Committed: use define_c_enum for the unspec/unspecv constants

2011-03-21 Thread Uros Bizjak
Hello! 2011-03-21 Uros Bizjak * config/alpha/alpha.md (unspec): New define_c_enum. (unspecv): Ditto. Tested on alphaev68-pc-linux-gnu, committed to mainline SVN. Uros. Index: alpha.md === --- alpha.md(revisio

Re: [Ada] Implement switches to reorder record components in gigi

2011-03-21 Thread Duncan Sands
Hi Eric, Not clear what kind of answer you're expecting. To cancel the previous define? I somehow failed to see the define. Sorry for the noise. Ciao, Duncan.

Re: [PATCH] Fix PR 48143

2011-03-21 Thread Andrey Belevantsev
On 21.03.2011 18:12, Uros Bizjak wrote: 2011/3/21 Andrey Belevantsev: Hello, As noted in the PR audit trail, the scheduler crashes for this bug because we see an SSE insn without proper reservation. Uros has pointed several such insns to me. The following patch adding athlon_decode attributes

Re: [PATCH, alpha]: Fix PR bootstrap/48214: ICE in dwarf2out_var_location

2011-03-21 Thread Richard Henderson
On 03/21/2011 11:40 AM, Uros Bizjak wrote: > Hello! > > Attached patch fixes recent bootstrap failure on alpha-pc-linux-gnu in > in dwarf2out_var_location, at dwarf2out.c:21983. > > 2011-03-21 Uros Bizjak > > * config/alpha/alpha.c (alpha_pad_noreturn): Do not emit an unop > betwe

Re: Cleaning up expand optabs code

2011-03-21 Thread Richard Henderson
On 03/19/2011 12:52 PM, Richard Sandiford wrote: > Given the mode stuff above, I've tried to be quite draconian as far > as caller-provided modes go. I think the caller really should know > what mode they're dealing with. The one case where I had to hold > back a bit was create_convert_operand_fr

Re: [PATCH] prune warn_compact_unwind warnings on darwin

2011-03-21 Thread Jakub Jelinek
On Mon, Mar 21, 2011 at 12:18:43PM -0700, Mike Stump wrote: > I didn't put them into the release branch yet... RM? You have a > preference on timing? Now, after .0 goes out? After .0 goes out. RC2 is building already and the fewer changes after RC2, the better. Jakub

Re: [PATCH] prune warn_compact_unwind warnings on darwin

2011-03-21 Thread Mike Stump
On Mar 12, 2011, at 1:01 PM, Jack Howarth wrote: > 2011-03-12 Jack Howarth > > libstdc++-v3/ > * testsuite/lib/prune.exp: Prune "could not create compact unwind for" > warnings. > > gcc/ > * testsuite/lib/prune.exp: Ditto. Checked in r171263. I modified the Changelogs slightly,

Re: [PATCH] libstdc++ ABI issues with __int128

2011-03-21 Thread Jakub Jelinek
On Mon, Mar 21, 2011 at 07:32:14PM +0100, Benjamin Kosnik wrote: > > Well, such stricter checking wouldn't be completely painless, e.g. currently > > both current baseline_symbols.txt and the ones I've posted don't include the > > two TLS: lines, because of the fear that not everyone has TLS enable

Re: [testsuite, build] Convert boehm-gc testsuite to DejaGnu (PR boehm-gc/11412)

2011-03-21 Thread Mike Stump
On Mar 21, 2011, at 11:33 AM, Paolo Bonzini wrote: >> So, this still seems unportable to me? > > These are libtool options. Ah, that's the part I was missing... Thanks.

[PATCH, alpha]: Fix PR bootstrap/48214: ICE in dwarf2out_var_location

2011-03-21 Thread Uros Bizjak
Hello! Attached patch fixes recent bootstrap failure on alpha-pc-linux-gnu in in dwarf2out_var_location, at dwarf2out.c:21983. 2011-03-21 Uros Bizjak * config/alpha/alpha.c (alpha_pad_noreturn): Do not emit an unop between a call and its CALL_ARG_LOCATION note. Bootstrapped a

PING: PATCH: PR target/46770: Use .init_array/.fini_array sections

2011-03-21 Thread H.J. Lu
On Mon, Mar 14, 2011 at 12:28 PM, H.J. Lu wrote: > On Thu, Jan 27, 2011 at 2:40 AM, Richard Guenther > wrote: >> On Thu, Jan 27, 2011 at 12:12 AM, H.J. Lu wrote: >>> On Tue, Dec 14, 2010 at 05:20:48PM -0800, H.J. Lu wrote: This patch uses .init_array/.fini_array sections instead of .ct

PING: PATCH: PR other/48007: Unwind library doesn't work with UNITS_PER_WORD > sizeof (void *)

2011-03-21 Thread H.J. Lu
On Mon, Mar 14, 2011 at 10:49 AM, H.J. Lu wrote: >>> >>> We shouldn't save call frame hard registers as "void *".  This patch >>> changes the unwind library to save call frame hard registers as >>> _Unwind_Word.  OK for 4.7? >> >> I think this will break the ABI for the MIP

Re: [testsuite, build] Convert boehm-gc testsuite to DejaGnu (PR boehm-gc/11412)

2011-03-21 Thread Paolo Bonzini
On Mon, Mar 21, 2011 at 19:27, Mike Stump wrote: > On Mar 21, 2011, at 5:59 AM, Rainer Orth wrote: >> I've applied those changes, and also managed to fix the problems I had >> in Tru64 UNIX testing.  Here's the patch I'd like to commit to 4.7. > >> +# Build shared library from LIB. >> +proc boehm-

Re: [testsuite, build] Convert boehm-gc testsuite to DejaGnu (PR boehm-gc/11412)

2011-03-21 Thread Rainer Orth
Mike Stump writes: > On Mar 21, 2011, at 5:59 AM, Rainer Orth wrote: >> I've applied those changes, and also managed to fix the problems I had >> in Tru64 UNIX testing. Here's the patch I'd like to commit to 4.7. > >> +# Build shared library from LIB. >> +proc boehm-gc-build-shlib { lib flags ex

Re: [PATCH] libstdc++ ABI issues with __int128

2011-03-21 Thread Benjamin Kosnik
> Well, such stricter checking wouldn't be completely painless, e.g. currently > both current baseline_symbols.txt and the ones I've posted don't include the > two TLS: lines, because of the fear that not everyone has TLS enabled. > If baseline_symbols.txt right now miss some symbols, it is not a b

Re: [testsuite, build] Convert boehm-gc testsuite to DejaGnu (PR boehm-gc/11412)

2011-03-21 Thread Mike Stump
On Mar 21, 2011, at 5:59 AM, Rainer Orth wrote: > I've applied those changes, and also managed to fix the problems I had > in Tru64 UNIX testing. Here's the patch I'd like to commit to 4.7. > +# Build shared library from LIB. > +proc boehm-gc-build-shlib { lib flags extra-flags } { > +# FIXM

Re: [Ada] Avoid duplicating branches in loop conditions

2011-03-21 Thread Eric Botcazou
> 2011-03-21 Eric Botcazou > > * gcc-interface/decl.c (elaborate_expression_1): When optimization is > disabled, use the variable for bounds of loop iteration scheme. This quite mysteriously triggers a bogus -Wuninitialized warning on IRIX. Fixed thusly, tested on i586-suse-linux,

Re: [PATCH] Fix shorten_compare (PR c/48197)

2011-03-21 Thread Jakub Jelinek
On Mon, Mar 21, 2011 at 04:31:31PM +, Joseph S. Myers wrote: > On Sun, 20 Mar 2011, Jakub Jelinek wrote: > > > 2011-03-20 Jakub Jelinek > > > > PR c/48197 > > * c-common.c (shorten_compare): If primopN is first sign-extended > > to opN and then zero-extended to result type, set

Re: [patch] Couple of tweaks to the gimplifier

2011-03-21 Thread Eric Botcazou
> >  1) Set TREE_THIS_NOTRAP on the INDIRECT_REF built for VLA decls.  This > > is correct since stack memory isn't considered as trapping in the IL. > > This is ok. Thanks. > >  2) Improve gimplification of complex conditions in COND_EXPR.  They are > >     naturally generated by the Ada compile

Re: [Ada] Implement switches to reorder record components in gigi

2011-03-21 Thread Eric Botcazou
> Hi Eric, this looks like a nice improvement. I noticed this mysterious > undef though - what is it for? > > +#undef MOVE_FROM_FIELD_LIST_TO Not clear what kind of answer you're expecting. To cancel the previous define? -- Eric Botcazou

Re: Fix a few use-after-free issues

2011-03-21 Thread Jakub Jelinek
On Mon, Mar 21, 2011 at 11:37:16AM -0600, Jeff Law wrote: > Similarly for redirect_edge_succ_nodup in this fragment: > > ret = redirect_edge_succ_nodup (e, dest); > if (dump_file) > fprintf (dump_file, "Fallthru edge %i->%i redirected to %i\n", >e->src->index,

Re: [PATCH 02/18] enforce TREE_CHAIN and TREE_TYPE accesses

2011-03-21 Thread Eric Botcazou
> You're right, though; they should probably be marked as such for > completeness. OK to commit with those additions and reordering as above > sight unseen, or would you like to see the patch? Note that I didn't ask for that, only for a comment explaining why they were set aside. So it's up to

Re: Fix a few use-after-free issues

2011-03-21 Thread Diego Novillo
On Mon, Mar 21, 2011 at 13:37, Jeff Law wrote: > Bootstrapped and regression tested on x86_64-unknown-linux-gnu.  Ok for > the trunk? OK. Diego.

[patch middle-end c c++]: Optimize cost of comp_type_attributes

2011-03-21 Thread Kai Tietz
Hello, this patch does some search-cost optimization for comp_type_attributes target hook. By recent patch about diagnostic of attributes affecting type indentity, it is possible to avoid some searching for such kind of type checks. In general it would be possible to remove this target-hook compl

Fix a few use-after-free issues

2011-03-21 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 This fixes a couple use-after-free problems and one use-after-free non-problem. In cfgrtl.c, redirect_branch_edge may delete its first argument, so this code is clearly erroneous: e->flags &= ~EDGE_FALLTHRU; redirected = redirect

Re: [PATCH][ARM] Fix RVCT interoperation issue

2011-03-21 Thread Andrew Stubbs
On 14/03/11 21:21, Ramana Radhakrishnan wrote: This is OK for trunk. Committed, thanks. Andrew

Re: [PATCH] libstdc++ ABI issues with __int128

2011-03-21 Thread Jakub Jelinek
On Mon, Mar 21, 2011 at 05:52:52PM +0100, Benjamin Kosnik wrote: > > The problem is that abicheck only flags as incompatible additions to > > GLIBCXX_3.4, but not other base versions.  Ideally it would flag addition > > to any version that hasn't been added (addition of the version is flagged > > b

Re: [build, doc] Obsolete Solaris 8 support

2011-03-21 Thread Rainer Orth
"Joseph S. Myers" writes: >> > When obsoleting targets or other features, please update the release notes >> > (gcc-4.7/changes.html). >> >> Like this? Or should I link to the UI threads removal patch, too? > > I don't think such a link is needed. Ok, I've checked in the patch. Thanks.

Re: [PATCH] libstdc++ ABI issues with __int128

2011-03-21 Thread Benjamin Kosnik
> Most of the linux targets are actually __GXX_TYPEINFO_EQUALITY_INLINE 1 > and thus perhaps if we want to maintain some ABI compatibility with 3.4-ish > inlined typeid comparisons, we might need to export _ZTS* for _ZTI* > that could somehow be emitted for user code too. Right, for C++98 types.

[cxx-mem-model] test invalid bitfield twiddling

2011-03-21 Thread Aldy Hernandez
This test is inspired by PR 48124. [Jakub, let me know if you find any additional variations on this theme.] In the C++ memory model, non contiguous bitfields are distinct memory locations and cannot be accessed by and/or magic if we introduce data races. In the test below, we can trigger inv

Re: [build, doc] Obsolete Solaris 8 support

2011-03-21 Thread Joseph S. Myers
On Mon, 21 Mar 2011, Rainer Orth wrote: > "Joseph S. Myers" writes: > > > On Mon, 21 Mar 2011, Rainer Orth wrote: > > > >> Therefore I'll soon commit this patch to obsolete Solaris 8 support in > >> 4.7. It will be removed in 4.8 then. > > > > When obsoleting targets or other features, please u

Re: [PATCH] libstdc++ ABI issues with __int128

2011-03-21 Thread Benjamin Kosnik
> The problem is that abicheck only flags as incompatible additions to > GLIBCXX_3.4, but not other base versions.  Ideally it would flag addition > to any version that hasn't been added (addition of the version is flagged > by > OBJECT:0:CXXABI_1.3.5 > etc. entries (no @s in it). > This patch for

Re: [build, doc] Obsolete Solaris 8 support

2011-03-21 Thread Rainer Orth
"Joseph S. Myers" writes: > On Mon, 21 Mar 2011, Rainer Orth wrote: > >> Therefore I'll soon commit this patch to obsolete Solaris 8 support in >> 4.7. It will be removed in 4.8 then. > > When obsoleting targets or other features, please update the release notes > (gcc-4.7/changes.html). Like

Re: [PATCH] Fix shorten_compare (PR c/48197)

2011-03-21 Thread Joseph S. Myers
On Sun, 20 Mar 2011, Jakub Jelinek wrote: > 2011-03-20 Jakub Jelinek > > PR c/48197 > * c-common.c (shorten_compare): If primopN is first sign-extended > to opN and then zero-extended to result type, set primopN to opN. > > * gcc.c-torture/execute/pr48197.c: New test.

[PATCH] More ternary RHS, less single

2011-03-21 Thread Richard Guenther
This removes odd things from being a valid RHS and transitions the two easy cases from single to ternary RHS. There are now two codes left that should IMHO move over to ternary, COND_EXPR and VEC_COND_EXPR. I'll tackle them separately. Bootstrapped and tested on x86_64-unknown-linux-gnu, testing

Re: PATCH: copy_addr_to_reg: Convert to Pmode if needed

2011-03-21 Thread Richard Sandiford
"H.J. Lu" writes: > diff --git a/gcc/explow.c b/gcc/explow.c > index 460af1f..9b3c082 100644 > --- a/gcc/explow.c > +++ b/gcc/explow.c > @@ -611,6 +611,8 @@ copy_to_reg (rtx x) > rtx > copy_addr_to_reg (rtx x) > { > + if (GET_MODE (x) != VOIDmode && GET_MODE (x) != Pmode) > +x = convert_to

Re: [PATCH] Fix PR48210

2011-03-21 Thread Richard Guenther
On Mon, 21 Mar 2011, Rainer Orth wrote: > Richard Guenther writes: > > > DEFPARAM (MIN_PARTITION_SIZE, > > "lto-min-partition", > > - "Size of minimal paritition for WHOPR (in estimated instructions)", > > + "Minimal size of a paritition for LTO (in estimated instructions)", > >

Re: [PATCH] Fix PR48210

2011-03-21 Thread Rainer Orth
Richard Guenther writes: > DEFPARAM (MIN_PARTITION_SIZE, > "lto-min-partition", > - "Size of minimal paritition for WHOPR (in estimated instructions)", > + "Minimal size of a paritition for LTO (in estimated instructions)", Another typo :-)^ Rainer

[patch committed] rs6000/t-freebsd trunk

2011-03-21 Thread Andreas Tobler
All, I committed this obvious patch also to trunk. Andreas 2011-03-21 Andreas Tobler * config/rs6000/t-freebsd: Remove duplication from file. Index: t-freebsd === --- t-freebsd (revision 171238) +++ t-freebsd (wor

Re: [PATCH] Fix PR48210

2011-03-21 Thread Richard Guenther
On Mon, 21 Mar 2011, Joseph S. Myers wrote: > On Mon, 21 Mar 2011, Rainer Orth wrote: > > > Richard Guenther writes: > > > > > Index: gcc/params.def > > > === > > > --- gcc/params.def(revision 171229) > > > +++ gcc/params.d

Re: [PATCH] Fix PR 48143

2011-03-21 Thread Uros Bizjak
2011/3/21 Andrey Belevantsev : > Hello, > > As noted in the PR audit trail, the scheduler crashes for this bug because > we see an SSE insn without proper reservation.  Uros has pointed several > such insns to me.  The following patch adding athlon_decode attributes is > bootstrapped and tested on

[patch commited] rs6000/t-freebsd 4.6 branch

2011-03-21 Thread Andreas Tobler
All, I committed the obvious patch below after asking Jakub. Andreas 2011-03-21 Andreas Tobler * config/rs6000/t-freebsd: Remove duplication from file. Index: t-freebsd === --- t-freebsd (revision 171238) +++ t-free

invoke.texi: Fix minor typos

2011-03-21 Thread Nick Clifton
Hi Gerald, Hi Jospeh, The patch below fixes a bunch of minor typos in the invoke.texi file. I suspect that it might qualify as obvious, but I though that I ought to run it past you first. OK to apply ? Cheers Nick gcc/ChangeLog 2011-03-21 Nick Clifton * doc/invoke.texi (Ov

Re: [build, doc] Obsolete Solaris 8 support

2011-03-21 Thread Joseph S. Myers
On Mon, 21 Mar 2011, Rainer Orth wrote: > Therefore I'll soon commit this patch to obsolete Solaris 8 support in > 4.7. It will be removed in 4.8 then. When obsoleting targets or other features, please update the release notes (gcc-4.7/changes.html). -- Joseph S. Myers jos...@codesourcery.com

Re: [PATCH] Fix PR48210

2011-03-21 Thread Joseph S. Myers
On Mon, 21 Mar 2011, Rainer Orth wrote: > Richard Guenther writes: > > > Index: gcc/params.def > > === > > --- gcc/params.def (revision 171229) > > +++ gcc/params.def (working copy) > > @@ -868,7 +868,7 @@ DEFPARAM (PARAM_DEVIRT_T

Re: [PATCH 02/18] enforce TREE_CHAIN and TREE_TYPE accesses

2011-03-21 Thread Nathan Froyd
On Sat, Mar 12, 2011 at 01:08:23PM +0100, Eric Botcazou wrote: > > gcc/ada/ > > * gcc-interface/ada-tree.h (union lang_tree_node): Check for > > TS_COMMON before calling TREE_CHAIN. > > * gcc-interface/misc.c (gnat_init_ts): New function. > > (LANG_HOOKS_INIT_TS): Define. > > I pre

[PATCH] Weaken PR47661

2011-03-21 Thread Richard Guenther
We force all maybe-trapping conditions into separate stmts, but this isn't necessary w/o -fno-call-exceptions. It also confuses our branch prediction. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. The bug remains for -fnon-call-exceptions. Richard. 2011-03-21 Richard

Re: [PATCH] Fix PR48210

2011-03-21 Thread Rainer Orth
Richard Guenther writes: > Index: gcc/params.def > === > --- gcc/params.def(revision 171229) > +++ gcc/params.def(working copy) > @@ -868,7 +868,7 @@ DEFPARAM (PARAM_DEVIRT_TYPE_LIST_SIZE, > DEFPARAM (PARAM_LTO_PARTITIONS, >

[PATCH] Fix PR48210

2011-03-21 Thread Richard Guenther
Pretty obvious, require at least 1 partition. Richard. 2011-03-31 Richard Guenther PR lto/48210 * params.def (lto-partitions): Require at least 1 partition. Index: gcc/params.def === --- gcc/params.def (rev

[Patch] Disable -mfdpic for bfin-uclinux target

2011-03-21 Thread Henderson, Stuart
-mfdpic is not supported for the bfin-uclinux target. this patch defines a macro in the uclinux header and then adds a check to bfin.c if the macro is defined. 2011-03-21 Stuart Henderson Originally from Bernd Schmidt * config/bfin/uclinux.h (SUBTARGET_FDPIC_NOT_SUPPORTED): New macro.

Re: [testsuite] Don't xfail 22_locale/num_put/put/char/14220.cc on 64-bit Solaris 10/SPARC

2011-03-21 Thread Richard Guenther
On Mon, 21 Mar 2011, Rainer Orth wrote: > 22_locale/num_put/put/char/14220.cc XPASSes on 64-bit Solaris 10/SPARC > for quite some time, so I'm removing the corresponding xfail. > > Tested with the appropriate runtest invocations, installed on 4.5 branch > and mainline. Ok for 4.6, too? Ok for 4

Re: [build] Honor --disable-symvers (PR bootstrap/48135)

2011-03-21 Thread Richard Guenther
On Mon, 21 Mar 2011, Rainer Orth wrote: > Richard Guenther writes: > > >>> Ok for 4.7, RMs should decide about 4.6.1. > >> > >> Applied, thanks.  It would be nice to get into 4.6 (even 4.6.1) since > >> otherwise install.texi which claims --disable-symvers as a workaround > >> lies :-) > > > > O

[testsuite] Don't xfail 22_locale/num_put/put/char/14220.cc on 64-bit Solaris 10/SPARC

2011-03-21 Thread Rainer Orth
22_locale/num_put/put/char/14220.cc XPASSes on 64-bit Solaris 10/SPARC for quite some time, so I'm removing the corresponding xfail. Tested with the appropriate runtest invocations, installed on 4.5 branch and mainline. Ok for 4.6, too? Thanks. Rainer 2011-03-12 Rainer Orth

Re: [build] Honor --disable-symvers (PR bootstrap/48135)

2011-03-21 Thread Rainer Orth
Richard Guenther writes: >>> Ok for 4.7, RMs should decide about 4.6.1. >> >> Applied, thanks.  It would be nice to get into 4.6 (even 4.6.1) since >> otherwise install.texi which claims --disable-symvers as a workaround >> lies :-) > > Ok. Just to make no mistake: ok for 4.6.0 or ok for 4.6.1?

Re: [build] Honor --disable-symvers (PR bootstrap/48135)

2011-03-21 Thread Richard Guenther
On Mon, Mar 21, 2011 at 1:39 PM, Rainer Orth wrote: > Paolo Bonzini writes: > >> On 03/21/2011 12:34 PM, Rainer Orth wrote: >>> Prompted by PR bootstrap/48135, I found that only libstdc++-v3 honors >>> --disable-symvers.  The following patch corrects this, but also notices >>> again the mess we'v

[PATCH] Fix PR 48143

2011-03-21 Thread Andrey Belevantsev
Hello, As noted in the PR audit trail, the scheduler crashes for this bug because we see an SSE insn without proper reservation. Uros has pointed several such insns to me. The following patch adding athlon_decode attributes is bootstrapped and tested on x86_64. Uros, you have preapproved t

Re: [Ada] Implement switches to reorder record components in gigi

2011-03-21 Thread Duncan Sands
Hi Eric, this looks like a nice improvement. I noticed this mysterious undef though - what is it for? +#undef MOVE_FROM_FIELD_LIST_TO Ciao, Duncan.

Re: [testsuite, build] Convert boehm-gc testsuite to DejaGnu (PR boehm-gc/11412)

2011-03-21 Thread Rainer Orth
Mike Stump writes: >> +catch { exec $libtool --mode=clean rm -f $basename.lo } > > If libtool runs on the host, this doesn't do that, try: > > [remote_exec host ... ] > > see gcc.exp for examples. > >> +catch { exec $libtool --mode=clean rm -f [file rootname [file tail >> $testcase]] $

Re: [build] Link with libstdc++ for libpwl test (PR bootstrap/48120)

2011-03-21 Thread Paolo Bonzini
On 03/21/2011 01:40 PM, Rainer Orth wrote: Paolo Bonzini writes: On 03/21/2011 12:48 PM, Rainer Orth wrote: As reported in the PR, the toplevel configure test for libpwl doesn't work since it fails to link with libstdc++. The following patch corrects this. There were a couple of issues: *

Re: [patch] Fix PR48183, NEON ICE in emit-rtl.c:immed_double_const() under -g

2011-03-21 Thread Chung-Lin Tang
On 2011/3/20 08:11 PM, Richard Guenther wrote: > On Sun, Mar 20, 2011 at 12:01 PM, Chung-Lin Tang > wrote: >> Hi, >> PR48183 is a case where ARM NEON instrinsics, under -O -g, produce debug >> insns that tries to expand OImode (32-byte integer) zero constants, much >> too large to represent as two

Re: [build] Link with libstdc++ for libpwl test (PR bootstrap/48120)

2011-03-21 Thread Rainer Orth
Paolo Bonzini writes: > On 03/21/2011 12:48 PM, Rainer Orth wrote: >> As reported in the PR, the toplevel configure test for libpwl doesn't >> work since it fails to link with libstdc++. The following patch >> corrects this. >> >> There were a couple of issues: >> >> * One cannot use LDFLAGS her

Re: [build] Honor --disable-symvers (PR bootstrap/48135)

2011-03-21 Thread Rainer Orth
Paolo Bonzini writes: > On 03/21/2011 12:34 PM, Rainer Orth wrote: >> Prompted by PR bootstrap/48135, I found that only libstdc++-v3 honors >> --disable-symvers. The following patch corrects this, but also notices >> again the mess we've created by having at least 3 different sets of >> checks f

[Ada] Implement switches to reorder record components in gigi

2011-03-21 Thread Eric Botcazou
This is the gigi part of the implementation of the -gnatd.r/-gnatd.v switches that allow record components to be reordered in certain cases. The algorithm is documented in the code. Tested on i586-suse-linux, applied on the mainline. 2011-03-21 Eric Botcazou * gcc-interface/decl.c

Re: [patch] Couple of tweaks to the gimplifier

2011-03-21 Thread Richard Guenther
On Mon, Mar 21, 2011 at 12:19 PM, Eric Botcazou wrote: > Hi, > > the attached patch makes a couple of tweaks to the gimplifier in order to help > Ada, but I think that they are of general usefulness: > >  1) Set TREE_THIS_NOTRAP on the INDIRECT_REF built for VLA decls.  This is >     correct since

PING^2: Reducing the overhead of dwarf2 location tracking

2011-03-21 Thread Richard Sandiford
Second ping for this patch (4.5+): http://gcc.gnu.org/ml/gcc-patches/2011-03/msg00255.html It fixes a source of quadratic memory use while generating dwarf2 var-tracking information. Thanks, Richard

[Ada] Avoid duplicating branches in loop conditions

2011-03-21 Thread Eric Botcazou
This avoids duplicating branches in the generated code at -O0 for complex loop conditions involving boolean expressions. Tested on i586-suse-linux, applied on the mainline. 2011-03-21 Eric Botcazou * gcc-interface/decl.c (elaborate_expression_1): When optimization is disable

Re: [doc] Update install.texi for Solaris 2, IRIX, Tru64 UNIX

2011-03-21 Thread Rainer Orth
Gerald Pfeifer writes: > On Thu, 17 Mar 2011, Rainer Orth wrote: >> I've just reviewed install.texi for my platforms and found it mostly >> accurate. A few minor changes are desirable for the 4.6 release, but >> also apply to mainline since the file is identical there: >> >> * Update recommenda

[libgo] Support GOTESTFLAGS

2011-03-21 Thread Rainer Orth
The following patch came in handy when I need to repeat an individual libgo test and investigate the resulting binary: $ make GOTESTFLAGS=--keep /check does the trick and is far neater than modifying the generated Makefile. Rainer 2011-03-20 Rainer Orth * Makefile.am (CHECK

[build, doc] Remove --enable-threads=solaris support

2011-03-21 Thread Rainer Orth
We still have support for --enable-threads=solaris (the old Unix International thread library) in GCC. This is pretty pointless IMO: * Both libthread and libpthread are only two different interfaces to the same implementation. * Most of our runtime libraries don't support libthread at all. *

Re: [build] Link with libstdc++ for libpwl test (PR bootstrap/48120)

2011-03-21 Thread Paolo Bonzini
On 03/21/2011 12:48 PM, Rainer Orth wrote: As reported in the PR, the toplevel configure test for libpwl doesn't work since it fails to link with libstdc++. The following patch corrects this. There were a couple of issues: * One cannot use LDFLAGS here since that is passed to $CC before -lpwl,

Re: [Patch][AVR]: Support tail calls

2011-03-21 Thread Georg-Johann Lay
Denis Chertykov schrieb: > 2011/3/18 Georg-Johann Lay : > >>> Is it tested for regressions ? >>> >>> Denis. >> I ran tests against svn 170942 (latest 4.7.0 snapshot). Besides >> timestamps, the diff looks like this: >> >> 1435a1436,1437 >>> XPASS: gcc.dg/sibcall-3.c execution test >>> XPASS: gcc.d

[build] Link with libstdc++ for libpwl test (PR bootstrap/48120)

2011-03-21 Thread Rainer Orth
As reported in the PR, the toplevel configure test for libpwl doesn't work since it fails to link with libstdc++. The following patch corrects this. There were a couple of issues: * One cannot use LDFLAGS here since that is passed to $CC before -lpwl, which of course fails with a static libpwl

Re: [doc] Improve support library requirements documentation (PR bootstrap/48135)

2011-03-21 Thread Richard Guenther
On Mon, 21 Mar 2011, Rainer Orth wrote: > While investigating PR bootstrap/48135, it came up that the submitter > had run into trouble bootstrapping with mpc 0.9.0. Unfortunately, > install.texi when describing support library requirements always lists > `(or later)' with the individual libraries

Re: [build] Honor --disable-symvers (PR bootstrap/48135)

2011-03-21 Thread Paolo Bonzini
On 03/21/2011 12:34 PM, Rainer Orth wrote: Prompted by PR bootstrap/48135, I found that only libstdc++-v3 honors --disable-symvers. The following patch corrects this, but also notices again the mess we've created by having at least 3 different sets of checks for symbol versioning, each with its

Re: [doc] Improve Perl requirements documentation (PR bootstrap/48135)

2011-03-21 Thread Richard Guenther
On Mon, 21 Mar 2011, Rainer Orth wrote: > While reviewing PR bootstrap/48135, it came up that the Perl > requirements in install.texi are inaccurate: > > * On some platforms (Darwin, Solaris), Perl is a build-time requirement, > but documented with the development requirements only. > > * On S

[build, doc] Avoid File::Glob in in make_sunver.pl (PR bootstrap/48135)

2011-03-21 Thread Rainer Orth
Prompted by PR bootstrap/48135, I had a closer look at make_sunver.pl and noticed that it doesn't use File::Glob at all, but turns globs in the input version script into a regular expression for matching. Therefore, the requirement of a Perl version newer than what is bundled in Solaris 8 should go

[build] Honor --disable-symvers (PR bootstrap/48135)

2011-03-21 Thread Rainer Orth
Prompted by PR bootstrap/48135, I found that only libstdc++-v3 honors --disable-symvers. The following patch corrects this, but also notices again the mess we've created by having at least 3 different sets of checks for symbol versioning, each with its own variations ;-( But this is not the time

extend.texi: Fix grammatical problems with builtin machine function descriptions

2011-03-21 Thread Nick Clifton
Hi Gerald, Hi Joseph, I have a attached a mostly mechanical patch to fix a couple of very minor grammatical issues with the built-in machine function descriptions in the extend.texi file. The first issue is with this sentence: They all generate the machine instruction that is part of

[build, doc] Obsolete Solaris 8 support

2011-03-21 Thread Rainer Orth
Continuing to support Solaris 8 is starting to cause lots of trouble: * Regular support has ended in March 2009, and even extended support with special arrangements will end in March 2012. * Access to Solaris 8 patches is restricted to customers with such arrangements, which I'm not. * While

  1   2   >