summaryrefslogtreecommitdiffstats
path: root/src/compiler/glsl/link_functions.cpp
diff options
context:
space:
mode:
authorKenneth Graunke <kenneth@whitecape.org>2014-11-11 22:32:27 -0800
committerKenneth Graunke <kenneth@whitecape.org>2016-09-23 16:40:40 -0700
commit943b69cddd2ae90e0b0fcab2dff4a7eea81bb3cc (patch)
tree6e6085abed4ed4fdb0ceca66a4da8c007afa823e /src/compiler/glsl/link_functions.cpp
parentf7a5c714b304b85fde606a716d47581384cdcf75 (diff)
downloadexternal_mesa3d-943b69cddd2ae90e0b0fcab2dff4a7eea81bb3cc.zip
external_mesa3d-943b69cddd2ae90e0b0fcab2dff4a7eea81bb3cc.tar.gz
external_mesa3d-943b69cddd2ae90e0b0fcab2dff4a7eea81bb3cc.tar.bz2
glsl: Delete linker stuff relating to built-in functions.
Now that we generate built-in functions inline, there's no need to link against the built-in shader, and no built-in prototypes to consider. This lets us delete a bunch of code. Signed-off-by: Kenneth Graunke <kenneth@whitecape.org> Reviewed-by; Ian Romanick <ian.d.romanick@intel.com>
Diffstat (limited to 'src/compiler/glsl/link_functions.cpp')
-rw-r--r--src/compiler/glsl/link_functions.cpp32
1 files changed, 13 insertions, 19 deletions
diff --git a/src/compiler/glsl/link_functions.cpp b/src/compiler/glsl/link_functions.cpp
index b4aae5e..e4f77be 100644
--- a/src/compiler/glsl/link_functions.cpp
+++ b/src/compiler/glsl/link_functions.cpp
@@ -32,7 +32,7 @@
static ir_function_signature *
find_matching_signature(const char *name, const exec_list *actual_parameters,
- glsl_symbol_table *symbols, bool use_builtin);
+ glsl_symbol_table *symbols);
namespace {
@@ -74,12 +74,15 @@ public:
assert(callee != NULL);
const char *const name = callee->function_name();
+ /* We don't actually need to find intrinsics; they're not real */
+ if (callee->is_intrinsic)
+ return visit_continue;
+
/* Determine if the requested function signature already exists in the
* final linked shader. If it does, use it as the target of the call.
*/
ir_function_signature *sig =
- find_matching_signature(name, &callee->parameters, linked->symbols,
- ir->use_builtin);
+ find_matching_signature(name, &callee->parameters, linked->symbols);
if (sig != NULL) {
ir->callee = sig;
return visit_continue;
@@ -90,8 +93,7 @@ public:
*/
for (unsigned i = 0; i < num_shaders; i++) {
sig = find_matching_signature(name, &ir->actual_parameters,
- shader_list[i]->symbols,
- ir->use_builtin);
+ shader_list[i]->symbols);
if (sig)
break;
}
@@ -122,9 +124,7 @@ public:
ir_function_signature *linked_sig =
f->exact_matching_signature(NULL, &callee->parameters);
- if ((linked_sig == NULL)
- || ((linked_sig != NULL)
- && (linked_sig->is_builtin() != ir->use_builtin))) {
+ if (linked_sig == NULL) {
linked_sig = new(linked) ir_function_signature(callee->return_type);
f->add_signature(linked_sig);
}
@@ -314,22 +314,16 @@ private:
*/
ir_function_signature *
find_matching_signature(const char *name, const exec_list *actual_parameters,
- glsl_symbol_table *symbols, bool use_builtin)
+ glsl_symbol_table *symbols)
{
ir_function *const f = symbols->get_function(name);
if (f) {
ir_function_signature *sig =
- f->matching_signature(NULL, actual_parameters, use_builtin);
-
- if (sig && (sig->is_defined || sig->is_intrinsic)) {
- /* If this function expects to bind to a built-in function and the
- * signature that we found isn't a built-in, keep looking. Also keep
- * looking if we expect a non-built-in but found a built-in.
- */
- if (use_builtin == sig->is_builtin())
- return sig;
- }
+ f->matching_signature(NULL, actual_parameters, false);
+
+ if (sig && (sig->is_defined || sig->is_intrinsic))
+ return sig;
}
return NULL;