On Fri, 18 Mar 2011, Ralf Wildenhues wrote:
> > @@ -744,7 +744,6 @@ case "${target}" in
> > libgloss_dir=cris
> > ;;
> > crx-*-*)
> > - noconfigdirs="$noconfigdirs target-libstdc++-v3 target-mudflap
> > ${libgcj}"
> > ;;
>
> Why not also remove the line before and after this one?
> Is that because crx is still supported in binutils or other
> src projects? If yes, the hunk is fine, but then I wonder
> whether it is too early to drop the config-ml.in bits for src.
Yes, it is still supported in binutils - and the general reason for not
removing the empty cases is that there are plenty of such cases already
there, some for truly ancient targets. It seems quite likely that some of
the targets mentioned at toplevel actually have no useful support anywhere
in gcc or src and that all of those target cases should be reviewed for
usefulness and correctness. (There are also lots of cases for "canonical
aliases" that have been deprecated and removed from GCC - that is, names
that should be aliases but that config.sub treats as canonical and so
cause problems because of needing duplication in case statements etc.
everywhere. In those cases it probably makes sense to remove support for
the duplicate names in src, along maybe with making config.sub convert
them to canonical form. i?86-*-pe as an alias for Cygwin is an example in
the present patch; various such aliases were obsoleted in 4.3 as well, but
support for "thumb-*" targets was obsolete long before the other ARM
aliases obsoleted in 4.3.)
--
Joseph S. Myers
jos...@codesourcery.com
