Re: RFA; MN10300: Fix AM33 clzsi2 pattern

2011-06-25 Thread Nick Clifton
Hi Richard, The clzsi2/bsch patterns in the MN10300 backend do not work. There are two problems - firstly the starting bit-search position for the BSCH instruction is not set. Yes it is. What do you think that "unused" second operand does? That actually works ? Gross! Secondly

Re: [testsuite] scan-assembler variants to use 'unresolved' for missing file

2011-06-25 Thread Mike Stump
On Jun 20, 2011, at 1:34 PM, Janis Johnson wrote: > Variants of scan-assembler, used in the dg-final steps of a test, do > not check that the output file exists, so it's reported as an error. > With this patch, if the file is missing then the check is reported as > unresolved using the same message

Re: [testsuite] skip ARM neon-fp16 tests for other -mcpu values

2011-06-25 Thread Mike Stump
On Jun 15, 2011, at 5:57 PM, Janis Johnson wrote: > The bug was in my attempt to run the tests with other -mfpu values, so > I'm very glad you caught that. I tried again, getting rid of the neon > requirement along the way, and found a way to run the VFP fp16 tests > with any of the fp16 values th

Re: [testsuite] dg-require-effective-target: skip unneeded checks

2011-06-25 Thread Mike Stump
On Jun 21, 2011, at 2:02 PM, Janis Johnson wrote: > This patch causes dg-require-effective-target to return early if the > test is already being skipped, saving some work. There's already > similar code in dg-skip-if. > > OK for trunk, and later for 4.6? Ok. Ok for 4.6. I'm not a huge fan of u

Re: Patch ping

2011-06-25 Thread Mike Stump
On Jun 25, 2011, at 11:53 AM, Eric Botcazou wrote: >> [testsuite] >> http://gcc.gnu.org/ml/gcc-patches/2011-06/msg01069.html >> PR tree-optimization/48377, PR middle-end/49191 >> trunk/4.6 >> non_strict_align testsuite support > > Mike, Rainer, can one of you two take a look at this? Done, re

Re: [PATCH] Only run pr48377.c testcase on i?86/x86_64

2011-06-25 Thread Mike Stump
On Jun 14, 2011, at 8:53 AM, Jakub Jelinek wrote: > On Tue, Jun 14, 2011 at 04:52:18PM +0200, Eric Botcazou wrote: >>> Well, Steve has a patch for non_strict_align effective_target >>> in http://gcc.gnu.org/ml/gcc-patches/2011-06/msg00673.html >>> (with s/strict_align/non_strict_align/g ), I was ho

Re: PATCH [8/n]: Prepare x32: PR other/48007: Unwind library doesn't work with UNITS_PER_WORD > sizeof (void *)

2011-06-25 Thread H.J. Lu
On Sat, Jun 25, 2011 at 1:19 PM, Joseph S. Myers wrote: > On Sat, 25 Jun 2011, H.J. Lu wrote: > >> +#ifndef UNIQUE_UNWIND_CONTEXT > > The use of #ifndef seems to imply that this is a target macro, to be > defined in libgcc_tm.h.  In that case it should be documented, and > poisoned in system.h und

Re: PATCH [8/n]: Prepare x32: PR other/48007: Unwind library doesn't work with UNITS_PER_WORD > sizeof (void *)

2011-06-25 Thread Joseph S. Myers
On Sat, 25 Jun 2011, H.J. Lu wrote: > +#ifndef UNIQUE_UNWIND_CONTEXT The use of #ifndef seems to imply that this is a target macro, to be defined in libgcc_tm.h. In that case it should be documented, and poisoned in system.h under the "only used for code built for the target" case, and this:

PR 49337: Make Gnatmake look for libgnat.so if it doesn't find libgnat.a.

2011-06-25 Thread Björn Persson
Hello! The maintainers of GCC in Fedora have split out libgnat.a and libgnarl.a to a separate subpackage which is by default not installed together with the Gnat tools. (Fedora has a policy to use only shared libraries as much as possible.) This causes Gnatmake to crash when it tries to find the d

Re: PR tree-optimize/49373 (IPA-PTA regression)

2011-06-25 Thread Jan Hubicka
Hi, just for those who are interested, this is quick&dirty patch adding another pass of local optimization passes at WPA time. I've added early inliner and IPA-SRA because I was curious how much of optimization oppurtunities we are missing by limiting those to early pass. With Early inlining it s

Re: Patch ping

2011-06-25 Thread Eric Botcazou
> [testsuite] > http://gcc.gnu.org/ml/gcc-patches/2011-06/msg01069.html > PR tree-optimization/48377, PR middle-end/49191 > trunk/4.6 > non_strict_align testsuite support Mike, Rainer, can one of you two take a look at this? -- Eric Botcazou

PATCH [11/n]: Prepare x32: PR rtl-optimization/48155: Reload doesn't handle subreg properly

