From: Manuel Mohr <manuel.mohr@kit.edu>
Date: Tue, 27 Sep 2011 12:36:35 +0000 (+0200)
Subject: sparc:  Warn if trying to emit stack relative Lds/Sts with small offsets.
X-Git-Url: http://nsz.repo.hu/git/?a=commitdiff_plain;h=a1589472df187a1de0fc4dd1c2a793accc91e423;p=libfirm

sparc:  Warn if trying to emit stack relative Lds/Sts with small offsets.
---

diff --git a/ir/be/sparc/sparc_emitter.c b/ir/be/sparc/sparc_emitter.c
index 36768f815..fe97b7055 100644
--- a/ir/be/sparc/sparc_emitter.c
+++ b/ir/be/sparc/sparc_emitter.c
@@ -166,6 +166,25 @@ void sparc_emit_offset(const ir_node *node, int offset_node_pos)
 	}
 }
 
+void sparc_emit_source_reg_and_offset(const ir_node *node, int regpos,
+                                      int offpos)
+{
+	const arch_register_t *reg = arch_get_irn_register_in(node, regpos);
+	const sparc_load_store_attr_t *attr;
+
+	if (reg == &sparc_registers[REG_SP]) {
+		attr = get_sparc_load_store_attr_const(node);
+		if (!attr->is_reg_reg
+		    && attr->base.immediate_value < SPARC_MIN_STACKSIZE) {
+
+			ir_fprintf(stderr, "warning: emitting stack pointer relative load/store with offset < %d\n", SPARC_MIN_STACKSIZE);
+		}
+	}
+
+	sparc_emit_source_register(node, regpos);
+	sparc_emit_offset(node, offpos);
+}
+
 void sparc_emit_float_load_store_mode(const ir_node *node)
 {
 	const sparc_load_store_attr_t *attr = get_sparc_load_store_attr_const(node);
@@ -179,7 +198,7 @@ void sparc_emit_float_load_store_mode(const ir_node *node)
 	case 64:  be_emit_char('d'); return;
 	case 128: be_emit_char('q'); return;
 	}
-	panic("invalid flaot load/store mode %+F", mode);
+	panic("invalid float load/store mode %+F", mode);
 }
 
 /**
diff --git a/ir/be/sparc/sparc_emitter.h b/ir/be/sparc/sparc_emitter.h
index 08aad3166..015314730 100644
--- a/ir/be/sparc/sparc_emitter.h
+++ b/ir/be/sparc/sparc_emitter.h
@@ -45,6 +45,8 @@ void sparc_emit_offset(const ir_node *node, int offset_node_pos);
 void sparc_emit_load_mode(const ir_node *node);
 void sparc_emit_store_mode(const ir_node *node);
 void sparc_emit_float_load_store_mode(const ir_node *node);
+void sparc_emit_source_reg_and_offset(const ir_node *node, int regpos,
+                                      int offpos);
 void sparc_emit_fp_mode_suffix(const ir_node *node);
 void sparc_emit_fp_conv_source(const ir_node *node);
 void sparc_emit_fp_conv_destination(const ir_node *node);
diff --git a/ir/be/sparc/sparc_spec.pl b/ir/be/sparc/sparc_spec.pl
index da1594629..e7a5f516d 100644
--- a/ir/be/sparc/sparc_spec.pl
+++ b/ir/be/sparc/sparc_spec.pl
@@ -116,6 +116,8 @@ $mode_fp4     = "mode_E"; # not correct, we need to register a new mode
 	FCONVD => "${arch}_emit_fp_conv_destination(node);",
 	O1     => "${arch}_emit_offset(node, 1);",
 	O2     => "${arch}_emit_offset(node, 2);",
+	S0O1   => "${arch}_emit_source_reg_and_offset(node, 0, 1);",
+	S1O2   => "${arch}_emit_source_reg_and_offset(node, 1, 2);",
 );
 
 $default_attr_type = "sparc_attr_t";
@@ -348,7 +350,7 @@ Ld => {
 	ins       => [ "ptr", "mem" ],
 	outs      => [ "res", "M" ],
 	attr_type => "sparc_load_store_attr_t",
-	emit      => '. ld%LM [%S0%O1], %D0'
+	emit      => '. ld%LM [%S0O1], %D0'
 },
 
 SetHi => {
@@ -382,7 +384,7 @@ St => {
 	ins       => [ "val", "ptr", "mem" ],
 	outs      => [ "M" ],
 	attr_type => "sparc_load_store_attr_t",
-	emit      => '. st%SM %S0, [%S1%O2]'
+	emit      => '. st%SM %S0, [%S1O2]'
 },
 
 Save => {