Patchwork Remove config.gcc support for *local* configurations

login
register
mail settings
Submitter Joseph S. Myers
Date June 30, 2011, 10:34 p.m.
Message ID <Pine.LNX.4.64.1106302233240.22846@digraph.polyomino.org.uk>
Download mbox | patch
Permalink /patch/102842/
State New
Headers show

Comments

Joseph S. Myers - June 30, 2011, 10:34 p.m.
config.gcc has special handling of triplets matching "*local*".  I
don't think this is particularly relevant to how GCC is used and built
today; a good GCC port is expected to include many pieces in different
places that are not covered by this generic code, and adding a
config.gcc stanza is one of the easiest parts of adding a port.  Thus,
this patch removes that code.

Bootstrapped with no regressions on x86_64-unknown-linux-gnu.  OK to
commit?

gcc:
2011-06-30  Joseph Myers  <joseph@codesourcery.com>

	* config.gcc (*local*): Remove.
	* doc/install-old.texi: Don't mention local configurations.

libgcc:
2011-06-30  Joseph Myers  <joseph@codesourcery.com>

	* config.host (*local*): Remove.
Joseph S. Myers - July 7, 2011, 3:49 p.m.
Ping.  This patch 
<http://gcc.gnu.org/ml/gcc-patches/2011-06/msg02408.html> is pending 
review.
Paolo Bonzini - July 7, 2011, 3:58 p.m.
On 07/07/2011 05:49 PM, Joseph S. Myers wrote:
> Ping.  This patch
> <http://gcc.gnu.org/ml/gcc-patches/2011-06/msg02408.html>  is pending
> review.
>

Ok.

Paolo

Patch

Index: libgcc/config.host
===================================================================
--- libgcc/config.host	(revision 175628)
+++ libgcc/config.host	(working copy)
@@ -222,13 +222,6 @@ 
 esac
 
 case ${host} in
-# Support site-specific machine types.
-*local*)
-	rest=`echo ${host} | sed -e "s/$cpu_type-//"`
-	if test -f $srcdir/config/${cpu_type}/t-$rest
-	then tmake_file=${cpu_type}/t-$rest
-	fi
-	;;
 alpha*-*-linux*)
 	tmake_file="${tmake_file} alpha/t-crtfm"
 	extra_parts="$extra_parts crtfastmath.o"
Index: gcc/doc/install-old.texi
===================================================================
--- gcc/doc/install-old.texi	(revision 175628)
+++ gcc/doc/install-old.texi	(working copy)
@@ -183,13 +183,3 @@ 
 @noindent
 Remember that a machine name specifies both the cpu type and the company
 name.
-If you want to install your own homemade configuration files, you can
-use @samp{local} as the company name to access them.  If you use
-configuration @samp{@var{cpu}-local}, the configuration name
-without the cpu prefix
-is used to form the configuration file names.
-
-Thus, if you specify @samp{m68k-local}, configuration uses
-files @file{m68k.md}, @file{local.h}, @file{m68k.c},
-@file{xm-local.h}, @file{t-local}, and @file{x-local}, all in the
-directory @file{config/m68k}.
Index: gcc/config.gcc
===================================================================
--- gcc/config.gcc	(revision 175628)
+++ gcc/config.gcc	(working copy)
@@ -749,17 +749,6 @@ 
 esac
 
 case ${target} in
-# Support site-specific machine types.
-*local*)
-	rest=`echo ${target} | sed -e "s/$cpu_type-//"`
-	tm_file=${cpu_type}/$rest.h
-	if test -f $srcdir/config/${cpu_type}/xm-$rest.h
-	then xm_file=${cpu_type}/xm-$rest.h
-	fi
-	if test -f $srcdir/config/${cpu_type}/t-$rest
-	then tmake_file=${cpu_type}/t-$rest
-	fi
-	;;
 alpha*-*-linux*)
 	tm_file="${tm_file} alpha/elf.h alpha/linux.h alpha/linux-elf.h glibc-stdint.h"
 	extra_options="${extra_options} alpha/elf.opt"