4.7 deprecated targets removal patch
Joseph S. Myers
joseph@codesourcery.com
Fri Mar 18 23:11:00 GMT 2011
On Fri, 18 Mar 2011, Ralf Wildenhues wrote:
> > > > - 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.
> > >
> > > I can understand keeping them when there are later case matches that
> > > might match the pattern in question. That doesn't apply to crx-*-*
> > > however.
> >
> > Well, it would match the *-*-* case (which is fairly harmless). But just
> > the same would seem to apply to the i370-*-opened* case, for example - I
> > don't see anything else that would match other than *-*-*.
>
> i370-ibm-opened* would. I have no idea how relevant that is; it only
> sets tentative_cc.
i370-ibm-opened* is in a different case statement. In the case statement
with an empty i370-*-opened* case the only other case matching it would
seem to be *-*-*.
Can some global reviewer review the patch apart from the toplevel
config-ml.in / configure.ac changes? Those toplevel changes are purely
cleanups that are not required by the rest of the patch.
--
Joseph S. Myers
joseph@codesourcery.com
More information about the Gcc-patches
mailing list