[MLton] Re: [MLton-commit] r4683

Matthew Fluet fluet@cs.cornell.edu
Tue, 18 Jul 2006 21:21:49 -0400 (EDT)


> Fixed several "cast increases required alignment" warnings that were
> showing up on Solaris.
>
> Modified: mlton/branches/on-20050822-x86_64-branch/runtime/gc/generational.h
> ===================================================================
> --- mlton/branches/on-20050822-x86_64-branch/runtime/gc/generational.h	2006-07-13 21:31:51 UTC (rev 4682)
> +++ mlton/branches/on-20050822-x86_64-branch/runtime/gc/generational.h	2006-07-17 17:27:13 UTC (rev 4683)
> @@ -14,6 +14,12 @@
>
> typedef uint8_t GC_cardMapElem;
> typedef uint8_t GC_crossMapElem;
> +
> +typedef uint8_t GC_cardMapStart __attribute__ ((aligned (4)));
> +typedef uint8_t GC_crossMapStart __attribute__ ((aligned (4)));
> +typedef GC_cardMapStart *GC_cardMap;
> +typedef GC_crossMapStart *GC_crossMap;
> +

Would

typdef GC_cardMapElem GC_cardMapStart __attribute__ ((aligned (4)));
typdef GC_crossMapElem GC_crossMapStart __attribute__ ((aligned (4)));

accomplish the same thing?  If so, it is a little better documentation of 
the relationship.