X-Git-Url: http://nitlanguage.org

diff --git a/src/separate_erasure_compiler.nit b/src/separate_erasure_compiler.nit
index b35842e..d7d88e1 100644
--- a/src/separate_erasure_compiler.nit
+++ b/src/separate_erasure_compiler.nit
@@ -15,40 +15,57 @@
 # Separate compilation of a Nit program with generic type erasure
 module separate_erasure_compiler
 
+intrude import separate_compiler
 
-import separate_compiler
-
+# Add separate erased compiler specific options
 redef class ToolContext
 	# --erasure
 	var opt_erasure: OptionBool = new OptionBool("Erase generic types", "--erasure")
+	# --no-check-erasure-cast
+	var opt_no_check_erasure_cast: OptionBool = new OptionBool("Disable implicit casts on unsafe return with erasure-typing policy (dangerous)", "--no-check-erasure-cast")
 
 	redef init
 	do
 		super
-		self.option_context.add_option(self.opt_erasure)
+		self.option_context.add_option(self.opt_erasure, self.opt_no_check_erasure_cast)
 	end
 end
 
 redef class ModelBuilder
-	fun run_separate_erasure_compiler(mainmodule: MModule, runtime_type_analysis: RapidTypeAnalysis)
+	fun run_separate_erasure_compiler(mainmodule: MModule, runtime_type_analysis: nullable RapidTypeAnalysis)
 	do
 		var time0 = get_time
-		self.toolcontext.info("*** COMPILING TO C ***", 1)
+		self.toolcontext.info("*** GENERATING C ***", 1)
 
-		var compiler = new SeparateErasureCompiler(mainmodule, runtime_type_analysis, self)
+		var compiler = new SeparateErasureCompiler(mainmodule, self, runtime_type_analysis)
+		compiler.compile_header
 
 		# compile class structures
+		self.toolcontext.info("Property coloring", 2)
+		compiler.new_file("{mainmodule.name}.tables")
+		compiler.do_property_coloring
 		for m in mainmodule.in_importation.greaters do
 			for mclass in m.intro_mclasses do
 				compiler.compile_class_to_c(mclass)
 			end
 		end
+		compiler.compile_color_consts(compiler.vt_layout.pos)
+
+		# The main function of the C
+		compiler.new_file("{mainmodule.name}.main")
+		compiler.compile_main_function
 
 		# compile methods
 		for m in mainmodule.in_importation.greaters do
+			self.toolcontext.info("Generate C for module {m}", 2)
+			compiler.new_file("{m.name}.sep")
 			compiler.compile_module_to_c(m)
 		end
 
+		compiler.display_stats
+
+		var time1 = get_time
+		self.toolcontext.info("*** END GENERATING C: {time1-time0} ***", 2)
 		write_and_make(compiler)
 	end
 end
@@ -56,103 +73,153 @@ end
 class SeparateErasureCompiler
 	super SeparateCompiler
 
-	private var class_ids: HashMap[MClass, Int] = new HashMap[MClass, Int]
-	private var class_colors: Map[MClass, Int]
-	private var class_tables: Map[MClass, Array[nullable MClass]]
+	private var class_layout: nullable Layout[MClass]
+	protected var vt_layout: nullable Layout[MVirtualTypeProp]
 
-	init(mainmodule: MModule, runtime_type_analysis: RapidTypeAnalysis, mmbuilder: ModelBuilder) do
-		var mclasses = new HashSet[MClass]
-		mclasses.add_all(mmbuilder.model.mclasses)
+	init(mainmodule: MModule, mmbuilder: ModelBuilder, runtime_type_analysis: nullable RapidTypeAnalysis) do
+		super
 
-		# classes coloration
-		if modelbuilder.toolcontext.opt_use_mod_perfect_hashing.value then
-			# set type unique id
-			for mclass in mclasses do
-				self.class_ids[mclass] = self.class_ids.length + 1
+		var mclasses = new HashSet[MClass].from(mmbuilder.model.mclasses)
+
+		var layout_builder: TypingLayoutBuilder[MClass]
+		var class_colorer = new MClassColorer(mainmodule)
+		if modelbuilder.toolcontext.opt_phmod_typing.value then
+			layout_builder = new MClassHasher(new PHModOperator, mainmodule)
+			class_colorer.build_layout(mclasses)
+		else if modelbuilder.toolcontext.opt_phand_typing.value then
+			layout_builder = new MClassHasher(new PHAndOperator, mainmodule)
+			class_colorer.build_layout(mclasses)
+		else if modelbuilder.toolcontext.opt_bm_typing.value then
+			layout_builder = new MClassBMizer(mainmodule)
+			class_colorer.build_layout(mclasses)
+		else
+			layout_builder = class_colorer
+		end
+		self.class_layout = layout_builder.build_layout(mclasses)
+		self.class_tables = self.build_class_typing_tables(mclasses)
+
+		# lookup vt to build layout with
+		var vts = new HashMap[MClass, Set[MVirtualTypeProp]]
+		for mclass in mclasses do
+			vts[mclass] = new HashSet[MVirtualTypeProp]
+			for mprop in self.mainmodule.properties(mclass) do
+				if mprop isa MVirtualTypeProp then
+					vts[mclass].add(mprop)
+				end
 			end
