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.

[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

[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

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

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

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: 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

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: [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 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: 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 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 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][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,

[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

[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

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

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

[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

[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 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

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

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

<    1   2