[m-rev.] for review: configure option for LARGE_CONFIG

Julien Fischer juliensf at cs.mu.OZ.AU
Mon Feb 13 17:23:58 AEDT 2006


On Mon, 13 Feb 2006, Mark Brown wrote:

> Estimated hours taken: 1.5
> Branches: main
>
> Add a configure option to enable LARGE_CONFIG when compiling boehm_gc.
> This should probably be made the default, but I'll leave that until after
> the performance impact has been measured.
>
> For future versions of boehm_gc, it might make sense to _not_ define
> LARGE_CONFIG by default.
>
> configure.in:
> 	Subst the variable ENABLE_BOEHM_LARGE_CONFIG with the appropriate
> 	C compiler option.
>
> Mmake.common.in:
> 	Set the BOEHM_CFLAGS variable to the value of the configured variable.
> 	In future, other options may also be specified in BOEHM_CFLAGS (which
> 	is why it has a more generic name).
>
> boehm_gc/Makefile:
> 	Include BOEHM_CFLAGS in the CFLAGS variable, in our version of the
> 	Makefile for this package.
>

That's fine.

Julien.
--------------------------------------------------------------------------
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
--------------------------------------------------------------------------



More information about the reviews mailing list