Simplify backends by removing unnecessary constructs:
[libfirm] / ir / be / benode.c
index ae98f6c..788b09c 100644 (file)
@@ -127,10 +127,7 @@ ir_op *op_be_AddSP;
 ir_op *op_be_SubSP;
 ir_op *op_be_SetSP;
 ir_op *op_be_RegParams;
-ir_op *op_be_StackParam;
 ir_op *op_be_FrameAddr;
-ir_op *op_be_FrameLoad;
-ir_op *op_be_FrameStore;
 ir_op *op_be_Barrier;
 
 static int beo_base = -1;
@@ -277,9 +274,6 @@ void be_node_init(void) {
        op_be_IncSP      = new_ir_op(beo_base + beo_IncSP,      "be_IncSP",      op_pin_state_pinned,     N, oparity_unary,    0, sizeof(be_stack_attr_t),   &be_node_op_ops);
        op_be_SetSP      = new_ir_op(beo_base + beo_SetSP,      "be_SetSP",      op_pin_state_pinned,     N, oparity_binary,   0, sizeof(be_stack_attr_t),   &be_node_op_ops);
        op_be_RegParams  = new_ir_op(beo_base + beo_RegParams,  "be_RegParams",  op_pin_state_pinned,     N, oparity_zero,     0, sizeof(be_node_attr_t),    &be_node_op_ops);
-       op_be_StackParam = new_ir_op(beo_base + beo_StackParam, "be_StackParam", op_pin_state_floats,     N, oparity_unary,    0, sizeof(be_frame_attr_t),   &be_node_op_ops);
-       op_be_FrameLoad  = new_ir_op(beo_base + beo_FrameLoad,  "be_FrameLoad",  op_pin_state_floats,     N, oparity_any,      0, sizeof(be_frame_attr_t),   &be_node_op_ops);
-       op_be_FrameStore = new_ir_op(beo_base + beo_FrameStore, "be_FrameStore", op_pin_state_floats,     N, oparity_any,      0, sizeof(be_frame_attr_t),   &be_node_op_ops);
        op_be_FrameAddr  = new_ir_op(beo_base + beo_FrameAddr,  "be_FrameAddr",  op_pin_state_floats,     N, oparity_unary,    0, sizeof(be_frame_attr_t),   &be_node_op_ops);
        op_be_Barrier    = new_ir_op(beo_base + beo_Barrier,    "be_Barrier",    op_pin_state_pinned,     N, oparity_dynamic,  0, sizeof(be_node_attr_t),    &be_node_op_ops);
 
@@ -311,12 +305,6 @@ void be_node_init(void) {
        op_be_IncSP->ops.node_cmp_attr = Stack_cmp_attr;
        set_op_tag(op_be_RegParams,  &be_node_tag);
        op_be_RegParams->ops.node_cmp_attr = node_cmp_attr;
-       set_op_tag(op_be_StackParam, &be_node_tag);
-       op_be_StackParam->ops.node_cmp_attr = FrameAddr_cmp_attr;
-       set_op_tag(op_be_FrameLoad,  &be_node_tag);
-       op_be_FrameLoad->ops.node_cmp_attr = FrameAddr_cmp_attr;
-       set_op_tag(op_be_FrameStore, &be_node_tag);
-       op_be_FrameStore->ops.node_cmp_attr = FrameAddr_cmp_attr;
        set_op_tag(op_be_FrameAddr,  &be_node_tag);
        op_be_FrameAddr->ops.node_cmp_attr = FrameAddr_cmp_attr;
        set_op_tag(op_be_Barrier,    &be_node_tag);
@@ -491,7 +479,7 @@ void be_Perm_reduce(ir_node *perm, int new_size, int *map)
 {
        ir_graph *irg           = get_irn_irg(perm);
        int n                   = get_irn_arity(perm);
-       be_reg_data_t *old_data = malloc(n * sizeof(old_data[0]));
+       be_reg_data_t *old_data = xmalloc(n * sizeof(old_data[0]));
        be_node_attr_t *attr    = get_irn_attr(perm);
        ir_node **new_in        = NEW_ARR_D(ir_node *, irg->obst, new_size + 1);
 
@@ -720,6 +708,7 @@ ir_node *be_new_AddSP(const arch_register_t *sp, ir_graph *irg, ir_node *bl, ir_
        be_node_attr_t *a;
        ir_node *irn;
        ir_node *in[be_pos_AddSP_last];
+       const arch_register_class_t *class;
 
        in[be_pos_AddSP_old_sp] = old_sp;
        in[be_pos_AddSP_size]   = sz;
@@ -727,13 +716,17 @@ ir_node *be_new_AddSP(const arch_register_t *sp, ir_graph *irg, ir_node *bl, ir_
        irn = new_ir_node(NULL, irg, bl, op_be_AddSP, mode_T, be_pos_AddSP_last, in);
        a   = init_node_attr(irn, be_pos_AddSP_last);
 
-       be_node_set_flags(irn, OUT_POS(pn_be_AddSP_res), arch_irn_flags_ignore | arch_irn_flags_modify_sp);
+       be_node_set_flags(irn, OUT_POS(pn_be_AddSP_sp),
+                         arch_irn_flags_ignore | arch_irn_flags_modify_sp);
 
        /* Set output constraint to stack register. */
        be_set_constr_single_reg(irn, be_pos_AddSP_old_sp, sp);
        be_node_set_reg_class(irn, be_pos_AddSP_size, arch_register_get_class(sp));
-       be_set_constr_single_reg(irn, OUT_POS(pn_be_AddSP_res), sp);
-       a->reg_data[pn_be_AddSP_res].reg = sp;
+       be_set_constr_single_reg(irn, OUT_POS(pn_be_AddSP_sp), sp);
+       a->reg_data[pn_be_AddSP_sp].reg = sp;
+
+       class = arch_register_get_class(sp);
+       be_node_set_reg_class(irn, OUT_POS(pn_be_AddSP_res), class);
 
        return irn;
 }
@@ -750,28 +743,30 @@ ir_node *be_new_SubSP(const arch_register_t *sp, ir_graph *irg, ir_node *bl, ir_
        irn = new_ir_node(NULL, irg, bl, op_be_SubSP, mode_T, be_pos_SubSP_last, in);
        a   = init_node_attr(irn, be_pos_SubSP_last);
 
-       be_node_set_flags(irn, OUT_POS(pn_be_SubSP_res), arch_irn_flags_ignore | arch_irn_flags_modify_sp);
+       be_node_set_flags(irn, OUT_POS(pn_be_SubSP_sp),
+                         arch_irn_flags_ignore | arch_irn_flags_modify_sp);
 
        /* Set output constraint to stack register. */
        be_set_constr_single_reg(irn, be_pos_SubSP_old_sp, sp);
        be_node_set_reg_class(irn, be_pos_SubSP_size, arch_register_get_class(sp));
-       be_set_constr_single_reg(irn, OUT_POS(pn_be_SubSP_res), sp);
-       a->reg_data[pn_be_SubSP_res].reg = sp;
+       be_set_constr_single_reg(irn, OUT_POS(pn_be_SubSP_sp), sp);
+       a->reg_data[pn_be_SubSP_sp].reg = sp;
 
        return irn;
 }
 
-ir_node *be_new_SetSP(const arch_register_t *sp, ir_graph *irg, ir_node *bl, ir_node *old_sp, ir_node *op, ir_node *mem)
+ir_node *be_new_SetSP(const arch_register_t *sp, ir_graph *irg, ir_node *bl,
+                      ir_node *old_sp, ir_node *op, ir_node *mem)
 {
        be_node_attr_t *a;
        ir_node *irn;
        ir_node *in[3];
 
-       in[0]    = mem;
-       in[1]    = old_sp;
-       in[2]    = op;
-       irn      = new_ir_node(NULL, irg, bl, op_be_SetSP, get_irn_mode(old_sp), 3, in);
-       a        = init_node_attr(irn, 3);
+       in[0] = mem;
+       in[1] = old_sp;
+       in[2] = op;
+       irn   = new_ir_node(NULL, irg, bl, op_be_SetSP, get_irn_mode(old_sp), 3, in);
+       a     = init_node_attr(irn, 3);
 
        be_node_set_flags(irn, OUT_POS(0), arch_irn_flags_ignore | arch_irn_flags_modify_sp);
 
@@ -783,24 +778,6 @@ ir_node *be_new_SetSP(const arch_register_t *sp, ir_graph *irg, ir_node *bl, ir_
        return irn;
 }
 
-ir_node *be_new_StackParam(const arch_register_class_t *cls, const arch_register_class_t *cls_frame, ir_graph *irg, ir_node *bl, ir_mode *mode, ir_node *frame_pointer, ir_entity *ent)
-{
-       be_frame_attr_t *a;
-       ir_node *irn;
-       ir_node *in[1];
-
-       in[0] = frame_pointer;
-       irn = new_ir_node(NULL, irg, bl, op_be_StackParam, mode, 1, in);
-       a = init_node_attr(irn, 1);
-       a->ent = ent;
-
-       be_node_set_reg_class(irn, 0, cls_frame);
-       be_node_set_reg_class(irn, OUT_POS(0), cls);
-       be_node_set_flags(irn, OUT_POS(0), arch_irn_flags_rematerializable);
-
-       return irn;
-}
-
 ir_node *be_new_RegParams(ir_graph *irg, ir_node *bl, int n_outs)
 {
        ir_node *res;
@@ -837,43 +814,6 @@ ir_node *be_RegParams_append_out_reg(ir_node *regparams,
        return proj;
 }
 
-ir_node *be_new_FrameLoad(const arch_register_class_t *cls_frame, const arch_register_class_t *cls_data,
-                                                 ir_graph *irg, ir_node *bl, ir_node *mem, ir_node *frame, ir_entity *ent)
-{
-       be_frame_attr_t *a;
-       ir_node *irn;
-       ir_node *in[2];
-
-       in[0]  = mem;
-       in[1]  = frame;
-       irn    = new_ir_node(NULL, irg, bl, op_be_FrameLoad, mode_T, 2, in);
-       a      = init_node_attr(irn, 3);
-       a->ent = ent;
-       a->offset = 0;
-       be_node_set_reg_class(irn, 1, cls_frame);
-       be_node_set_reg_class(irn, OUT_POS(pn_Load_res), cls_data);
-       return irn;
-}
-
-ir_node *be_new_FrameStore(const arch_register_class_t *cls_frame, const arch_register_class_t *cls_data,
-                                                  ir_graph *irg, ir_node *bl, ir_node *mem, ir_node *frame, ir_node *data, ir_entity *ent)
-{
-       be_frame_attr_t *a;
-       ir_node *irn;
-       ir_node *in[3];
-
-       in[0]  = mem;
-       in[1]  = frame;
-       in[2]  = data;
-       irn    = new_ir_node(NULL, irg, bl, op_be_FrameStore, mode_T, 3, in);
-       a      = init_node_attr(irn, 3);
-       a->ent = ent;
-       a->offset = 0;
-       be_node_set_reg_class(irn, 1, cls_frame);
-       be_node_set_reg_class(irn, 2, cls_data);
-       return irn;
-}
-
 ir_node *be_new_FrameAddr(const arch_register_class_t *cls_frame, ir_graph *irg, ir_node *bl, ir_node *frame, ir_entity *ent)
 {
        be_frame_attr_t *a;
@@ -969,20 +909,14 @@ int be_is_SetSP         (const ir_node *irn) { return be_get_irn_opcode(irn) ==
 int be_is_AddSP         (const ir_node *irn) { return be_get_irn_opcode(irn) == beo_AddSP          ; }
 int be_is_SubSP         (const ir_node *irn) { return be_get_irn_opcode(irn) == beo_SubSP          ; }
 int be_is_RegParams     (const ir_node *irn) { return be_get_irn_opcode(irn) == beo_RegParams      ; }
-int be_is_StackParam    (const ir_node *irn) { return be_get_irn_opcode(irn) == beo_StackParam     ; }
 int be_is_FrameAddr     (const ir_node *irn) { return be_get_irn_opcode(irn) == beo_FrameAddr      ; }
-int be_is_FrameLoad     (const ir_node *irn) { return be_get_irn_opcode(irn) == beo_FrameLoad      ; }
-int be_is_FrameStore    (const ir_node *irn) { return be_get_irn_opcode(irn) == beo_FrameStore     ; }
 int be_is_Barrier       (const ir_node *irn) { return be_get_irn_opcode(irn) == beo_Barrier        ; }
 
 int be_has_frame_entity(const ir_node *irn)
 {
        switch(be_get_irn_opcode(irn)) {
-       case beo_StackParam:
        case beo_Spill:
        case beo_Reload:
-       case beo_FrameStore:
-       case beo_FrameLoad:
        case beo_FrameAddr:
                return 1;
        default:
@@ -1261,7 +1195,6 @@ static arch_irn_class_t be_node_classify(const void *self, const ir_node *irn)
                XXX(Perm, perm);
                XXX(Copy, copy);
                XXX(Return, branch);
-               XXX(StackParam, stackparam);
 #undef XXX
                default:
                return arch_irn_class_normal;
@@ -1605,8 +1538,9 @@ arch_irn_handler_t *be_phi_handler_new(const arch_env_t *arch_env)
 
 void be_phi_handler_free(arch_irn_handler_t *handler)
 {
-       phi_handler_t *h = (void *) handler;
+       phi_handler_t *h = get_phi_handler_from_handler(handler);
        pmap_destroy(h->phi_attrs);
+       h->phi_attrs = NULL;
        free(handler);
 }
 
@@ -1696,6 +1630,11 @@ static int dump_node(ir_node *irn, FILE *f, dump_reason_t reason)
                        fprintf(f, get_mode_name(get_irn_mode(irn)));
                        break;
                case dump_node_nodeattr_txt:
+                       if(be_is_Call(irn)) {
+                               be_call_attr_t *a = (be_call_attr_t *) at;
+                               if (a->ent)
+                                       fprintf(f, " [%s] ", get_entity_name(a->ent));
+                       }
                        break;
                case dump_node_info_txt:
                        dump_node_reqs(f, irn);