2011-06-25 Thread H.J. Lu
Hi, This is the last target independent patch for x32. I will start submitting x86 specific patches in a week. Given input: (plus:SI (subreg:SI (plus:DI (reg/f:DI 7 sp) (const_int 16 [0x10])) 0) (const_int -1 [0x])) reloads tries to add (subreg:SI (plus:DI (reg

PATCH [10/n]: Prepare x32: PR rtl-optimization/49114: Reload failed to handle (set reg:X (plus:X (subreg:X (reg:Y) 0) (const_int)))

2011-06-25 Thread H.J. Lu
Hi, When reload gets: (insn 588 587 589 28 (set (mem:DF (zero_extend:DI (plus:SI (subreg:SI (reg/v/f:DI 182 [ b ]) 0) (const_int 8 [0x8]))) [4 MEM[base: b_96(D), index: D.15020_278, step: 8, offset: 0B]+0 S8 A64]) (reg:DF 340 [ D.14980 ])) spooles.c:291 106 {*movdf_in

Re: RFA; MN10300: Fix AM33 clzsi2 pattern

2011-06-25 Thread Richard Henderson
On 06/25/2011 01:18 AM, Nick Clifton wrote: > Hi Alex, Hi Jeff, Hi Richard, > > The clzsi2/bsch patterns in the MN10300 backend do not work. There > are two problems - firstly the starting bit-search position for the > BSCH instruction is not set. Yes it is. What do you think that "unuse

PATCH [9/n]: Prepare x32: PR middle-end/47383: ivopts miscompiles Pmode != ptr_mode

2011-06-25 Thread H.J. Lu
Hi, I was informed that MEM_REF only works in ptr_mode. This patch changes addr_for_mem_ref to use ptr_mode. OK for trunk? Thanks. H.J. --- 2011-06-25 H.J. Lu PR middle-end/47383 * tree-ssa-address.c (addr_for_mem_ref): Use ptr_mode instead of targetm.addr_space.ad

PATCH [8/n]: Prepare x32: PR other/48007: Unwind library doesn't work with UNITS_PER_WORD > sizeof (void *)

2011-06-25 Thread H.J. Lu
Hi, This patch introduces UNIQUE_UNWIND_CONTEXT and properly saves/stores registers with UNITS_PER_WORD > sizeof (void *) as suggested in http://gcc.gnu.org/ml/gcc-patches/2011-03/msg01526.html OK for trunk? Thanks. H.J. --- 2011-04-09 H.J. Lu PR other/48007 * unwind-dw2.c

Re: PATCH [5/n]: Prepare x32: PR middle-end/48016: Inconsistency in non-local goto save area

2011-06-25 Thread H.J. Lu
On Thu, Jun 16, 2011 at 10:18 AM, H.J. Lu wrote: > On Thu, Jun 16, 2011 at 12:56 AM, Richard Guenther > wrote: >> On Wed, Jun 15, 2011 at 9:55 PM, H.J. Lu wrote: >>> On Wed, Jun 15, 2011 at 8:16 AM, Michael Matz wrote: Hi, On Wed, 15 Jun 2011, H.J. Lu wrote: > >> +  /* F

Re: [PATCH (0/7)] Improve use of Widening Multiplies

2011-06-25 Thread Bernd Schmidt
On 06/23/11 16:34, Andrew Stubbs wrote: > The patches provide a number of improvements: > > * Support for instructions that widen by more than one mode >(e.g. from HImode to DImode). > > * Use of widening multiplies even when the input mode is narrower than >the instruction uses. (e.g.

[patch, darwin, committed] fix PR49371

2011-06-25 Thread Iain Sandoe
I committed the following patch as approved in the PR thread. cheers Iain gcc/ PR driver/49371 * config/darwin.c (darwin_override_options): Improve warning when mdynamic-no-pic is given together with fPIC/fpic, also warn when it is given with fpie/fPIE. * c

Re: Simplify Solaris configuration

2011-06-25 Thread Eric Botcazou
> All bootstraps completed without regressions, so I've installed the > patch. I'll address eventual issues and further simplifications as a > followup. I cannot bootstrap SPARC64/Solaris anymore though: /nile.build/botcazou/gcc-head/sparc64-sun-solaris2.9/./gcc/xgcc -B/nile.build/botcazou/gcc-

Re: PATCH TRUNK: better format output for time reports.

2011-06-25 Thread Eric Botcazou
> > Ok. > > Committed revision 175396. Please avoid posting messages like this. See http://gcc.gnu.org/svnwrite.html: "When you have checked in a patch exactly as it has been approved, you do not need to tell that to people -- including the approver. People interested in when a particular patc

Re: PATCH TRUNK: better format output for time reports.

2011-06-25 Thread Basile Starynkevitch
On Sat, 25 Jun 2011 11:38:20 +0200 Richard Guenther wrote: > > ### patch against trunk 175201 > > Index: gcc/timevar.c > > === > > --- gcc/timevar.c       (revision 175201) > > +++ gcc/timevar.c       (working copy) > > @@ -478,7

Re: PATCH TRUNK: better format output for time reports.

2011-06-25 Thread Richard Guenther
On Sat, Jun 25, 2011 at 8:40 AM, Basile Starynkevitch wrote: > Hello All, > > When cc1 report timing, the timing variable name has a too short width for df > reg dead/unused notes: > > Execution times (seconds) >  phase setup           :   0.00 ( 0%) usr   0.00 ( 0%) sys   0.01 ( 0%) wall   >  1

Re: [PATCH (3/7)] Widening multiply-and-accumulate pattern matching

2011-06-25 Thread Richard Guenther
On Fri, Jun 24, 2011 at 6:58 PM, Stubbs, Andrew wrote: > On 24/06/11 16:47, Richard Guenther wrote: >>> >  I can certainly add checks to make sure that the skipped operations >>> >  actually don't make any important changes to the value, but do I need to? >> Yes. > > Ok, I'll go away and do that t

RFA; MN10300: Fix AM33 clzsi2 pattern

2011-06-25 Thread Nick Clifton
Hi Alex, Hi Jeff, Hi Richard, The clzsi2/bsch patterns in the MN10300 backend do not work. There are two problems - firstly the starting bit-search position for the BSCH instruction is not set. Secondly the BSCH instruction returns the bit position of the highest set bit, not the number