On 10/04/2012 04:07 PM, Eric Anholt wrote:
For tracking live variables, we want to know when a register is completely
rewritten, so we need to be able to compare a writemask to the size of the
register. There's also potential use for this in register coalescing.
Two comments:
1. Please move the last hunk from patch 11 into this patch:
diff --git a/src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp
b/src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp
index 1dfdcce..9482d47 100644
--- a/src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp
+++ b/src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp
@@ -2695,6 +2695,7 @@ vec4_visitor::vec4_visitor(struct brw_vs_compile *c,
this->virtual_grf_def = NULL;
this->virtual_grf_use = NULL;
this->virtual_grf_sizes = NULL;
+ this->virtual_grf_chans = NULL;
this->virtual_grf_count = 0;
this->virtual_grf_reg_map = NULL;
this->virtual_grf_reg_count = 0;
2. I don't see any patches in your series which use this.
The tracking itself looks fine, but without any consumers of this data,
it's hard to tell whether this is the most sensible way to do it.
So you get a
Reviewed-by: Kenneth Graunke <kenn...@whitecape.org>
but I might hold off on pushing it until you have a patch that actually
uses it.
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
