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