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

[Fergus Henderson]

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.

-- 
Fergus Henderson <fjh at cs.mu.oz.au>  |  "I have always known that the pursuit
The University of Melbourne         |  of excellence is a lethal habit"
WWW: <http://www.cs.mu.oz.au/~fjh>  |     -- the last words of T. S. Garp.
--------------------------------------------------------------------------
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
--------------------------------------------------------------------------