[m-rev.] diff: improve mmake support for high level data on IL backend

Tyson Dowd trd at cs.mu.OZ.AU
Wed Jul 18 20:05:21 AEST 2001


On 18-Jul-2001, Peter Ross <peter.ross at miscrit.be> wrote:
> Hi,
> 
> 
> ===================================================================
> 
> 
> Estimated hours taken: 2
> Branches: main
> 
> Improve the mmake support in preperation for high level data on the IL
> backend.
> 
> compiler/modules.m:
>     Ensure that all the mercury modules a foreign code module might
>     refer to are built before the foreign code module.
>     Add .foreign_dlls to the main build target to ensure that all the
>     foreign code dlls get built.
> 
> compiler/mlds_to_mcpp.m:
>     Fix an XXX about high level data.
> 

>  		]),
>  
> -		% If we are on the IL backend and the current module
> -		% contains some foreign code, generate a dependency
> -		% between the dll containing the mercury code and the
> -		% dll containing the foreign code.  Also generate
> -		% dependencies between the foreign code dll and how to
> -		% it relates to the il file (dll -> cpp -> il).
> +		% Generate the following dependency.  This dependency is
> +		% needed because module__cpp_code.dll might refer to
> +		% high level data in any of the mercury modules it
> +		% imports plus itself.
> +		%
> +		% 	module__cpp_code.dll : module.dll imports.dll

This should be $(imports.dll) or something to show that it is in fact a
variable that is the list of imported dlls for this module.

Otherwise this seems ok.

-- 
       Tyson Dowd           # 
                            #  Surreal humour isn't everyone's cup of fur.
     trd at cs.mu.oz.au        # 
http://www.cs.mu.oz.au/~trd #
--------------------------------------------------------------------------
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