summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMatt Turner <mattst88@gmail.com>2014-04-09 14:23:07 -0700
committerMatt Turner <mattst88@gmail.com>2014-04-18 09:16:19 -0700
commit04a4e43eb22c7b78c0b9ab4e6b722747b5c4ce27 (patch)
tree9295a2935ebf776711e81e8ea5ef0400de9e31b4
parenta975b2f55cd40a0ca53b72a17b05c0de08254f6b (diff)
i965/fs: Unindent can_coalesce_vars().
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
-rw-r--r--src/mesa/drivers/dri/i965/brw_fs_register_coalesce.cpp55
1 files changed, 28 insertions, 27 deletions
diff --git a/src/mesa/drivers/dri/i965/brw_fs_register_coalesce.cpp b/src/mesa/drivers/dri/i965/brw_fs_register_coalesce.cpp
index 6a8842d07d0..01b672f36dd 100644
--- a/src/mesa/drivers/dri/i965/brw_fs_register_coalesce.cpp
+++ b/src/mesa/drivers/dri/i965/brw_fs_register_coalesce.cpp
@@ -80,39 +80,40 @@ can_coalesce_vars(brw::fs_live_variables *live_intervals,
const exec_list *instructions, const fs_inst *inst,
int var_to, int var_from)
{
- if (live_intervals->vars_interfere(var_from, var_to)) {
- /* We know that the live ranges of A (var_from) and B (var_to)
- * interfere because of the ->vars_interfere() call above. If the end
- * of B's live range is after the end of A's range, then we know two
- * things:
- * - the start of B's live range must be in A's live range (since we
- * already know the two ranges interfere, this is the only remaining
- * possibility)
- * - the interference isn't of the form we're looking for (where B is
- * entirely inside A)
- */
- if (live_intervals->end[var_to] > live_intervals->end[var_from])
- return false;
+ if (!live_intervals->vars_interfere(var_from, var_to))
+ return true;
+
+ /* We know that the live ranges of A (var_from) and B (var_to)
+ * interfere because of the ->vars_interfere() call above. If the end
+ * of B's live range is after the end of A's range, then we know two
+ * things:
+ * - the start of B's live range must be in A's live range (since we
+ * already know the two ranges interfere, this is the only remaining
+ * possibility)
+ * - the interference isn't of the form we're looking for (where B is
+ * entirely inside A)
+ */
+ if (live_intervals->end[var_to] > live_intervals->end[var_from])
+ return false;
- int scan_ip = -1;
+ int scan_ip = -1;
- foreach_list(n, instructions) {
- fs_inst *scan_inst = (fs_inst *)n;
- scan_ip++;
+ foreach_list(n, instructions) {
+ fs_inst *scan_inst = (fs_inst *)n;
+ scan_ip++;
- if (scan_inst->is_control_flow())
- return false;
+ if (scan_inst->is_control_flow())
+ return false;
- if (scan_ip <= live_intervals->start[var_to])
- continue;
+ if (scan_ip <= live_intervals->start[var_to])
+ continue;
- if (scan_ip > live_intervals->end[var_to])
- break;
+ if (scan_ip > live_intervals->end[var_to])
+ break;
- if (scan_inst->dst.equals(inst->dst) ||
- scan_inst->dst.equals(inst->src[0]))
- return false;
- }
+ if (scan_inst->dst.equals(inst->dst) ||
+ scan_inst->dst.equals(inst->src[0]))
+ return false;
}
return true;