glsl: Pass _mesa_glsl_parse_state into matching_signature and such.
During compilation, we'll use this to determine built-in availability. The plan is to have a single shader containing every built-in in every version of the language, but filter out the ones that aren't actually available to the shader being compiled. At link time, we don't actually need this filtering capability: we've already imported prototypes for every built-in that the shader actually calls, and they're flagged as is_builtin(). The linker doesn't import any additional prototypes, so it won't pull in any unavailable built-ins. When resolving prototypes to function definitions, the linker ensures the values of is_builtin() match, which means that a shader can't trick the linker into importing the body of an unavailable built-in by defining a suspiciously similar prototype. In other words, during linking, we can just pass in NULL. It will work out fine. Signed-off-by: Kenneth Graunke <kenneth@whitecape.org> Reviewed-by: Matt Turner <mattst88@gmail.com> Reviewed-by: Paul Berry <stereotype441@gmail.com>
This commit is contained in:
@@ -388,7 +388,8 @@ match_function_by_name(const char *name,
|
|||||||
if (f != NULL) {
|
if (f != NULL) {
|
||||||
/* Look for a match in the local shader. If exact, we're done. */
|
/* Look for a match in the local shader. If exact, we're done. */
|
||||||
bool is_exact = false;
|
bool is_exact = false;
|
||||||
sig = local_sig = f->matching_signature(actual_parameters, &is_exact);
|
sig = local_sig = f->matching_signature(state, actual_parameters,
|
||||||
|
&is_exact);
|
||||||
if (is_exact)
|
if (is_exact)
|
||||||
goto done;
|
goto done;
|
||||||
|
|
||||||
@@ -411,7 +412,7 @@ match_function_by_name(const char *name,
|
|||||||
|
|
||||||
bool is_exact = false;
|
bool is_exact = false;
|
||||||
ir_function_signature *builtin_sig =
|
ir_function_signature *builtin_sig =
|
||||||
builtin->matching_signature(actual_parameters, &is_exact);
|
builtin->matching_signature(state, actual_parameters, &is_exact);
|
||||||
|
|
||||||
if (builtin_sig == NULL)
|
if (builtin_sig == NULL)
|
||||||
continue;
|
continue;
|
||||||
|
@@ -3576,7 +3576,7 @@ ast_function::hir(exec_list *instructions,
|
|||||||
*/
|
*/
|
||||||
f = state->symbols->get_function(name);
|
f = state->symbols->get_function(name);
|
||||||
if (f != NULL && (state->es_shader || f->has_user_signature())) {
|
if (f != NULL && (state->es_shader || f->has_user_signature())) {
|
||||||
sig = f->exact_matching_signature(&hir_parameters);
|
sig = f->exact_matching_signature(state, &hir_parameters);
|
||||||
if (sig != NULL) {
|
if (sig != NULL) {
|
||||||
const char *badvar = sig->qualifiers_match(&hir_parameters);
|
const char *badvar = sig->qualifiers_match(&hir_parameters);
|
||||||
if (badvar != NULL) {
|
if (badvar != NULL) {
|
||||||
|
@@ -765,20 +765,23 @@ public:
|
|||||||
* Find a signature that matches a set of actual parameters, taking implicit
|
* Find a signature that matches a set of actual parameters, taking implicit
|
||||||
* conversions into account. Also flags whether the match was exact.
|
* conversions into account. Also flags whether the match was exact.
|
||||||
*/
|
*/
|
||||||
ir_function_signature *matching_signature(const exec_list *actual_param,
|
ir_function_signature *matching_signature(_mesa_glsl_parse_state *state,
|
||||||
|
const exec_list *actual_param,
|
||||||
bool *match_is_exact);
|
bool *match_is_exact);
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Find a signature that matches a set of actual parameters, taking implicit
|
* Find a signature that matches a set of actual parameters, taking implicit
|
||||||
* conversions into account.
|
* conversions into account.
|
||||||
*/
|
*/
|
||||||
ir_function_signature *matching_signature(const exec_list *actual_param);
|
ir_function_signature *matching_signature(_mesa_glsl_parse_state *state,
|
||||||
|
const exec_list *actual_param);
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Find a signature that exactly matches a set of actual parameters without
|
* Find a signature that exactly matches a set of actual parameters without
|
||||||
* any implicit type conversions.
|
* any implicit type conversions.
|
||||||
*/
|
*/
|
||||||
ir_function_signature *exact_matching_signature(const exec_list *actual_ps);
|
ir_function_signature *exact_matching_signature(_mesa_glsl_parse_state *state,
|
||||||
|
const exec_list *actual_ps);
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Name of the function.
|
* Name of the function.
|
||||||
|
@@ -116,14 +116,16 @@ parameter_lists_match(const exec_list *list_a, const exec_list *list_b)
|
|||||||
|
|
||||||
|
|
||||||
ir_function_signature *
|
ir_function_signature *
|
||||||
ir_function::matching_signature(const exec_list *actual_parameters)
|
ir_function::matching_signature(_mesa_glsl_parse_state *state,
|
||||||
|
const exec_list *actual_parameters)
|
||||||
{
|
{
|
||||||
bool is_exact;
|
bool is_exact;
|
||||||
return matching_signature(actual_parameters, &is_exact);
|
return matching_signature(state, actual_parameters, &is_exact);
|
||||||
}
|
}
|
||||||
|
|
||||||
ir_function_signature *
|
ir_function_signature *
|
||||||
ir_function::matching_signature(const exec_list *actual_parameters,
|
ir_function::matching_signature(_mesa_glsl_parse_state *state,
|
||||||
|
const exec_list *actual_parameters,
|
||||||
bool *is_exact)
|
bool *is_exact)
|
||||||
{
|
{
|
||||||
ir_function_signature *match = NULL;
|
ir_function_signature *match = NULL;
|
||||||
@@ -203,7 +205,8 @@ parameter_lists_match_exact(const exec_list *list_a, const exec_list *list_b)
|
|||||||
}
|
}
|
||||||
|
|
||||||
ir_function_signature *
|
ir_function_signature *
|
||||||
ir_function::exact_matching_signature(const exec_list *actual_parameters)
|
ir_function::exact_matching_signature(_mesa_glsl_parse_state *state,
|
||||||
|
const exec_list *actual_parameters)
|
||||||
{
|
{
|
||||||
foreach_iter(exec_list_iterator, iter, signatures) {
|
foreach_iter(exec_list_iterator, iter, signatures) {
|
||||||
ir_function_signature *const sig =
|
ir_function_signature *const sig =
|
||||||
|
@@ -254,7 +254,8 @@ ir_reader::read_function_sig(ir_function *f, s_expression *expr, bool skip_body)
|
|||||||
hir_parameters.push_tail(var);
|
hir_parameters.push_tail(var);
|
||||||
}
|
}
|
||||||
|
|
||||||
ir_function_signature *sig = f->exact_matching_signature(&hir_parameters);
|
ir_function_signature *sig =
|
||||||
|
f->exact_matching_signature(state, &hir_parameters);
|
||||||
if (sig == NULL && skip_body) {
|
if (sig == NULL && skip_body) {
|
||||||
/* If scanning for prototypes, generate a new signature. */
|
/* If scanning for prototypes, generate a new signature. */
|
||||||
/* ir_reader doesn't know what languages support a given built-in, so
|
/* ir_reader doesn't know what languages support a given built-in, so
|
||||||
@@ -668,7 +669,7 @@ ir_reader::read_call(s_expression *expr)
|
|||||||
return NULL;
|
return NULL;
|
||||||
}
|
}
|
||||||
|
|
||||||
ir_function_signature *callee = f->matching_signature(¶meters);
|
ir_function_signature *callee = f->matching_signature(state, ¶meters);
|
||||||
if (callee == NULL) {
|
if (callee == NULL) {
|
||||||
ir_read_error(expr, "couldn't find matching signature for function "
|
ir_read_error(expr, "couldn't find matching signature for function "
|
||||||
"%s", name->value());
|
"%s", name->value());
|
||||||
|
@@ -113,7 +113,7 @@ public:
|
|||||||
}
|
}
|
||||||
|
|
||||||
ir_function_signature *linked_sig =
|
ir_function_signature *linked_sig =
|
||||||
f->exact_matching_signature(&callee->parameters);
|
f->exact_matching_signature(NULL, &callee->parameters);
|
||||||
if ((linked_sig == NULL)
|
if ((linked_sig == NULL)
|
||||||
|| ((linked_sig != NULL)
|
|| ((linked_sig != NULL)
|
||||||
&& (linked_sig->is_builtin() != ir->use_builtin))) {
|
&& (linked_sig->is_builtin() != ir->use_builtin))) {
|
||||||
@@ -288,7 +288,8 @@ find_matching_signature(const char *name, const exec_list *actual_parameters,
|
|||||||
if (f == NULL)
|
if (f == NULL)
|
||||||
continue;
|
continue;
|
||||||
|
|
||||||
ir_function_signature *sig = f->matching_signature(actual_parameters);
|
ir_function_signature *sig =
|
||||||
|
f->matching_signature(NULL, actual_parameters);
|
||||||
|
|
||||||
if ((sig == NULL) || !sig->is_defined)
|
if ((sig == NULL) || !sig->is_defined)
|
||||||
continue;
|
continue;
|
||||||
|
@@ -982,7 +982,7 @@ get_main_function_signature(gl_shader *sh)
|
|||||||
* We don't have to check for multiple definitions of main (in multiple
|
* We don't have to check for multiple definitions of main (in multiple
|
||||||
* shaders) because that would have already been caught above.
|
* shaders) because that would have already been caught above.
|
||||||
*/
|
*/
|
||||||
ir_function_signature *sig = f->matching_signature(&void_parameters);
|
ir_function_signature *sig = f->matching_signature(NULL, &void_parameters);
|
||||||
if ((sig != NULL) && sig->is_defined) {
|
if ((sig != NULL) && sig->is_defined) {
|
||||||
return sig;
|
return sig;
|
||||||
}
|
}
|
||||||
@@ -1167,7 +1167,7 @@ link_intrastage_shaders(void *mem_ctx,
|
|||||||
continue;
|
continue;
|
||||||
|
|
||||||
ir_function_signature *other_sig =
|
ir_function_signature *other_sig =
|
||||||
other->exact_matching_signature(& sig->parameters);
|
other->exact_matching_signature(NULL, &sig->parameters);
|
||||||
|
|
||||||
if ((other_sig != NULL) && other_sig->is_defined
|
if ((other_sig != NULL) && other_sig->is_defined
|
||||||
&& !other_sig->is_builtin()) {
|
&& !other_sig->is_builtin()) {
|
||||||
|
@@ -658,7 +658,7 @@ lower_packed_varyings(void *mem_ctx, unsigned location_base,
|
|||||||
ir_function *main_func = shader->symbols->get_function("main");
|
ir_function *main_func = shader->symbols->get_function("main");
|
||||||
exec_list void_parameters;
|
exec_list void_parameters;
|
||||||
ir_function_signature *main_func_sig
|
ir_function_signature *main_func_sig
|
||||||
= main_func->matching_signature(&void_parameters);
|
= main_func->matching_signature(NULL, &void_parameters);
|
||||||
exec_list new_instructions;
|
exec_list new_instructions;
|
||||||
lower_packed_varyings_visitor visitor(mem_ctx, location_base,
|
lower_packed_varyings_visitor visitor(mem_ctx, location_base,
|
||||||
locations_used, mode,
|
locations_used, mode,
|
||||||
|
@@ -2072,7 +2072,7 @@ fs_visitor::visit(ir_function *ir)
|
|||||||
const ir_function_signature *sig;
|
const ir_function_signature *sig;
|
||||||
exec_list empty;
|
exec_list empty;
|
||||||
|
|
||||||
sig = ir->matching_signature(&empty);
|
sig = ir->matching_signature(NULL, &empty);
|
||||||
|
|
||||||
assert(sig);
|
assert(sig);
|
||||||
|
|
||||||
|
@@ -1056,7 +1056,7 @@ vec4_visitor::visit(ir_function *ir)
|
|||||||
const ir_function_signature *sig;
|
const ir_function_signature *sig;
|
||||||
exec_list empty;
|
exec_list empty;
|
||||||
|
|
||||||
sig = ir->matching_signature(&empty);
|
sig = ir->matching_signature(NULL, &empty);
|
||||||
|
|
||||||
assert(sig);
|
assert(sig);
|
||||||
|
|
||||||
|
@@ -837,7 +837,7 @@ ir_to_mesa_visitor::visit(ir_function *ir)
|
|||||||
const ir_function_signature *sig;
|
const ir_function_signature *sig;
|
||||||
exec_list empty;
|
exec_list empty;
|
||||||
|
|
||||||
sig = ir->matching_signature(&empty);
|
sig = ir->matching_signature(NULL, &empty);
|
||||||
|
|
||||||
assert(sig);
|
assert(sig);
|
||||||
|
|
||||||
|
@@ -1209,7 +1209,7 @@ glsl_to_tgsi_visitor::visit(ir_function *ir)
|
|||||||
const ir_function_signature *sig;
|
const ir_function_signature *sig;
|
||||||
exec_list empty;
|
exec_list empty;
|
||||||
|
|
||||||
sig = ir->matching_signature(&empty);
|
sig = ir->matching_signature(NULL, &empty);
|
||||||
|
|
||||||
assert(sig);
|
assert(sig);
|
||||||
|
|
||||||
|
Reference in New Issue
Block a user