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 jos...@codesourcery.com
