[MLton] commit r4899: Announcing merge of x86_64 branch revisions into trunk

Vesa Karvonen vesa.karvonen at cs.helsinki.fi
Mon Dec 4 04:15:25 PST 2006


I just compiled MLton on MinGW and ran bin/regression (*).  The main issue
I noticed that there was a (presumably harmless) warning while compiling the
runtime:

gcc -std=gnu99 -I. -Iplatform  -O2 -fomit-frame-pointer -falign-loops=2 -falign-jumps=2 -falign-functions=5  -pedantic -Wall -W
-Wformat=2 -Wno-format-nonliteral -Wuninitialized -Wfloat-equal -Wundef -Wshadow -Wpointer-arith -Wbad-function-cast -Wcast-qual
-Wcast-align -Wwrite-strings -Waggregate-return -Wstrict-prototypes -Wmissing-prototypes -Wmissing-declarations
-Wmissing-noreturn -Wmissing-format-attribute -Wpacked -Wredundant-decls -Wnested-externs -Wlong-long  -Wdisabled-optimization 
-Winline -c -o util.o util.c
In file included from ./util.h:39,
                 from util/assert.c:8,
                 from util.c:8:
./util/read_write.h: In function `writeUintmaxU':
./util/read_write.h:64: warning: ISO C does not support the `I64' printf length modifier
./util/read_write.h: In function `writeUintmaxX':
./util/read_write.h:79: warning: ISO C does not support the `I64' printf length modifier
./util/read_write.h:81: warning: ISO C does not support the `I64' printf length modifier
./util/read_write.h:83: warning: ISO C does not support the `I64' printf length modifier

As util.h seems to be included from almost everywhere, the above warning
gets repeated many times.  The GCC version is "3.4.5 (mingw special)".

(I have a full log of compiling (MLton repository revision) 4902 on 4902,
but the above error is the only thing that I noticed.)

(*) See the patch below that I made to the regression script.  Unless there
is some reason to prefer the explicit /c/cygwin/bin/sed pathname, I'd like
to commit the patch, because otherwise the script just doesn't work at all
under MSYS ("the MinGW shell") without Cygwin.  The leading $ (in $'s/$/\r/')
specifies C escape sequences.  There were also some other lesser (I presume)
script problems (weird whitespace diffs and trouble finding some executables
during the last few tests).  Unless someone else does it first, I'll
investigate those the next time I build MLton MinGW.

-Vesa Karvonen

Index: bin/regression
===================================================================
--- bin/regression	(revision 4902)
+++ bin/regression	(working copy)
@@ -215,7 +215,7 @@
                         fi
 			if $forMinGW; then
                                 compare="$f.sed.ok"
-                                /c/cygwin/bin/sed 's/$/\r/' <"$f.ok" >"$compare"
+                                sed $'s/$/\r/' <"$f.ok" >"$compare"
 			fi
                         if ! diff "$compare" "$tmp"; then
                                 echo "difference with ${flags[*]} ${extraFlags[*]}"



More information about the MLton mailing list