[m-rev.] for review: work towards building the compiler in the java grade

Peter Wang novalazy at gmail.com
Wed Jan 19 10:45:21 AEDT 2011


On 2011-01-19, Julien Fischer <juliensf at csse.unimelb.edu.au> wrote:
> 
> For review by Peter Wang.
> 
> Note that there other problems with compiler/Mmakefile that prevent the
> compiler from building in the java grade -- I commented out the problems
> when I tested this, but since that would break the C grades, it
> isn't a fix -- I will address them separately.
> 
> The other problem I encountered in the compiler directory is that
> when building the .class files, the runtime classes cannot be found;
> presumably this is just a matter of passing --java-classpath to mmc
> and pointing it back at the library directory?

Yes, probably similar to what is in SSDB_FLAGS.

> Index: compiler/Mmakefile
> ===================================================================
> RCS file: /home/mercury/mercury1/repository/mercury/compiler/Mmakefile,v
> retrieving revision 1.107
> diff -u -r1.107 Mmakefile
> --- compiler/Mmakefile	30 Sep 2010 07:23:31 -0000	1.107
> +++ compiler/Mmakefile	18 Jan 2011 15:12:24 -0000
> @@ -88,6 +88,14 @@
>  MCFLAGS += --linkage static
>  endif
> 
> +# -Xmx256m doesn't always seem to be enough memory to build the complier.
> +# This bumps up the memory when building the standard library

the compiler

> +# if the javac executable accepts the -J-Xmx flag, without bumping up
> +# the memory requirements in general.
> +ifneq ("$(findstring -J-Xmx,$(JAVACFLAGS))","")
> +JAVACFLAGS   += -J-Xmx2048m
> +endif
> +
>  #-----------------------------------------------------------------------------#
> 
>  # Rules for preprocessing `.pp' files.

> Index: compiler/md4.m
> ===================================================================
> RCS file: /home/mercury/mercury1/repository/mercury/compiler/md4.m,v
> retrieving revision 1.3
> diff -u -r1.3 md4.m
> --- compiler/md4.m	2 Jan 2011 14:37:54 -0000	1.3
> +++ compiler/md4.m	18 Jan 2011 15:12:24 -0000
> @@ -41,6 +41,8 @@
> 
>  :- implementation.
> 
> +:- import_module require.
> +

Is this used?

Peter
--------------------------------------------------------------------------
mercury-reviews mailing list
Post messages to:       mercury-reviews at csse.unimelb.edu.au
Administrative Queries: owner-mercury-reviews at csse.unimelb.edu.au
Subscriptions:          mercury-reviews-request at csse.unimelb.edu.au
--------------------------------------------------------------------------



More information about the reviews mailing list