don't change size for spillslots because of align.
[libfirm] / ir / tr / type.c
index 2a11023..9ef7daa 100644 (file)
@@ -21,7 +21,6 @@
  * @file    type.c
  * @brief   Representation of types.
  * @author  Goetz Lindenmaier, Michael Beck
- * @version $Id$
  * @brief
  *
  *  Implementation of the datastructure to hold
@@ -56,7 +55,7 @@
 #include "tpop_t.h"
 #include "tv_t.h"
 #include "irhooks.h"
-#include "irtools.h"
+#include "util.h"
 #include "entity_t.h"
 #include "error.h"
 #include "dbginfo.h"
@@ -87,16 +86,13 @@ void ir_init_type(void)
        firm_none_type = new_type(tpop_none, mode_BAD, NULL);
        set_type_size_bytes(firm_none_type, 0);
        set_type_state (firm_none_type, layout_fixed);
-       remove_irp_type(firm_none_type);
 
        firm_code_type = new_type(tpop_code, mode_ANY, NULL);
        set_type_state(firm_code_type, layout_fixed);
-       remove_irp_type(firm_code_type);
 
        firm_unknown_type = new_type(tpop_unknown, mode_ANY, NULL);
        set_type_size_bytes(firm_unknown_type, 0);
        set_type_state (firm_unknown_type, layout_fixed);
-       remove_irp_type(firm_unknown_type);
 }
 
 void ir_finish_type(void)
@@ -152,7 +148,6 @@ ir_type *new_type(const tp_op *type_op, ir_mode *mode, type_dbg_info *db)
        res->visit      = 0;
        res->link       = NULL;
        res->dbi        = db;
-       res->assoc_type = NULL;
 #ifdef DEBUG_libfirm
        res->nr         = get_irp_new_node_nr();
 #endif /* defined DEBUG_libfirm */
@@ -1142,34 +1137,6 @@ void set_struct_size(ir_type *tp, unsigned size)
        tp->size = size;
 }
 
-
-/**
- * Lazy construction of value argument / result representation.
- * Constructs a struct type and its member.  The types of the members
- * are passed in the argument list.
- *
- * @param name    name of the type constructed
- * @param len     number of fields
- * @param tps     array of field types with length len
- */
-static ir_type *build_value_type(char const* name, size_t len, tp_ent_pair *tps)
-{
-       size_t i;
-       ir_type *res = new_type_struct(new_id_from_str(name));
-       res->flags |= tf_value_param_type;
-       /* Remove type from type list.  Must be treated differently than other types. */
-       remove_irp_type(res);
-       for (i = 0; i < len; ++i) {
-               ident *id = new_id_from_str("elt");
-
-               /* use res as default if corresponding type is not yet set. */
-               ir_type *elt_type = tps[i].tp ? tps[i].tp : res;
-               tps[i].ent = new_entity(res, id, elt_type);
-               set_entity_allocation(tps[i].ent, allocation_parameter);
-       }
-       return res;
-}
-
 ir_type *new_d_type_method(size_t n_param, size_t n_res, type_dbg_info *db)
 {
        ir_type *res;
@@ -1180,7 +1147,6 @@ ir_type *new_d_type_method(size_t n_param, size_t n_res, type_dbg_info *db)
        res->size                         = get_mode_size_bytes(mode_P_code);
        res->attr.ma.n_params             = n_param;
        res->attr.ma.params               = XMALLOCNZ(tp_ent_pair, n_param);
-       res->attr.ma.value_params         = NULL;
        res->attr.ma.n_res                = n_res;
        res->attr.ma.res_type             = XMALLOCNZ(tp_ent_pair, n_res);
        res->attr.ma.variadicity          = variadicity_non_variadic;
@@ -1212,12 +1178,11 @@ ir_type *clone_type_method(ir_type *tp)
        res = new_type(type_method, mode, db);
 
        res->flags                         = tp->flags;
-       res->assoc_type                    = tp->assoc_type;
+       res->higher_type                   = tp->higher_type;
        res->size                          = tp->size;
        res->attr.ma.n_params              = n_params;
        res->attr.ma.params                = XMALLOCN(tp_ent_pair, n_params);
        memcpy(res->attr.ma.params, tp->attr.ma.params, n_params * sizeof(res->attr.ma.params[0]));
-       res->attr.ma.value_params          = tp->attr.ma.value_params;
        res->attr.ma.n_res                 = n_res;
        res->attr.ma.res_type              = XMALLOCN(tp_ent_pair, n_res);
        memcpy(res->attr.ma.res_type, tp->attr.ma.res_type, n_res * sizeof(res->attr.ma.res_type[0]));
@@ -1239,12 +1204,6 @@ void free_method_attrs(ir_type *method)
        assert(method && (method->type_op == type_method));
        free(method->attr.ma.params);
        free(method->attr.ma.res_type);
-       /* cannot free it yet, type could be cloned ...
-       if (method->attr.ma.value_params) {
-               free_type_entities(method->attr.ma.value_params);
-               free_type(method->attr.ma.value_params);
-       }
-       */
 }
 
 size_t (get_method_n_params)(const ir_type *method)
@@ -1252,7 +1211,7 @@ size_t (get_method_n_params)(const ir_type *method)
        return _get_method_n_params(method);
 }
 
