fdo-mirrors/mesa
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/mesa/mesa.git synced 2026-05-08 02:38:04 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 4

i965: Drop pointless check for variable declarations in splitting.

Browse source 
We're walking the whole instruction stream, so we know the declaration
will be found.

Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
This commit is contained in:
Eric Anholt 2014-03-25 12:44:01 -07:00
parent 66b15ad9db
commit cf40ebacb1
1 changed files with 5 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
src/mesa/drivers/dri/i965/brw_fs_vector_splitting.cpp
View file

@ -55,7 +55,6 @@ public:
{
this->var = var;
this->whole_vector_access = 0;
this->declaration = false;
this->mem_ctx = NULL;
}
@ -64,8 +63,6 @@ public:
/** Number of times the variable is referenced, including assignments. */
unsigned whole_vector_access;
bool declaration; /* If the variable had a decl in the instruction stream */
ir_variable *components[4];
/** ralloc_parent(this->var) -- the shader's ralloc context. */
@ -139,10 +136,8 @@ ir_vector_reference_visitor::get_variable_entry(ir_variable *var)
ir_visitor_status
ir_vector_reference_visitor::visit(ir_variable *ir)
{
variable_entry *entry = this->get_variable_entry(ir);
if (entry)
entry->declaration = true;
/* Make sure splitting looks at splitting this variable */
(void)this->get_variable_entry(ir);
return visit_continue;
}
@ -347,12 +342,12 @@ brw_do_vector_splitting(exec_list *instructions)
variable_entry *entry = (variable_entry *)node;
if (debug) {
fprintf(stderr, "vector %s@%p: decl %d, whole_access %d\n",
entry->var->name, (void *) entry->var, entry->declaration,
fprintf(stderr, "vector %s@%p: whole_access %d\n",
entry->var->name, (void *) entry->var,
entry->whole_vector_access);
}
if (!entry->declaration || entry->whole_vector_access) {
if (entry->whole_vector_access) {
entry->remove();
}
}