diff options
author | Alyssa Rosenzweig <[email protected]> | 2019-10-26 10:08:18 -0400 |
---|---|---|
committer | Alyssa Rosenzweig <[email protected]> | 2019-11-14 02:36:21 +0000 |
commit | 23c83f3f05d95c9ee519914a5798fd72d880d8bc (patch) | |
tree | b58fbf5d801e4e54ae6e965c1dfc694288364635 | |
parent | eb6352162dde1445f1fe7a408e79f4ecfdf3703c (diff) |
pan/midgard: Compute spill costs
Signed-off-by: Alyssa Rosenzweig <[email protected]>
-rw-r--r-- | src/panfrost/midgard/midgard_schedule.c | 15 |
1 files changed, 13 insertions, 2 deletions
diff --git a/src/panfrost/midgard/midgard_schedule.c b/src/panfrost/midgard/midgard_schedule.c index e5ec8f87225..588f7c6302e 100644 --- a/src/panfrost/midgard/midgard_schedule.c +++ b/src/panfrost/midgard/midgard_schedule.c @@ -1195,10 +1195,21 @@ static void mir_spill_register( * spill node. All nodes are equal in spill cost, but we can't spill * nodes written to from an unspill */ - for (unsigned i = 0; i < ctx->temp_count; ++i) { - lcra_set_node_spill_cost(l, i, 1); + unsigned *cost = calloc(ctx->temp_count, sizeof(cost[0])); + + mir_foreach_instr_global(ctx, ins) { + if (ins->dest < ctx->temp_count) + cost[ins->dest]++; + + mir_foreach_src(ins, s) { + if (ins->src[s] < ctx->temp_count) + cost[ins->src[s]]++; + } } + for (unsigned i = 0; i < ctx->temp_count; ++i) + lcra_set_node_spill_cost(l, i, cost[i]); + /* We can't spill any bundles that contain unspills. This could be * optimized to allow use of r27 to spill twice per bundle, but if * you're at the point of optimizing spilling, it's too late. |