-ir_type *get_method_param_type(ir_type *method, size_t pos)
+ir_type *get_method_param_type(const ir_type *method, size_t pos)
 {
        ir_type *res;
        assert(method->type_op == type_method);
@@ -1267,54 +1226,6 @@ void set_method_param_type(ir_type *method, size_t pos, ir_type *tp)
        assert(method->type_op == type_method);
        assert(pos < get_method_n_params(method));
        method->attr.ma.params[pos].tp = tp;
-       /* If information constructed set pass-by-value representation. */
-       if (method->attr.ma.value_params) {
-               assert(get_method_n_params(method) == get_struct_n_members(method->attr.ma.value_params));
-               set_entity_type(get_struct_member(method->attr.ma.value_params, pos), tp);
-       }
-}
-
-ir_entity *get_method_value_param_ent(ir_type *method, size_t pos)
-{
-       assert(method && (method->type_op == type_method));
-       assert(pos < get_method_n_params(method));
-
-       if (!method->attr.ma.value_params) {
-               /* parameter value type not created yet, build */
-               method->attr.ma.value_params = build_value_type("<value param>",
-                       get_method_n_params(method), method->attr.ma.params);
-       }
-       /*
-        * build_value_type() sets the method->attr.ma.value_params type as default if
-        * no type is set!
-        */
-       assert((get_entity_type(method->attr.ma.params[pos].ent) != method->attr.ma.value_params)
-              && "param type not yet set");
-       return method->attr.ma.params[pos].ent;
-}
-
-void set_method_value_param_type(ir_type *method, ir_type *tp)
-{
-       size_t i;
-       size_t n;
-
-       assert(method && (method->type_op == type_method));
-       assert(is_value_param_type(tp));
-       assert(get_method_n_params(method) == get_struct_n_members(tp));
-
-       method->attr.ma.value_params = tp;
-
-       n = get_struct_n_members(tp);
-       for (i = 0; i < n; i++) {
-               ir_entity *ent = get_struct_member(tp, i);
-               method->attr.ma.params[i].ent = ent;
-       }
-}
-
-ir_type *get_method_value_param_type(const ir_type *method)
-{
-       assert(method && (method->type_op == type_method));
-       return method->attr.ma.value_params;
 }
 
 size_t (get_method_n_ress)(const ir_type *method)
@@ -1322,7 +1233,7 @@ size_t (get_method_n_ress)(const ir_type *method)
        return _get_method_n_ress(method);
 }
 
-ir_type *get_method_res_type(ir_type *method, size_t pos)
+ir_type *get_method_res_type(const ir_type *method, size_t pos)
 {
        ir_type *res;
        assert(method->type_op == type_method);
@@ -2040,32 +1951,12 @@ int is_frame_type(const ir_type *tp)
        return tp->flags & tf_frame_type;
 }
 
-int is_value_param_type(const ir_type *tp)
-{
-       return tp->flags & tf_value_param_type;
-}
-
-ir_type *new_type_value(void)
-{
-       ir_type *res = new_type_struct(new_id_from_str("<value_type>"));
-
-       res->flags |= tf_value_param_type;
-
-       /* Remove type from type list.  Must be treated differently than other types. */
-       remove_irp_type(res);
-
-       return res;
-}
-
 ir_type *new_type_frame(void)
 {
        ir_type *res = new_type_class(new_id_from_str("<frame_type>"));
 
        res->flags |= tf_frame_type;
 
-       /* Remove type from type list.  Must be treated differently than other types. */
-       remove_irp_type(res);
-
        /* It is not possible to derive from the frame type. Set the final flag. */
        set_class_final(res, 1);
 
@@ -2079,7 +1970,7 @@ ir_type *clone_frame_type(ir_type *type)
 
        assert(is_frame_type(type));
        /* the entity link resource should be allocated if this function is called */
-       assert(irp_resources_reserved(irp) & IR_RESOURCE_ENTITY_LINK);
+       assert(irp_resources_reserved(irp) & IRP_RESOURCE_ENTITY_LINK);
 
        res = new_type_frame();
        for (i = 0, n = get_class_n_members(type); i < n; ++i) {
@@ -2140,7 +2031,6 @@ ir_entity *frame_alloc_area(ir_type *frame_type, int size, unsigned alignment,
        ir_type *tp;
        ident *name;
        char buf[32];
-       unsigned frame_align;
        int offset, frame_size;
        static unsigned area_cnt = 0;
        static ir_type *a_byte = NULL;
@@ -2156,26 +2046,25 @@ ir_entity *frame_alloc_area(ir_type *frame_type, int size, unsigned alignment,
        snprintf(buf, sizeof(buf), "area%u", area_cnt++);
        name = new_id_from_str(buf);
 
-       /* align the size */
-       frame_align = get_type_alignment_bytes(frame_type);
-       size = (size + frame_align - 1) & ~(frame_align - 1);
-
        tp = new_type_array(1, a_byte);
        set_array_bounds_int(tp, 0, 0, size);
        set_type_alignment_bytes(tp, alignment);
+       set_type_size_bytes(tp, size);
 
        frame_size = get_type_size_bytes(frame_type);
        if (at_start) {
                size_t i, n;
+               unsigned frame_align = get_type_alignment_bytes(frame_type);
+               unsigned delta = (size + frame_align - 1) & ~(frame_align - 1);
                /* fix all offsets so far */
                for (i = 0, n = get_class_n_members(frame_type); i < n; ++i) {
                        ir_entity *ent = get_class_member(frame_type, i);
 
-                       set_entity_offset(ent, get_entity_offset(ent) + size);
+                       set_entity_offset(ent, get_entity_offset(ent) + delta);
                }
                /* calculate offset and new type size */
                offset = 0;
-               frame_size += size;
+               frame_size += delta;
 
                /* increase size to match alignment... */
                if (alignment > frame_align) {