+		end
 
-			var class_coloring = new ClassModPerfectHashing(mainmodule)
-			self.class_colors = class_coloring.compute_masks(mclasses, class_ids)
-			self.class_tables = class_coloring.hash_type_tables(mclasses, class_ids, class_colors)
-
-			self.header.add_decl("int HASH(int, int);")
-			var v = new_visitor
-			v.add_decl("int HASH(int mask, int id) \{")
-			v.add_decl("return mask % id;")
-			v.add_decl("\}")
-		else if modelbuilder.toolcontext.opt_use_and_perfect_hashing.value then
-			# set type unique id
-			for mclass in mclasses do
-				self.class_ids[mclass] = self.class_ids.length + 1
+		# vt coloration
+		var vt_coloring = new MPropertyColorer[MVirtualTypeProp](mainmodule, class_colorer)
+		var vt_layout = vt_coloring.build_layout(vts)
+		self.vt_tables = build_vt_tables(mclasses, vt_layout)
+		self.vt_layout = vt_layout
+	end
+
+	fun build_vt_tables(mclasses: Set[MClass], layout: Layout[MProperty]): Map[MClass, Array[nullable MPropDef]] do
+		var tables = new HashMap[MClass, Array[nullable MPropDef]]
+		for mclass in mclasses do
+			var table = new Array[nullable MPropDef]
+			# first, fill table from parents by reverse linearization order
+			var parents = new Array[MClass]
+			if mainmodule.flatten_mclass_hierarchy.has(mclass) then
+				parents = mclass.in_hierarchy(mainmodule).greaters.to_a
+				self.mainmodule.linearize_mclasses(parents)
+			end
+			for parent in parents do
+				if parent == mclass then continue
+				for mproperty in self.mainmodule.properties(parent) do
+					if not mproperty isa MVirtualTypeProp then continue
+					var color = layout.pos[mproperty]
+					if table.length <= color then
+						for i in [table.length .. color[ do
+							table[i] = null
+						end
+					end
+					for mpropdef in mproperty.mpropdefs do
+						if mpropdef.mclassdef.mclass == parent then
+							table[color] = mpropdef
+						end
+					end
+				end
 			end
 
-			var class_coloring = new ClassAndPerfectHashing(mainmodule)
-			self.class_colors = class_coloring.compute_masks(mclasses, class_ids)
-			self.class_tables = class_coloring.hash_type_tables(mclasses, class_ids, class_colors)
+			# then override with local properties
+			for mproperty in self.mainmodule.properties(mclass) do
+				if not mproperty isa MVirtualTypeProp then continue
+				var color = layout.pos[mproperty]
+				if table.length <= color then
+					for i in [table.length .. color[ do
+						table[i] = null
+					end
+				end
+				for mpropdef in mproperty.mpropdefs do
+					if mpropdef.mclassdef.mclass == mclass then
+						table[color] = mpropdef
+					end
+				end
+			end
+			tables[mclass] = table
+		end
+		return tables
+	end
 
-			self.header.add_decl("int HASH(int, int);")
-			var v = new_visitor
-			v.add_decl("int HASH(int mask, int id) \{")
-			v.add_decl("return mask & id;")
-			v.add_decl("\}")
-		else
-			var class_coloring
-			if modelbuilder.toolcontext.opt_use_naive_coloring.value then
-				class_coloring = new NaiveClassColoring(mainmodule)
-			else
-				class_coloring = new ClassColoring(mainmodule)
+	# Build class tables
+	fun build_class_typing_tables(mclasses: Set[MClass]): Map[MClass, Array[nullable MClass]] do
+		var tables = new HashMap[MClass, Array[nullable MClass]]
+		var layout = self.class_layout
+		for mclass in mclasses do
+			var table = new Array[nullable MClass]
+			var supers = new Array[MClass]
+			if mainmodule.flatten_mclass_hierarchy.has(mclass) then
+				supers = mclass.in_hierarchy(mainmodule).greaters.to_a
 			end
-			# set type unique id
-			for mclass in mclasses do
-				self.class_ids[mclass] = self.class_ids.length + 1
+			for sup in supers do
+				var color: Int
+				if layout isa PHLayout[MClass, MClass] then
+					color = layout.hashes[mclass][sup]
+				else
+					color = layout.pos[sup]
+				end
+				if table.length <= color then
+					for i in [table.length .. color[ do
+						table[i] = null
+					end
+				end
+				table[color] = sup
 			end
-			self.class_colors = class_coloring.colorize(modelbuilder.model.mclasses)
-			self.class_tables = class_coloring.build_type_tables(modelbuilder.model.mclasses, class_colors)
+			tables[mclass] = table
 		end
-
-		# for the class_name and output_class_name methods
-		self.compile_class_names
+		return tables
 	end
 
 	redef fun compile_header_structs do
 		self.header.add_decl("typedef void(*nitmethod_t)(void); /* general C type representing a Nit method. */")
-		self.header.add_decl("typedef void* nitattribute_t; /* general C type representing a Nit attribute. */")
-		self.header.add_decl("struct class \{ int id; int box_kind; int color; struct vts_table *vts_table; struct type_table *type_table; nitmethod_t vft[1]; \}; /* general C type representing a Nit class. */")
-		self.header.add_decl("struct type_table \{ int size; int table[1]; \}; /* colorized type table. */")
-		self.header.add_decl("struct vts_entry \{ short int is_nullable; struct class *class; \}; /* link (nullable or not) between the vts and is bound. */")
-		self.header.add_decl("struct vts_table \{ struct vts_entry vts[1]; \}; /* vts list of a C type representation. */")
-		self.header.add_decl("typedef struct \{ struct class *class; nitattribute_t attrs[1]; \} val; /* general C type representing a Nit instance. */")
-		self.header.add_decl("extern const char const * class_names[];")
-	end
+		self.compile_header_attribute_structs
+		self.header.add_decl("struct class \{ int id; const char *name; int box_kind; int color; const struct vts_table *vts_table; const struct type_table *type_table; nitmethod_t vft[]; \}; /* general C type representing a Nit class. */")
+		self.header.add_decl("struct type_table \{ int size; int table[]; \}; /* colorized type table. */")
+		self.header.add_decl("struct vts_entry \{ short int is_nullable; const struct class *class; \}; /* link (nullable or not) between the vts and is bound. */")
 
-	redef fun compile_class_names do
-		# Build type names table
-		var type_array = new Array[nullable MClass]
-		for t, id in class_ids do
-			if id >= type_array.length then
-				for i in [type_array.length..id[ do
-					type_array[i] = null
-				end
-			end
-			type_array[id] = t
+		if self.vt_layout isa PHLayout[MClass, MVirtualTypeProp] then
+			self.header.add_decl("struct vts_table \{ int mask; const struct vts_entry vts[]; \}; /* vts list of a C type representation. */")
+		else
+			self.header.add_decl("struct vts_table \{ int dummy; const struct vts_entry vts[]; \}; /* vts list of a C type representation. */")
 		end
 
-		var v = self.new_visitor
-		v.add("const char const * class_names[] = \{")
-		for t in type_array do
-			if t == null then
-				v.add("NULL, /* empty */")
-			else
-				v.add("\"{t}\",")
-			end
+		if modelbuilder.toolcontext.opt_phmod_typing.value then
+			self.header.add_decl("#define HASH(mask, id) ((mask)%(id))")
+		else if modelbuilder.toolcontext.opt_phand_typing.value then
+			self.header.add_decl("#define HASH(mask, id) ((mask)&(id))")
 		end
-		v.add("\};")
+
+		self.header.add_decl("typedef struct instance \{ const struct class *class; nitattribute_t attrs[1]; \} val; /* general C type representing a Nit instance. */")
 	end
 
 	redef fun compile_class_to_c(mclass: MClass)
 	do
 		var mtype = mclass.intro.bound_mtype
 		var c_name = mclass.c_name
+		var c_instance_name = mclass.c_instance_name
 
 		var vft = self.method_tables[mclass]
 		var attrs = self.attr_tables[mclass]
@@ -160,34 +227,28 @@ class SeparateErasureCompiler
 		var v = self.new_visitor
 
 		v.add_decl("/* runtime class {c_name} */")
-		var idnum = classids.length
-		var idname = "ID_" + c_name
-		self.classids[mtype] = idname
-		#self.header.add_decl("#define {idname} {idnum} /* {c_name} */")
-
-		self.header.add_decl("extern const struct class_{c_name} class_{c_name};")
-		self.header.add_decl("struct class_{c_name} \{")
-		self.header.add_decl("int id;")
-		self.header.add_decl("int box_kind;")
-		self.header.add_decl("int color;")
-		self.header.add_decl("const struct vts_table *vts_table;")
-		self.header.add_decl("struct type_table *type_table;")
-		self.header.add_decl("nitmethod_t vft[{vft.length}];")
-		self.header.add_decl("\};")
-
-		# extern const struct vts_table_X vts_table_X
-		self.header.add_decl("extern const struct vts_table_{c_name} vts_table_{c_name};")
-		self.header.add_decl("struct vts_table_{c_name} \{")
-		self.header.add_decl("struct vts_entry vts[{self.vt_tables[mclass].length}];")
-		self.header.add_decl("\};")
+
+		self.provide_declaration("class_{c_name}", "extern const struct class class_{c_name};")
+		v.add_decl("extern const struct type_table type_table_{c_name};")
 
 		# Build class vft
-		v.add_decl("const struct class_{c_name} class_{c_name} = \{")
-		v.add_decl("{self.class_ids[mclass]},")
+		v.add_decl("const struct class class_{c_name} = \{")
+		v.add_decl("{self.class_layout.ids[mclass]},")
+		v.add_decl("\"{mclass.name}\", /* class_name_string */")
 		v.add_decl("{self.box_kind_of(mclass)}, /* box_kind */")
-		v.add_decl("{self.class_colors[mclass]},")
-		v.add_decl("(const struct vts_table*) &vts_table_{c_name},")
-		v.add_decl("(struct type_table*) &type_table_{c_name},")
+		var layout = self.class_layout
+		if layout isa PHLayout[MClass, MClass] then
+			v.add_decl("{layout.masks[mclass]},")
+		else
+			v.add_decl("{layout.pos[mclass]},")
+		end
+		if build_class_vts_table(mclass) then
+			v.require_declaration("vts_table_{c_name}")
+			v.add_decl("&vts_table_{c_name},")
+		else
+			v.add_decl("NULL,")
+		end
+		v.add_decl("&type_table_{c_name},")
 		v.add_decl("\{")
 		for i in [0 .. vft.length[ do
 			var mpropdef = vft[i]
@@ -195,8 +256,10 @@ class SeparateErasureCompiler
 				v.add_decl("NULL, /* empty */")
 			else
 				if true or mpropdef.mclassdef.bound_mtype.ctype != "val*" then
+					v.require_declaration("VIRTUAL_{mpropdef.c_name}")
 					v.add_decl("(nitmethod_t)VIRTUAL_{mpropdef.c_name}, /* pointer to {mclass.intro_mmodule}:{mclass}:{mpropdef} */")
 				else
+					v.require_declaration("{mpropdef.c_name}")
 					v.add_decl("(nitmethod_t){mpropdef.c_name}, /* pointer to {mclass.intro_mmodule}:{mclass}:{mpropdef} */")
 				end
 			end
@@ -204,89 +267,90 @@ class SeparateErasureCompiler
 		v.add_decl("\}")
 		v.add_decl("\};")
 
-		build_class_vts_table(mclass, v.as(SeparateErasureCompilerVisitor))
-
 		# Build class type table
-		self.header.add_decl("extern const struct type_table_{c_name} type_table_{c_name};")
-		self.header.add_decl("struct type_table_{c_name} \{")
-		self.header.add_decl("int size;")
-		self.header.add_decl("int table[{class_table.length}];")
-		self.header.add_decl("\};")
 
-		v.add_decl("const struct type_table_{c_name} type_table_{c_name} = \{")
+		v.add_decl("const struct type_table type_table_{c_name} = \{")
 		v.add_decl("{class_table.length},")
 		v.add_decl("\{")
 		for msuper in class_table do
 			if msuper == null then
 				v.add_decl("-1, /* empty */")
 			else
-				v.add_decl("{self.class_ids[msuper]}, /* {msuper} */")
+				v.add_decl("{self.class_layout.ids[msuper]}, /* {msuper} */")
 			end
 		end
 		v.add_decl("\}")
 		v.add_decl("\};")
 
-		#Build instance struct
 		if mtype.ctype != "val*" then
-			self.header.add_decl("struct instance_{c_name} \{")
-			self.header.add_decl("const struct class *class;")
-			self.header.add_decl("{mtype.ctype} value;")
-			self.header.add_decl("\};")
+			if mtype.mclass.name == "Pointer" or mtype.mclass.kind != extern_kind then
+				#Build instance struct
+				self.header.add_decl("struct instance_{c_instance_name} \{")
+				self.header.add_decl("const struct class *class;")
+				self.header.add_decl("{mtype.ctype} value;")
+				self.header.add_decl("\};")
+			end
 
-			self.header.add_decl("val* BOX_{c_name}({mtype.ctype});")
+			#Build BOX
+			self.provide_declaration("BOX_{c_name}", "val* BOX_{c_name}({mtype.ctype});")
 			v.add_decl("/* allocate {mtype} */")
 			v.add_decl("val* BOX_{mtype.c_name}({mtype.ctype} value) \{")
-			v.add("struct instance_{c_name}*res = GC_MALLOC(sizeof(struct instance_{c_name}));")
-			v.add("res->class = (struct class*) &class_{c_name};")
+			v.add("struct instance_{c_instance_name}*res = nit_alloc(sizeof(struct instance_{c_instance_name}));")
+			v.require_declaration("class_{c_name}")
+			v.add("res->class = &class_{c_name};")
 			v.add("res->value = value;")
 			v.add("return (val*)res;")
 			v.add("\}")
 			return
-		end
-
-		var is_native_array = mclass.name == "NativeArray"
-
-		var sig
-		if is_native_array then
-			sig = "int length"
-		else
-			sig = ""
-		end
+		else if mclass.name == "NativeArray" then
+			#Build instance struct
+			self.header.add_decl("struct instance_{c_name} \{")
+			self.header.add_decl("const struct class *class;")
+			self.header.add_decl("val* values[];")
+			self.header.add_decl("\};")
 
-		#Build instance struct
-		#extern const struct instance_array__NativeArray instance_array__NativeArray;
-		self.header.add_decl("struct instance_{c_name} \{")
-		self.header.add_decl("const struct class *class;")
-		self.header.add_decl("nitattribute_t attrs[{attrs.length}];")
-		if is_native_array then
-			# NativeArrays are just a instance header followed by an array of values
-			self.header.add_decl("val* values[0];")
+			#Build NEW
+			self.provide_declaration("NEW_{c_name}", "{mtype.ctype} NEW_{c_name}(int length);")
+			v.add_decl("/* allocate {mtype} */")
+			v.add_decl("{mtype.ctype} NEW_{c_name}(int length) \{")
+			var res = v.new_named_var(mtype, "self")
+			res.is_exact = true
+			var mtype_elt = mtype.arguments.first
+			v.add("{res} = nit_alloc(sizeof(struct instance_{c_name}) + length*sizeof({mtype_elt.ctype}));")
+			v.require_declaration("class_{c_name}")
+			v.add("{res}->class = &class_{c_name};")
+			v.add("return {res};")
+			v.add("\}")
+			return
 		end
-		self.header.add_decl("\};")
-
 
-		self.header.add_decl("{mtype.ctype} NEW_{c_name}({sig});")
+		#Build NEW
+		self.provide_declaration("NEW_{c_name}", "{mtype.ctype} NEW_{c_name}(void);")
 		v.add_decl("/* allocate {mtype} */")
-		v.add_decl("{mtype.ctype} NEW_{c_name}({sig}) \{")
+		v.add_decl("{mtype.ctype} NEW_{c_name}(void) \{")
 		var res = v.new_named_var(mtype, "self")
 		res.is_exact = true
-		if is_native_array then
-			var mtype_elt = mtype.arguments.first
-			v.add("{res} = GC_MALLOC(sizeof(struct instance_{c_name}) + length*sizeof({mtype_elt.ctype}));")
-		else
-			v.add("{res} = GC_MALLOC(sizeof(struct instance_{c_name}));")
-		end
-		v.add("{res}->class = (struct class*) &class_{c_name};")
-
+		v.add("{res} = nit_alloc(sizeof(struct instance) + {attrs.length}*sizeof(nitattribute_t));")
+		v.require_declaration("class_{c_name}")
+		v.add("{res}->class = &class_{c_name};")
 		self.generate_init_attr(v, res, mtype)
 		v.add("return {res};")
 		v.add("\}")
-
-		generate_check_init_instance(mtype)
 	end
 
-	private fun build_class_vts_table(mclass: MClass, v: SeparateCompilerVisitor) do
-		v.add_decl("const struct vts_table_{mclass.c_name} vts_table_{mclass.c_name} = \{")
+	private fun build_class_vts_table(mclass: MClass): Bool do
+		if self.vt_tables[mclass].is_empty then return false
+
+		self.provide_declaration("vts_table_{mclass.c_name}", "extern const struct vts_table vts_table_{mclass.c_name};")
+
+		var v = new_visitor
+		v.add_decl("const struct vts_table vts_table_{mclass.c_name} = \{")
+		if self.vt_layout isa PHLayout[MClass, MVirtualTypeProp] then
+			#TODO redo this when PHPropertyLayoutBuilder will be implemented
+			#v.add_decl("{vt_masks[mclass]},")
+		else
+			v.add_decl("0, /* dummy */")
+		end
 		v.add_decl("\{")
 
 		for vt in self.vt_tables[mclass] do
@@ -294,16 +358,19 @@ class SeparateErasureCompiler
 				v.add_decl("\{-1, NULL\}, /* empty */")
 			else
 				var is_null = 0
-				var bound = retrieve_vt_bound(mclass.intro.bound_mtype, vt.bound)
+				var bound = retrieve_vt_bound(mclass.intro.bound_mtype, vt.as(MVirtualTypeDef).bound)
 				while bound isa MNullableType do
 					bound = retrieve_vt_bound(mclass.intro.bound_mtype, bound.mtype)
 					is_null = 1
 				end
-				v.add_decl("\{{is_null}, (struct class*)&class_{bound.as(MClassType).mclass.c_name}\}, /* {vt} */")
+				var vtclass = bound.as(MClassType).mclass
+				v.require_declaration("class_{vtclass.c_name}")
+				v.add_decl("\{{is_null}, &class_{vtclass.c_name}\}, /* {vt} */")
 			end
 		end
 		v.add_decl("\},")
 		v.add_decl("\};")
+		return true
 	end
 
 	private fun retrieve_vt_bound(anchor: MClassType, mtype: nullable MType): MType do
@@ -321,17 +388,85 @@ class SeparateErasureCompiler
 	end
 
 	redef fun new_visitor do return new SeparateErasureCompilerVisitor(self)
+
+	# Stats
+
+	private var class_tables: Map[MClass, Array[nullable MClass]]
+	private var vt_tables: Map[MClass, Array[nullable MPropDef]]
+
+	redef fun display_sizes
+	do
+		print "# size of subtyping tables"
+		print "\ttotal \tholes"
+		var total = 0
+		var holes = 0
+		for t, table in class_tables do
+			total += table.length
+			for e in table do if e == null then holes += 1
+		end
+		print "\t{total}\t{holes}"
+
+		print "# size of resolution tables"
+		print "\ttotal \tholes"
+		total = 0
+		holes = 0
+		for t, table in vt_tables do
+			total += table.length
+			for e in table do if e == null then holes += 1
+		end
+		print "\t{total}\t{holes}"
+
+		print "# size of methods tables"
+		print "\ttotal \tholes"
+		total = 0
+		holes = 0
+		for t, table in method_tables do
+			total += table.length
+			for e in table do if e == null then holes += 1
+		end
+		print "\t{total}\t{holes}"
+
+		print "# size of attributes tables"
+		print "\ttotal \tholes"
+		total = 0
+		holes = 0
+		for t, table in attr_tables do
+			total += table.length
+			for e in table do if e == null then holes += 1
+		end
+		print "\t{total}\t{holes}"
+	end
 end
 
 class SeparateErasureCompilerVisitor
 	super SeparateCompilerVisitor
 
+	redef fun compile_callsite(callsite, arguments)
+	do
+		var res = super
+		if callsite.erasure_cast and not self.compiler.as(SeparateErasureCompiler).modelbuilder.toolcontext.opt_no_check_erasure_cast.value then
+			assert res != null
+			var mtype = callsite.msignature.return_mtype
+			assert mtype != null
+			self.add("/* Erasure cast for return {res} isa {mtype} */")
+			var cond = self.type_test(res, mtype, "erasure")
+			self.add("if (!{cond}) \{")
+			#var x = self.class_name_string(res)
+			#var y = self.class_name_string(arguments.first)
+			#self.add("fprintf(stderr, \"Erasure cast: expected {mtype} (self is %s), got %s for {res}\\n\", {y}, {x});")
+			self.add_abort("Cast failed")
+			self.add("\}")
+		end
+		return res
+	end
+
 	redef fun init_instance(mtype)
 	do
+		self.require_declaration("NEW_{mtype.mclass.c_name}")
 		return self.new_expr("NEW_{mtype.mclass.c_name}()", mtype)
 	end
 
-	redef fun type_test(value, mtype)
+	redef fun type_test(value, mtype, tag)
 	do
 		self.add("/* type test for {value.inspect} isa {mtype} */")
 
@@ -341,37 +476,28 @@ class SeparateErasureCompilerVisitor
 		self.add_decl("int {cltype};")
 		var idtype = self.get_name("idtype")
 		self.add_decl("int {idtype};")
-		var is_nullable = self.get_name("is_nullable")
-		self.add_decl("short int {is_nullable};")
-		var is_null = self.get_name("is_null")
-		self.add_decl("short int {is_null};")
 
-		var maybe_null = 0
+		var maybe_null = self.maybe_null(value)
+		var accept_null = "0"
 		if mtype isa MNullableType then
 			mtype = mtype.mtype
-			maybe_null = 1
-			self.add("{is_nullable} = 1;")
+			accept_null = "1"
 		end
 		if mtype isa MParameterType then
 			# Here we get the bound of the the formal type (eh, erasure...)
 			mtype = mtype.resolve_for(self.frame.mpropdef.mclassdef.bound_mtype, self.frame.mpropdef.mclassdef.bound_mtype, self.frame.mpropdef.mclassdef.mmodule, false)
 			if mtype isa MNullableType then
 				mtype = mtype.mtype
-				maybe_null = 1
-				self.add("{is_nullable} = 1;")
+				accept_null = "1"
 			end
 		end
-		if mtype isa MVirtualType then
-			# FIXME virtual types should not be erased but got from the class table of the current receiver (self.frame.arguments.first)
-			#mtype = mtype.resolve_for(self.frame.mpropdef.mclassdef.bound_mtype, self.frame.mpropdef.mclassdef.bound_mtype, self.frame.mpropdef.mclassdef.mmodule, true)
-			#if mtype isa MNullableType then
-			#	mtype = mtype.mtype
-			#	maybe_null = true
-			#end
-		end
 
 		if value.mcasttype.is_subtype(self.frame.mpropdef.mclassdef.mmodule, self.frame.mpropdef.mclassdef.bound_mtype, mtype) then
 			self.add("{res} = 1; /* easy {value.inspect} isa {mtype}*/")
+			if compiler.modelbuilder.toolcontext.opt_typing_test_metrics.value then
+				self.compiler.count_type_test_skipped[tag] += 1
+				self.add("count_type_test_skipped_{tag}++;")
+			end
 			return res
 		end
 
@@ -379,26 +505,49 @@ class SeparateErasureCompilerVisitor
 		var type_table
 		if value.mtype.ctype == "val*" then
 			class_ptr = "{value}->class->"
-			self.add("{is_null} = {value} == NULL;")
 		else
 			var mclass = value.mtype.as(MClassType).mclass
+			self.require_declaration("class_{mclass.c_name}")
 			class_ptr = "class_{mclass.c_name}."
-			self.add("{is_null} = 0;")
 		end
 
 		if mtype isa MClassType then
+			self.require_declaration("class_{mtype.mclass.c_name}")
 			self.add("{cltype} = class_{mtype.mclass.c_name}.color;")
 			self.add("{idtype} = class_{mtype.mclass.c_name}.id;")
-			if maybe_null == 0 then
-				self.add("{is_nullable} = 0;")
+			if compiler.modelbuilder.toolcontext.opt_typing_test_metrics.value then
+				self.compiler.count_type_test_resolved[tag] += 1
+				self.add("count_type_test_resolved_{tag}++;")
 			end
 		else if mtype isa MVirtualType then
 			var recv = self.frame.arguments.first
-			var recv_boxed = self.autobox(recv, self.object_type)
-			self.add("{cltype} = {recv_boxed}->class->vts_table->vts[{mtype.mproperty.const_color}].class->color;")
-			self.add("{idtype} = {recv_boxed}->class->vts_table->vts[{mtype.mproperty.const_color}].class->id;")
-			if maybe_null == 0 then
-				self.add("{is_nullable} = {recv_boxed}->class->vts_table->vts[{mtype.mproperty.const_color}].is_nullable;")
+			var recv_ptr
+			if recv.mtype.ctype == "val*" then
+				recv_ptr = "{recv}->class->"
+			else
+				var mclass = recv.mtype.as(MClassType).mclass
+				self.require_declaration("class_{mclass.c_name}")
+				recv_ptr = "class_{mclass.c_name}."
+			end
+			var entry = self.get_name("entry")
+			self.add("struct vts_entry {entry};")
+			self.require_declaration(mtype.mproperty.const_color)
+			if self.compiler.as(SeparateErasureCompiler).vt_layout isa PHLayout[MClass, MVirtualTypeProp] then
+				self.add("{entry} = {recv_ptr}vts_table->vts[HASH({recv_ptr}vts_table->mask, {mtype.mproperty.const_color})];")
+			else
+				self.add("{entry} = {recv_ptr}vts_table->vts[{mtype.mproperty.const_color}];")
+			end
+			self.add("{cltype} = {entry}.class->color;")
+			self.add("{idtype} = {entry}.class->id;")
+			if maybe_null and accept_null == "0" then
+				var is_nullable = self.get_name("is_nullable")
+				self.add_decl("short int {is_nullable};")
+				self.add("{is_nullable} = {entry}.is_nullable;")
+				accept_null = is_nullable.to_s
+			end
+			if compiler.modelbuilder.toolcontext.opt_typing_test_metrics.value then
+				self.compiler.count_type_test_unresolved[tag] += 1
+				self.add("count_type_test_unresolved_{tag}++;")
 			end
 		else
 			self.debug("type_test({value.inspect}, {mtype})")
@@ -406,10 +555,12 @@ class SeparateErasureCompilerVisitor
 		end
 
 		# check color is in table
-		self.add("if({is_null}) \{")
-		self.add("{res} = {is_nullable};")
-		self.add("\} else \{")
-		if compiler.modelbuilder.toolcontext.opt_use_mod_perfect_hashing.value or compiler.modelbuilder.toolcontext.opt_use_and_perfect_hashing.value then
+		if maybe_null then
+			self.add("if({value} == NULL) \{")
+			self.add("{res} = {accept_null};")
+			self.add("\} else \{")
+		end
+		if self.compiler.as(SeparateErasureCompiler).class_layout isa PHLayout[MClass, MClass] then
 			self.add("{cltype} = HASH({class_ptr}color, {idtype});")
 		end
 		self.add("if({cltype} >= {class_ptr}type_table->size) \{")
@@ -417,7 +568,9 @@ class SeparateErasureCompilerVisitor
 		self.add("\} else \{")
 		self.add("{res} = {class_ptr}type_table->table[{cltype}] == {idtype};")
 		self.add("\}")
-		self.add("\}")
+		if maybe_null then
+			self.add("\}")
+		end
 
 		return res
 	end
@@ -425,8 +578,13 @@ class SeparateErasureCompilerVisitor
 	redef fun class_name_string(value)
 	do
 		var res = self.get_name("var_class_name")
-		self.add_decl("const char *{res};")
-		self.add("{res} = class_names[{value}->class->id];")
+		self.add_decl("const char* {res};")
+		if value.mtype.ctype == "val*" then
+			self.add "{res} = {value} == NULL ? \"null\" : {value}->class->name;"
+		else
+			self.require_declaration("class_{value.mtype.c_name}")
+			self.add "{res} = class_{value.mtype.c_name}.name;"
+		end
 		return res
 	end
 
@@ -439,14 +597,14 @@ class SeparateErasureCompilerVisitor
 		self.add("\{ /* {res} = array_instance Array[{elttype}] */")
 		var nat = self.new_var(self.get_class("NativeArray").get_mtype([elttype]))
 		nat.is_exact = true
+		self.require_declaration("NEW_{nclass.c_name}")
 		self.add("{nat} = NEW_{nclass.c_name}({array.length});")
 		for i in [0..array.length[ do
 			var r = self.autobox(array[i], self.object_type)
-			self.add("((struct instance_{nclass.c_name}*){nat})->values[{i}] = (val*) {r};")
+			self.add("((struct instance_{nclass.c_instance_name}*){nat})->values[{i}] = (val*) {r};")
 		end
 		var length = self.int_instance(array.length)
 		self.send(self.get_property("with_native", arraytype), [res, nat, length])
-		self.check_init_instance(res, arraytype)
 		self.add("\}")
 		return res
 	end
@@ -454,6 +612,7 @@ class SeparateErasureCompilerVisitor
 	redef fun calloc_array(ret_type, arguments)
 	do
 		var ret = ret_type.as(MClassType)
+		self.require_declaration("NEW_{ret.mclass.c_name}")
 		self.ret(self.new_expr("NEW_{ret.mclass.c_name}({arguments[1]})", ret_type))
 	end
 end