OpenHarmony/third_party_mesa3d
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

glsl: Make accessor functions for ir_variable::interface_type.

Browse Source 
In a future patch, this will allow us to enforce invariants when the
interface type is updated.

Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>
This commit is contained in:
Paul Berry
2013-09-24 14:30:29 -07:00
parent 6f19e552af
commit 22d3ef2df1
9 changed files with 51 additions and 33 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/glsl/ast_to_hir.cpp
View File

@@ -4725,7 +4725,7 @@ ast_interface_block::hir(exec_list *instructions,
var_mode);
}
var->interface_type = block_type;
var->init_interface_type(block_type);
if (state->target == geometry_shader && var_mode == ir_var_shader_in)
handle_geometry_shader_input_decl(state, loc, var);
state->symbols->add_variable(var);
@@ -4741,7 +4741,7 @@ ast_interface_block::hir(exec_list *instructions,
new(state) ir_variable(fields[i].type,
ralloc_strdup(state, fields[i].name),
var_mode);
var->interface_type = block_type;
var->init_interface_type(block_type);
/* Propagate the "binding" keyword into this UBO's fields;
* the UBO declaration itself doesn't get an ir_variable unless it

2
src/glsl/builtin_variables.cpp
View File

@@ -831,7 +831,7 @@ builtin_variable_generator::generate_varyings()
"gl_in");
ir_variable *var = add_variable("gl_in", array(per_vertex_type, 0),
ir_var_shader_in, 0);
var->interface_type = per_vertex_type;
var->init_interface_type(per_vertex_type);
}
}

15
src/glsl/ir.h
View File

@@ -391,6 +391,20 @@ public:
|| (t->is_array() && t->fields.array == this->interface_type);
}
/**
* Set this->interface_type on a newly created variable.
*/
void init_interface_type(const struct glsl_type *type)
{
assert(this->interface_type == NULL);
this->interface_type = type;
}
const glsl_type *get_interface_type() const
{
return this->interface_type;
}
/**
* Declared type of the variable
*/
@@ -582,6 +596,7 @@ public:
*/
ir_constant *constant_initializer;
private:
/**
* For variables that are in an interface block or are an instance of an
* interface block, this is the \c GLSL_TYPE_INTERFACE type for that block.

17
src/glsl/link_interface_blocks.cpp
View File

@@ -47,7 +47,7 @@ validate_intrastage_interface_blocks(struct gl_shader_program *prog,
if (!var)
continue;
const glsl_type *iface_type = var->interface_type;
const glsl_type *iface_type = var->get_interface_type();
if (iface_type == NULL)
continue;
@@ -81,32 +81,33 @@ validate_interstage_interface_blocks(struct gl_shader_program *prog,
/* Add non-output interfaces from the consumer to the symbol table. */
foreach_list(node, consumer->ir) {
ir_variable *var = ((ir_instruction *) node)->as_variable();
if (!var || !var->interface_type || var->mode == ir_var_shader_out)
if (!var || !var->get_interface_type() || var->mode == ir_var_shader_out)
continue;
interfaces.add_interface(var->interface_type->name,
var->interface_type,
interfaces.add_interface(var->get_interface_type()->name,
var->get_interface_type(),
(enum ir_variable_mode) var->mode);
}
/* Verify that the producer's interfaces match. */
foreach_list(node, producer->ir) {
ir_variable *var = ((ir_instruction *) node)->as_variable();
if (!var || !var->interface_type || var->mode == ir_var_shader_in)
if (!var || !var->get_interface_type() || var->mode == ir_var_shader_in)
continue;
enum ir_variable_mode consumer_mode =
var->mode == ir_var_uniform ? ir_var_uniform : ir_var_shader_in;
const glsl_type *expected_type =
interfaces.get_interface(var->interface_type->name, consumer_mode);
interfaces.get_interface(var->get_interface_type()->name,
consumer_mode);
/* The consumer doesn't use this output block. Ignore it. */
if (expected_type == NULL)
continue;
if (var->interface_type != expected_type) {
if (var->get_interface_type() != expected_type) {
linker_error(prog, "definitions of interface block `%s' do not "
"match\n", var->interface_type->name);
"match\n", var->get_interface_type()->name);
return;
}
}

12
src/glsl/link_uniform_block_active_visitor.cpp
View File

@@ -27,12 +27,12 @@
link_uniform_block_active *
process_block(void *mem_ctx, struct hash_table *ht, ir_variable *var)
{
const uint32_t h = _mesa_hash_string(var->interface_type->name);
const uint32_t h = _mesa_hash_string(var->get_interface_type()->name);
const hash_entry *const existing_block =
_mesa_hash_table_search(ht, h, var->interface_type->name);
_mesa_hash_table_search(ht, h, var->get_interface_type()->name);
const glsl_type *const block_type = var->is_interface_instance()
? var->type : var->interface_type;
? var->type : var->get_interface_type();
/* If a block with this block-name has not previously been seen, add it.
@@ -46,7 +46,7 @@ process_block(void *mem_ctx, struct hash_table *ht, ir_variable *var)
b->type = block_type;
b->has_instance_name = var->is_interface_instance();
_mesa_hash_table_insert(ht, h, var->interface_type->name,
_mesa_hash_table_insert(ht, h, var->get_interface_type()->name,
(void *) b);
return b;
} else {
@@ -90,7 +90,7 @@ link_uniform_block_active_visitor::visit_enter(ir_dereference_array *ir)
if (b == NULL) {
linker_error(prog,
"uniform block `%s' has mismatching definitions",
var->interface_type->name);
var->get_interface_type()->name);
this->success = false;
return visit_stop;
}
@@ -149,7 +149,7 @@ link_uniform_block_active_visitor::visit(ir_dereference_variable *ir)
if (b == NULL) {
linker_error(this->prog,
"uniform block `%s' has mismatching definitions",
var->interface_type->name);
var->get_interface_type()->name);
this->success = false;
return visit_stop;
}

13
src/glsl/link_uniforms.cpp
View File

@@ -199,8 +199,8 @@ public:
{
this->is_ubo_var = var->is_in_uniform_block();
if (var->is_interface_instance())
program_resource_visitor::process(var->interface_type,
var->interface_type->name);
program_resource_visitor::process(var->get_interface_type(),
var->get_interface_type()->name);
else
program_resource_visitor::process(var);
}
@@ -317,10 +317,10 @@ public:
ubo_block_index = -1;
if (var->is_in_uniform_block()) {
if (var->is_interface_instance() && var->type->is_array()) {
unsigned l = strlen(var->interface_type->name);
unsigned l = strlen(var->get_interface_type()->name);
for (unsigned i = 0; i < prog->NumUniformBlocks; i++) {
if (strncmp(var->interface_type->name,
if (strncmp(var->get_interface_type()->name,
prog->UniformBlocks[i].Name,
l) == 0
&& prog->UniformBlocks[i].Name[l] == '[') {
@@ -330,7 +330,7 @@ public:
}
} else {
for (unsigned i = 0; i < prog->NumUniformBlocks; i++) {
if (strcmp(var->interface_type->name,
if (strcmp(var->get_interface_type()->name,
prog->UniformBlocks[i].Name) == 0) {
ubo_block_index = i;
break;
@@ -362,7 +362,8 @@ public:
}
if (var->is_interface_instance())
process(var->interface_type, var->interface_type->name);
process(var->get_interface_type(),
var->get_interface_type()->name);
else
process(var);
} else

12
src/glsl/link_varyings.cpp
View File

@@ -972,8 +972,8 @@ public:
this->toplevel_var = var;
this->varying_floats = 0;
if (var->is_interface_instance())
program_resource_visitor::process(var->interface_type,
var->interface_type->name);
program_resource_visitor::process(var->get_interface_type(),
var->get_interface_type()->name);
else
program_resource_visitor::process(var);
}
@@ -1083,10 +1083,10 @@ assign_varying_locations(struct gl_context *ctx,
((ir_instruction *) node)->as_variable();
if ((input_var != NULL) && (input_var->mode == ir_var_shader_in)) {
if (input_var->interface_type != NULL) {
if (input_var->get_interface_type() != NULL) {
char *const iface_field_name =
ralloc_asprintf(mem_ctx, "%s.%s",
input_var->interface_type->name,
input_var->get_interface_type()->name,
input_var->name);
hash_table_insert(consumer_interface_inputs, input_var,
iface_field_name);
@@ -1108,10 +1108,10 @@ assign_varying_locations(struct gl_context *ctx,
g.process(output_var);
ir_variable *input_var;
if (output_var->interface_type != NULL) {
if (output_var->get_interface_type() != NULL) {
char *const iface_field_name =
ralloc_asprintf(mem_ctx, "%s.%s",
output_var->interface_type->name,
output_var->get_interface_type()->name,
output_var->name);
input_var =
(ir_variable *) hash_table_find(consumer_interface_inputs,

6
src/glsl/lower_named_interface_blocks.cpp
View File

@@ -152,7 +152,7 @@ flatten_named_interface_blocks_declarations::run(exec_list *instructions)
}
new_var->location = iface_t->fields.structure[i].location;
new_var->interface_type = iface_t;
new_var->init_interface_type(iface_t);
hash_table_insert(interface_namespace, new_var,
iface_field_name);
insert_pos->insert_after(new_var);
@@ -208,9 +208,9 @@ flatten_named_interface_blocks_declarations::handle_rvalue(ir_rvalue **rvalue)
if (var->mode == ir_var_uniform)
return;
if (var->interface_type != NULL) {
if (var->get_interface_type() != NULL) {
char *iface_field_name =
ralloc_asprintf(mem_ctx, "%s.%s", var->interface_type->name,
ralloc_asprintf(mem_ctx, "%s.%s", var->get_interface_type()->name,
ir->field);
/* Find the variable in the set of flattened interface blocks */
ir_variable *found_var =

3
src/glsl/lower_ubo_reference.cpp
View File

@@ -132,7 +132,8 @@ lower_ubo_reference_visitor::handle_rvalue(ir_rvalue **rvalue)
mem_ctx = ralloc_parent(*rvalue);
const char *const field_name =
interface_field_name(mem_ctx, (char *) var->interface_type->name, deref);
interface_field_name(mem_ctx, (char *) var->get_interface_type()->name,
deref);
this->uniform_block = -1;
for (unsigned i = 0; i < shader->NumUniformBlocks; i++) {