From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 3715 invoked by alias); 1 Aug 2002 18:40:09 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Received: (qmail 3700 invoked from network); 1 Aug 2002 18:40:07 -0000 Received: from unknown (HELO hiauly1.hia.nrc.ca) (132.246.100.193) by sources.redhat.com with SMTP; 1 Aug 2002 18:40:07 -0000 Received: from hiauly1.hia.nrc.ca (localhost [127.0.0.1]) by hiauly1.hia.nrc.ca (8.12.0.Beta16/8.12.0.Beta16) with ESMTP id g71Idk00004955; Thu, 1 Aug 2002 14:39:46 -0400 (EDT) Received: (from dave@localhost) by hiauly1.hia.nrc.ca (8.12.0.Beta16/8.12.0.Beta16) id g71Idjtt004954; Thu, 1 Aug 2002 14:39:45 -0400 (EDT) Message-Id: <200208011839.g71Idjtt004954@hiauly1.hia.nrc.ca> Subject: Re: RFC: New approach to --with-cpu To: drow@mvista.com (Daniel Jacobowitz) Date: Thu, 01 Aug 2002 11:40:00 -0000 From: "John David Anglin" Cc: Richard.Earnshaw@arm.com, gcc-patches@gcc.gnu.org In-Reply-To: <20020801152250.GA14239@nevyn.them.org> from "Daniel Jacobowitz" at Aug 1, 2002 11:22:50 am MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-SW-Source: 2002-08/txt/msg00039.txt.bz2 > On Thu, Aug 01, 2002 at 11:15:10AM -0400, John David Anglin wrote: > > > + hppa*-*-* | parisc*-*-*) > > > + if test x$gas = xyes > > > + then > > > + target_cpu_default2="MASK_GAS|MASK_JUMP_IN_DELAY" > > > + fi > > > + case $machine in > > > + hppa1* | parisc1*) > > > + with_cpu=7100LC > > ^^^^^^^^ > > > > I believe that this should be "with_schedule". I was also wondering > > if a comment should be added to indicate that the default scheduling > > model is "8000". > > And you're completely correct on both counts. I've added that to my > working copy. I enclose below an update to above code. I have attempted to bullet proof it so that it handles error conditions gracefully and only overrides the PA8000 default based on the target cpu when the user doesn't specify --with-schedule. I have run through partial builds checking most of the option permutations and the patch seems to be working well. I like the ease with which the scheduling model can now be specified. The only additional wish is for an easy way to print the target defaults. Dave -- J. David Anglin dave.anglin@nrc.ca National Research Council of Canada (613) 990-0752 (FAX: 952-6605) --- config.gcc.orig 2002-08-01 09:36:20.000000000 -0700 +++ config.gcc 2002-08-01 11:19:24.000000000 -0700 @@ -2908,9 +2908,21 @@ then target_cpu_default2="MASK_GAS|MASK_JUMP_IN_DELAY" fi - case $machine in - hppa1* | parisc1*) - with_cpu=7100LC + case ".$with_schedule" in + .700 | .7100 | .7100LC | .7200 | .7300 | .8000) + ;; + .) + case $machine in + hppa1* | parisc1*) + # Override default PA8000 + # scheduling model. + with_schedule=7100LC + ;; + esac + ;; + *) + echo "Unknown cpu used with --with-schedule=$with_schedule" 1>&2 + exit 1 ;; esac ;;