[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