aboutsummaryrefslogtreecommitdiff
path: root/cse.c
diff options
context:
space:
mode:
authorLinus Torvalds <torvalds@ppc970.osdl.org>2004-11-28 10:40:56 -0700
committerLinus Torvalds <torvalds@ppc970.osdl.org>2005-04-07 21:05:08 -0700
commit64fa258e7391609db6411f6e84edabff5c212b1f (patch)
tree4225e68da8ed47f68f0aa2ed39e457b39587ec4f /cse.c
parentAllow CSE to run after bb packing. (diff)
downloadsparse-64fa258e7391609db6411f6e84edabff5c212b1f.tar.gz
sparse-64fa258e7391609db6411f6e84edabff5c212b1f.tar.bz2
sparse-64fa258e7391609db6411f6e84edabff5c212b1f.zip
Do CSE over children with trivially common parents.
Hey, it's simple, and it happens.
Diffstat (limited to 'cse.c')
-rw-r--r--cse.c22
1 files changed, 21 insertions, 1 deletions
diff --git a/cse.c b/cse.c
index a35177e..cf55985 100644
--- a/cse.c
+++ b/cse.c
@@ -257,9 +257,22 @@ static int bb_dominates(struct entrypoint *ep, struct basic_block *bb1, struct b
return 1;
}
+static inline void remove_instruction(struct instruction_list **list, struct instruction *insn, int count)
+{
+ delete_ptr_list_entry((struct ptr_list **)list, insn, count);
+}
+
+static void add_instruction_to_end(struct instruction *insn, struct basic_block *bb)
+{
+ struct instruction *br = delete_last_instruction(&bb->insns);
+ insn->bb = bb;
+ add_instruction(&bb->insns, insn);
+ add_instruction(&bb->insns, br);
+}
+
static struct instruction * try_to_cse(struct entrypoint *ep, struct instruction *i1, struct instruction *i2)
{
- struct basic_block *b1, *b2;
+ struct basic_block *b1, *b2, *common;
/*
* Ok, i1 and i2 are the same instruction, modulo "target".
@@ -290,6 +303,13 @@ static struct instruction * try_to_cse(struct entrypoint *ep, struct instruction
return cse_one_instruction(i1, i2);
/* No direct dominance - but we could try to find a common ancestor.. */
+ common = trivial_common_parent(b1, VOID, b2, VOID);
+ if (common) {
+ i1 = cse_one_instruction(i2, i1);
+ remove_instruction(&b1->insns, i1, 1);
+ add_instruction_to_end(i1, common);
+ }
+
return i1;
}