[m-rev.] for review: bootcheck using `mmc --make'

[Simon Taylor]

On 20-Jun-2002, Fergus Henderson <fjh at cs.mu.OZ.AU> wrote:
> On 20-Jun-2002, Simon Taylor <stayl at cs.mu.OZ.AU> wrote:
> > 
> > compiler/Mmakefile:
> > tools/bootcheck:
> > tools/binary_step:
> > 	`mmc --make' doesn't handle `<module>_init.c' targets, so
> > 	don't make the `cs' target depend on `top_level_init.c'.
> > 	As far as I can tell, that dependency is only present so
> > 	that the stage2 and stage3 compilers have the same set of
> > 	C files. bootcheck and binary_step now only compare `.c'
> > 	files which are present in the stage3 directory.
> 
> The cs target should include all the .c files.
> 
> If you want a variable or target named "cs_except_for_init_cs",
> that would be OK; but defining the "cs" target so that it doesn?t
> include all the .c files is asking for trouble, IMHO.

The top_level.cs variable doesn't include top_level_init.c,
so my change is consistent with existing practice.

Simon.
--------------------------------------------------------------------------
mercury-reviews mailing list
post:  mercury-reviews at cs.mu.oz.au
administrative address: owner-mercury-reviews at cs.mu.oz.au
unsubscribe: Address: mercury-reviews-request at cs.mu.oz.au Message: unsubscribe
subscribe:   Address: mercury-reviews-request at cs.mu.oz.au Message: subscribe
--------------------------------------------------------------------------