Re: [PATCH] ARM autogenerate the tune attribute

[Richard Earnshaw <rearnsha at gcc dot gnu dot org>]

On Tue, 2004-08-03 at 15:17, Paul Brook wrote:
> On Tuesday 03 August 2004 15:03, Phil Edwards wrote:
> > On Tue, Aug 03, 2004 at 02:30:43PM +0100, Richard Earnshaw wrote:
> > > + #!/bin/sh
> > > + # Generate arm-tune.md, a file containing the tune attribute from the
> > > list of + # CPUs in arm-cores.def
> > > +
> > > + echo ";; -*- buffer-read-only: t -*-"
> >
> > Hey, that's editor discrimination!
> >
> > More seriously, I might try moving that script to config/ and generalizing
> > it so it can be used for other cpus, since this looks like a good idea.
> > If I do, I will update it for ARM and run the changes past you first.
> > (And might also emit a read-only directive for vim.  *grin*)
> 
> I had a (failed) attempt at automating the dependency tracking a while ago, 
> but never got round to fixing it properly.
> http://gcc.gnu.org/ml/gcc-patches/2004-01/msg00190.html
> 
> Paul

Yeah!  For the moment I side-stepped that issue by adding a manual
dependency in t-arm.  It would be better to move at least part of that
out into Makefile.in (the various s-<xxx> targets could depend on
MD_INCLUDES in my patch).  The back-ends would still have to manually
generate the dependency list, but that's still a step forward from where
we are at present (no checks at all).

I did wonder about making the autogenerated arm-tune.md appear in the
current build directory (which would simplify some things), but that
would be hard(er) to factor into the auto-depend generation.

R.