2013-05-20 23:42:49 -07:00
|
|
|
/*
|
|
|
|
|
* Copyright © 2013 Intel Corporation
|
|
|
|
|
*
|
|
|
|
|
* Permission is hereby granted, free of charge, to any person obtaining a
|
|
|
|
|
* copy of this software and associated documentation files (the "Software"),
|
|
|
|
|
* to deal in the Software without restriction, including without limitation
|
|
|
|
|
* the rights to use, copy, modify, merge, publish, distribute, sublicense,
|
|
|
|
|
* and/or sell copies of the Software, and to permit persons to whom the
|
|
|
|
|
* Software is furnished to do so, subject to the following conditions:
|
|
|
|
|
*
|
|
|
|
|
* The above copyright notice and this permission notice (including the next
|
|
|
|
|
* paragraph) shall be included in all copies or substantial portions of the
|
|
|
|
|
* Software.
|
|
|
|
|
*
|
|
|
|
|
* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
|
|
|
|
|
* IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
|
|
|
|
|
* FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
|
|
|
|
|
* THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
|
|
|
|
|
* LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
|
|
|
|
|
* FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
|
|
|
|
|
* DEALINGS IN THE SOFTWARE.
|
|
|
|
|
*/
|
|
|
|
|
|
|
|
|
|
/**
|
|
|
|
|
* \file link_interface_blocks.cpp
|
|
|
|
|
* Linker support for GLSL's interface blocks.
|
|
|
|
|
*/
|
|
|
|
|
|
|
|
|
|
#include "ir.h"
|
|
|
|
|
#include "glsl_symbol_table.h"
|
|
|
|
|
#include "linker.h"
|
|
|
|
|
#include "main/macros.h"
|
2022-01-07 15:11:21 +10:00
|
|
|
#include "main/shader_types.h"
|
2015-12-02 17:53:19 +11:00
|
|
|
#include "util/hash_table.h"
|
2018-08-01 13:22:00 +03:00
|
|
|
#include "util/u_string.h"
|
glsl: Rework interface block linking.
Previously, when doing intrastage and interstage interface block
linking, we only checked the interface type; this prevented us from
catching some link errors.
We now check the following additional constraints:
- For intrastage linking, the presence/absence of interface names must
match.
- For shader ins/outs, the interface names themselves must match when
doing intrastage linking (note: it's not clear from the spec whether
this is necessary, but Mesa's implementation currently relies on
it).
- Array vs. nonarray must be consistent, taking into account the
special rules for vertex-geometry linkage.
- Array sizes must be consistent (exception: during intrastage
linking, an unsized array matches a sized array).
Note: validate_interstage_interface_blocks currently handles both
uniforms and in/out variables. As a result, if all three shader types
are present (VS, GS, and FS), and a uniform interface block is
mentioned in the VS and FS but not the GS, it won't be validated. I
plan to address this in later patches.
Fixes the following piglit tests in spec/glsl-1.50/linker:
- interface-blocks-vs-fs-array-size-mismatch
- interface-vs-array-to-fs-unnamed
- interface-vs-unnamed-to-fs-array
- intrastage-interface-unnamed-array
v2: Simplify logic in intrastage_match() for handling array sizes.
Make extra_array_level const. Use an unnamed temporary
interface_block_definition in validate_interstage_interface_blocks()'s
first call to definitions->store().
Cc: "10.0" <mesa-stable@lists.freedesktop.org>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>
2013-10-29 14:41:32 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
namespace {
|
|
|
|
|
|
2016-03-08 22:25:58 +11:00
|
|
|
/**
|
|
|
|
|
* Return true if interface members mismatch and its not allowed by GLSL.
|
|
|
|
|
*/
|
|
|
|
|
static bool
|
|
|
|
|
interstage_member_mismatch(struct gl_shader_program *prog,
|
|
|
|
|
const glsl_type *c, const glsl_type *p) {
|
|
|
|
|
|
|
|
|
|
if (c->length != p->length)
|
|
|
|
|
return true;
|
|
|
|
|
|
|
|
|
|
for (unsigned i = 0; i < c->length; i++) {
|
|
|
|
|
if (c->fields.structure[i].type != p->fields.structure[i].type)
|
|
|
|
|
return true;
|
|
|
|
|
if (strcmp(c->fields.structure[i].name,
|
|
|
|
|
p->fields.structure[i].name) != 0)
|
|
|
|
|
return true;
|
|
|
|
|
if (c->fields.structure[i].location !=
|
|
|
|
|
p->fields.structure[i].location)
|
|
|
|
|
return true;
|
2021-05-12 15:44:02 +10:00
|
|
|
if (c->fields.structure[i].component !=
|
|
|
|
|
p->fields.structure[i].component)
|
|
|
|
|
return true;
|
2016-03-08 22:25:58 +11:00
|
|
|
if (c->fields.structure[i].patch !=
|
|
|
|
|
p->fields.structure[i].patch)
|
|
|
|
|
return true;
|
|
|
|
|
|
|
|
|
|
/* From Section 4.5 (Interpolation Qualifiers) of the GLSL 4.40 spec:
|
|
|
|
|
*
|
|
|
|
|
* "It is a link-time error if, within the same stage, the
|
|
|
|
|
* interpolation qualifiers of variables of the same name do not
|
|
|
|
|
* match."
|
|
|
|
|
*/
|
2023-02-08 15:28:08 +01:00
|
|
|
if (prog->IsES || prog->GLSL_Version < 440)
|
2016-03-08 22:25:58 +11:00
|
|
|
if (c->fields.structure[i].interpolation !=
|
|
|
|
|
p->fields.structure[i].interpolation)
|
|
|
|
|
return true;
|
|
|
|
|
|
|
|
|
|
/* From Section 4.3.4 (Input Variables) of the GLSL ES 3.0 spec:
|
|
|
|
|
*
|
|
|
|
|
* "The output of the vertex shader and the input of the fragment
|
|
|
|
|
* shader form an interface. For this interface, vertex shader
|
|
|
|
|
* output variables and fragment shader input variables of the same
|
|
|
|
|
* name must match in type and qualification (other than precision
|
|
|
|
|
* and out matching to in).
|
|
|
|
|
*
|
|
|
|
|
* The table in Section 9.2.1 Linked Shaders of the GLSL ES 3.1 spec
|
|
|
|
|
* says that centroid no longer needs to match for varyings.
|
|
|
|
|
*
|
|
|
|
|
* The table in Section 9.2.1 Linked Shaders of the GLSL ES 3.2 spec
|
|
|
|
|
* says that sample need not match for varyings.
|
|
|
|
|
*/
|
2023-02-08 15:28:08 +01:00
|
|
|
if (!prog->IsES || prog->GLSL_Version < 310)
|
2016-03-08 22:25:58 +11:00
|
|
|
if (c->fields.structure[i].centroid !=
|
|
|
|
|
p->fields.structure[i].centroid)
|
|
|
|
|
return true;
|
|
|
|
|
if (!prog->IsES)
|
|
|
|
|
if (c->fields.structure[i].sample !=
|
|
|
|
|
p->fields.structure[i].sample)
|
|
|
|
|
return true;
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
return false;
|
|
|
|
|
}
|
glsl: Rework interface block linking.
Previously, when doing intrastage and interstage interface block
linking, we only checked the interface type; this prevented us from
catching some link errors.
We now check the following additional constraints:
- For intrastage linking, the presence/absence of interface names must
match.
- For shader ins/outs, the interface names themselves must match when
doing intrastage linking (note: it's not clear from the spec whether
this is necessary, but Mesa's implementation currently relies on
it).
- Array vs. nonarray must be consistent, taking into account the
special rules for vertex-geometry linkage.
- Array sizes must be consistent (exception: during intrastage
linking, an unsized array matches a sized array).
Note: validate_interstage_interface_blocks currently handles both
uniforms and in/out variables. As a result, if all three shader types
are present (VS, GS, and FS), and a uniform interface block is
mentioned in the VS and FS but not the GS, it won't be validated. I
plan to address this in later patches.
Fixes the following piglit tests in spec/glsl-1.50/linker:
- interface-blocks-vs-fs-array-size-mismatch
- interface-vs-array-to-fs-unnamed
- interface-vs-unnamed-to-fs-array
- intrastage-interface-unnamed-array
v2: Simplify logic in intrastage_match() for handling array sizes.
Make extra_array_level const. Use an unnamed temporary
interface_block_definition in validate_interstage_interface_blocks()'s
first call to definitions->store().
Cc: "10.0" <mesa-stable@lists.freedesktop.org>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>
2013-10-29 14:41:32 -07:00
|
|
|
|
2016-10-18 10:52:58 +02:00
|
|
|
/**
|
|
|
|
|
* Check if two interfaces match, according to intrastage interface matching
|
|
|
|
|
* rules. If they do, and the first interface uses an unsized array, it will
|
|
|
|
|
* be updated to reflect the array size declared in the second interface.
|
|
|
|
|
*/
|
|
|
|
|
bool
|
|
|
|
|
intrastage_match(ir_variable *a,
|
|
|
|
|
ir_variable *b,
|
2019-04-23 16:52:36 +02:00
|
|
|
struct gl_shader_program *prog,
|
|
|
|
|
bool match_precision)
|
2016-10-18 10:52:58 +02:00
|
|
|
{
|
2021-04-21 09:54:38 +03:00
|
|
|
/* From section 4.7 "Precision and Precision Qualifiers" in GLSL 4.50:
|
|
|
|
|
*
|
|
|
|
|
* "For the purposes of determining if an output from one shader
|
|
|
|
|
* stage matches an input of the next stage, the precision qualifier
|
|
|
|
|
* need not match."
|
|
|
|
|
*/
|
|
|
|
|
bool interface_type_match =
|
|
|
|
|
(prog->IsES ?
|
|
|
|
|
a->get_interface_type() == b->get_interface_type() :
|
2023-12-14 22:21:26 -08:00
|
|
|
glsl_type_compare_no_precision(a->get_interface_type(), b->get_interface_type()));
|
2021-04-21 09:54:38 +03:00
|
|
|
|
2016-10-18 10:52:58 +02:00
|
|
|
/* Types must match. */
|
2021-04-21 09:54:38 +03:00
|
|
|
if (!interface_type_match) {
|
2016-10-18 10:52:58 +02:00
|
|
|
/* Exception: if both the interface blocks are implicitly declared,
|
|
|
|
|
* don't force their types to match. They might mismatch due to the two
|
|
|
|
|
* shaders using different GLSL versions, and that's ok.
|
|
|
|
|
*/
|
2016-10-18 09:38:30 +02:00
|
|
|
if ((a->data.how_declared != ir_var_declared_implicitly ||
|
|
|
|
|
b->data.how_declared != ir_var_declared_implicitly) &&
|
2017-11-09 09:58:25 +01:00
|
|
|
(!prog->IsES ||
|
2016-10-18 09:38:30 +02:00
|
|
|
interstage_member_mismatch(prog, a->get_interface_type(),
|
|
|
|
|
b->get_interface_type())))
|
2016-10-18 10:52:58 +02:00
|
|
|
return false;
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
/* Presence/absence of interface names must match. */
|
|
|
|
|
if (a->is_interface_instance() != b->is_interface_instance())
|
|
|
|
|
return false;
|
|
|
|
|
|
|
|
|
|
/* For uniforms, instance names need not match. For shader ins/outs,
|
|
|
|
|
* it's not clear from the spec whether they need to match, but
|
|
|
|
|
* Mesa's implementation relies on them matching.
|
|
|
|
|
*/
|
|
|
|
|
if (a->is_interface_instance() && b->data.mode != ir_var_uniform &&
|
|
|
|
|
b->data.mode != ir_var_shader_storage &&
|
|
|
|
|
strcmp(a->name, b->name) != 0) {
|
|
|
|
|
return false;
|
|
|
|
|
}
|
|
|
|
|
|
2019-04-23 16:52:36 +02:00
|
|
|
bool type_match = (match_precision ?
|
|
|
|
|
a->type == b->type :
|
2023-12-14 22:21:26 -08:00
|
|
|
glsl_type_compare_no_precision(a->type, b->type));
|
2019-04-23 16:52:36 +02:00
|
|
|
|
2016-10-18 10:52:58 +02:00
|
|
|
/* If a block is an array then it must match across the shader.
|
|
|
|
|
* Unsized arrays are also processed and matched agaist sized arrays.
|
|
|
|
|
*/
|
2023-12-14 22:21:26 -08:00
|
|
|
if (!type_match && (glsl_type_is_array(b->type) || glsl_type_is_array(a->type)) &&
|
2016-10-18 10:52:58 +02:00
|
|
|
(b->is_interface_instance() || a->is_interface_instance()) &&
|
2019-04-23 16:52:36 +02:00
|
|
|
!validate_intrastage_arrays(prog, b, a, match_precision))
|
2016-10-18 10:52:58 +02:00
|
|
|
return false;
|
|
|
|
|
|
|
|
|
|
return true;
|
|
|
|
|
}
|
|
|
|
|
|
glsl: Rework interface block linking.
Previously, when doing intrastage and interstage interface block
linking, we only checked the interface type; this prevented us from
catching some link errors.
We now check the following additional constraints:
- For intrastage linking, the presence/absence of interface names must
match.
- For shader ins/outs, the interface names themselves must match when
doing intrastage linking (note: it's not clear from the spec whether
this is necessary, but Mesa's implementation currently relies on
it).
- Array vs. nonarray must be consistent, taking into account the
special rules for vertex-geometry linkage.
- Array sizes must be consistent (exception: during intrastage
linking, an unsized array matches a sized array).
Note: validate_interstage_interface_blocks currently handles both
uniforms and in/out variables. As a result, if all three shader types
are present (VS, GS, and FS), and a uniform interface block is
mentioned in the VS and FS but not the GS, it won't be validated. I
plan to address this in later patches.
Fixes the following piglit tests in spec/glsl-1.50/linker:
- interface-blocks-vs-fs-array-size-mismatch
- interface-vs-array-to-fs-unnamed
- interface-vs-unnamed-to-fs-array
- intrastage-interface-unnamed-array
v2: Simplify logic in intrastage_match() for handling array sizes.
Make extra_array_level const. Use an unnamed temporary
interface_block_definition in validate_interstage_interface_blocks()'s
first call to definitions->store().
Cc: "10.0" <mesa-stable@lists.freedesktop.org>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>
2013-10-29 14:41:32 -07:00
|
|
|
|
|
|
|
|
/**
|
|
|
|
|
* This class keeps track of a mapping from an interface block name to the
|
|
|
|
|
* necessary information about that interface block to determine whether to
|
|
|
|
|
* generate a link error.
|
|
|
|
|
*
|
|
|
|
|
* Note: this class is expected to be short lived, so it doesn't make copies
|
|
|
|
|
* of the strings it references; it simply borrows the pointers from the
|
|
|
|
|
* ir_variable class.
|
|
|
|
|
*/
|
|
|
|
|
class interface_block_definitions
|
|
|
|
|
{
|
|
|
|
|
public:
|
|
|
|
|
interface_block_definitions()
|
|
|
|
|
: mem_ctx(ralloc_context(NULL)),
|
2020-01-18 01:38:31 -05:00
|
|
|
ht(_mesa_hash_table_create(NULL, _mesa_hash_string,
|
2015-12-02 17:53:19 +11:00
|
|
|
_mesa_key_string_equal))
|
glsl: Rework interface block linking.
Previously, when doing intrastage and interstage interface block
linking, we only checked the interface type; this prevented us from
catching some link errors.
We now check the following additional constraints:
- For intrastage linking, the presence/absence of interface names must
match.
- For shader ins/outs, the interface names themselves must match when
doing intrastage linking (note: it's not clear from the spec whether
this is necessary, but Mesa's implementation currently relies on
it).
- Array vs. nonarray must be consistent, taking into account the
special rules for vertex-geometry linkage.
- Array sizes must be consistent (exception: during intrastage
linking, an unsized array matches a sized array).
Note: validate_interstage_interface_blocks currently handles both
uniforms and in/out variables. As a result, if all three shader types
are present (VS, GS, and FS), and a uniform interface block is
mentioned in the VS and FS but not the GS, it won't be validated. I
plan to address this in later patches.
Fixes the following piglit tests in spec/glsl-1.50/linker:
- interface-blocks-vs-fs-array-size-mismatch
- interface-vs-array-to-fs-unnamed
- interface-vs-unnamed-to-fs-array
- intrastage-interface-unnamed-array
v2: Simplify logic in intrastage_match() for handling array sizes.
Make extra_array_level const. Use an unnamed temporary
interface_block_definition in validate_interstage_interface_blocks()'s
first call to definitions->store().
Cc: "10.0" <mesa-stable@lists.freedesktop.org>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>
2013-10-29 14:41:32 -07:00
|
|
|
{
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
~interface_block_definitions()
|
|
|
|
|
{
|
|
|
|
|
ralloc_free(mem_ctx);
|
2015-12-02 17:53:19 +11:00
|
|
|
_mesa_hash_table_destroy(ht, NULL);
|
glsl: Rework interface block linking.
Previously, when doing intrastage and interstage interface block
linking, we only checked the interface type; this prevented us from
catching some link errors.
We now check the following additional constraints:
- For intrastage linking, the presence/absence of interface names must
match.
- For shader ins/outs, the interface names themselves must match when
doing intrastage linking (note: it's not clear from the spec whether
this is necessary, but Mesa's implementation currently relies on
it).
- Array vs. nonarray must be consistent, taking into account the
special rules for vertex-geometry linkage.
- Array sizes must be consistent (exception: during intrastage
linking, an unsized array matches a sized array).
Note: validate_interstage_interface_blocks currently handles both
uniforms and in/out variables. As a result, if all three shader types
are present (VS, GS, and FS), and a uniform interface block is
mentioned in the VS and FS but not the GS, it won't be validated. I
plan to address this in later patches.
Fixes the following piglit tests in spec/glsl-1.50/linker:
- interface-blocks-vs-fs-array-size-mismatch
- interface-vs-array-to-fs-unnamed
- interface-vs-unnamed-to-fs-array
- intrastage-interface-unnamed-array
v2: Simplify logic in intrastage_match() for handling array sizes.
Make extra_array_level const. Use an unnamed temporary
interface_block_definition in validate_interstage_interface_blocks()'s
first call to definitions->store().
Cc: "10.0" <mesa-stable@lists.freedesktop.org>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>
2013-10-29 14:41:32 -07:00
|
|
|
}
|
|
|
|
|
|
|
|
|
|
/**
|
2015-12-02 17:53:19 +11:00
|
|
|
* Lookup the interface definition. Return NULL if none is found.
|
glsl: Rework interface block linking.
Previously, when doing intrastage and interstage interface block
linking, we only checked the interface type; this prevented us from
catching some link errors.
We now check the following additional constraints:
- For intrastage linking, the presence/absence of interface names must
match.
- For shader ins/outs, the interface names themselves must match when
doing intrastage linking (note: it's not clear from the spec whether
this is necessary, but Mesa's implementation currently relies on
it).
- Array vs. nonarray must be consistent, taking into account the
special rules for vertex-geometry linkage.
- Array sizes must be consistent (exception: during intrastage
linking, an unsized array matches a sized array).
Note: validate_interstage_interface_blocks currently handles both
uniforms and in/out variables. As a result, if all three shader types
are present (VS, GS, and FS), and a uniform interface block is
mentioned in the VS and FS but not the GS, it won't be validated. I
plan to address this in later patches.
Fixes the following piglit tests in spec/glsl-1.50/linker:
- interface-blocks-vs-fs-array-size-mismatch
- interface-vs-array-to-fs-unnamed
- interface-vs-unnamed-to-fs-array
- intrastage-interface-unnamed-array
v2: Simplify logic in intrastage_match() for handling array sizes.
Make extra_array_level const. Use an unnamed temporary
interface_block_definition in validate_interstage_interface_blocks()'s
first call to definitions->store().
Cc: "10.0" <mesa-stable@lists.freedesktop.org>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>
2013-10-29 14:41:32 -07:00
|
|
|
*/
|
2015-12-02 17:53:19 +11:00
|
|
|
ir_variable *lookup(ir_variable *var)
|
glsl: Rework interface block linking.
Previously, when doing intrastage and interstage interface block
linking, we only checked the interface type; this prevented us from
catching some link errors.
We now check the following additional constraints:
- For intrastage linking, the presence/absence of interface names must
match.
- For shader ins/outs, the interface names themselves must match when
doing intrastage linking (note: it's not clear from the spec whether
this is necessary, but Mesa's implementation currently relies on
it).
- Array vs. nonarray must be consistent, taking into account the
special rules for vertex-geometry linkage.
- Array sizes must be consistent (exception: during intrastage
linking, an unsized array matches a sized array).
Note: validate_interstage_interface_blocks currently handles both
uniforms and in/out variables. As a result, if all three shader types
are present (VS, GS, and FS), and a uniform interface block is
mentioned in the VS and FS but not the GS, it won't be validated. I
plan to address this in later patches.
Fixes the following piglit tests in spec/glsl-1.50/linker:
- interface-blocks-vs-fs-array-size-mismatch
- interface-vs-array-to-fs-unnamed
- interface-vs-unnamed-to-fs-array
- intrastage-interface-unnamed-array
v2: Simplify logic in intrastage_match() for handling array sizes.
Make extra_array_level const. Use an unnamed temporary
interface_block_definition in validate_interstage_interface_blocks()'s
first call to definitions->store().
Cc: "10.0" <mesa-stable@lists.freedesktop.org>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>
2013-10-29 14:41:32 -07:00
|
|
|
{
|
2015-12-01 10:34:18 +11:00
|
|
|
if (var->data.explicit_location &&
|
|
|
|
|
var->data.location >= VARYING_SLOT_VAR0) {
|
|
|
|
|
char location_str[11];
|
2018-11-20 11:59:28 +00:00
|
|
|
snprintf(location_str, 11, "%d", var->data.location);
|
2015-12-01 10:34:18 +11:00
|
|
|
|
|
|
|
|
const struct hash_entry *entry =
|
|
|
|
|
_mesa_hash_table_search(ht, location_str);
|
|
|
|
|
return entry ? (ir_variable *) entry->data : NULL;
|
|
|
|
|
} else {
|
|
|
|
|
const struct hash_entry *entry =
|
2016-03-11 16:15:02 +11:00
|
|
|
_mesa_hash_table_search(ht,
|
2023-12-14 22:21:26 -08:00
|
|
|
glsl_get_type_name(glsl_without_array(var->get_interface_type())));
|
2015-12-01 10:34:18 +11:00
|
|
|
return entry ? (ir_variable *) entry->data : NULL;
|
|
|
|
|
}
|
glsl: Rework interface block linking.
Previously, when doing intrastage and interstage interface block
linking, we only checked the interface type; this prevented us from
catching some link errors.
We now check the following additional constraints:
- For intrastage linking, the presence/absence of interface names must
match.
- For shader ins/outs, the interface names themselves must match when
doing intrastage linking (note: it's not clear from the spec whether
this is necessary, but Mesa's implementation currently relies on
it).
- Array vs. nonarray must be consistent, taking into account the
special rules for vertex-geometry linkage.
- Array sizes must be consistent (exception: during intrastage
linking, an unsized array matches a sized array).
Note: validate_interstage_interface_blocks currently handles both
uniforms and in/out variables. As a result, if all three shader types
are present (VS, GS, and FS), and a uniform interface block is
mentioned in the VS and FS but not the GS, it won't be validated. I
plan to address this in later patches.
Fixes the following piglit tests in spec/glsl-1.50/linker:
- interface-blocks-vs-fs-array-size-mismatch
- interface-vs-array-to-fs-unnamed
- interface-vs-unnamed-to-fs-array
- intrastage-interface-unnamed-array
v2: Simplify logic in intrastage_match() for handling array sizes.
Make extra_array_level const. Use an unnamed temporary
interface_block_definition in validate_interstage_interface_blocks()'s
first call to definitions->store().
Cc: "10.0" <mesa-stable@lists.freedesktop.org>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>
2013-10-29 14:41:32 -07:00
|
|
|
}
|
|
|
|
|
|
|
|
|
|
/**
|
|
|
|
|
* Add a new interface definition.
|
|
|
|
|
*/
|
2015-12-02 17:53:19 +11:00
|
|
|
void store(ir_variable *var)
|
glsl: Rework interface block linking.
Previously, when doing intrastage and interstage interface block
linking, we only checked the interface type; this prevented us from
catching some link errors.
We now check the following additional constraints:
- For intrastage linking, the presence/absence of interface names must
match.
- For shader ins/outs, the interface names themselves must match when
doing intrastage linking (note: it's not clear from the spec whether
this is necessary, but Mesa's implementation currently relies on
it).
- Array vs. nonarray must be consistent, taking into account the
special rules for vertex-geometry linkage.
- Array sizes must be consistent (exception: during intrastage
linking, an unsized array matches a sized array).
Note: validate_interstage_interface_blocks currently handles both
uniforms and in/out variables. As a result, if all three shader types
are present (VS, GS, and FS), and a uniform interface block is
mentioned in the VS and FS but not the GS, it won't be validated. I
plan to address this in later patches.
Fixes the following piglit tests in spec/glsl-1.50/linker:
- interface-blocks-vs-fs-array-size-mismatch
- interface-vs-array-to-fs-unnamed
- interface-vs-unnamed-to-fs-array
- intrastage-interface-unnamed-array
v2: Simplify logic in intrastage_match() for handling array sizes.
Make extra_array_level const. Use an unnamed temporary
interface_block_definition in validate_interstage_interface_blocks()'s
first call to definitions->store().
Cc: "10.0" <mesa-stable@lists.freedesktop.org>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>
2013-10-29 14:41:32 -07:00
|
|
|
{
|
2015-12-01 10:34:18 +11:00
|
|
|
if (var->data.explicit_location &&
|
|
|
|
|
var->data.location >= VARYING_SLOT_VAR0) {
|
|
|
|
|
/* If explicit location is given then lookup the variable by location.
|
|
|
|
|
* We turn the location into a string and use this as the hash key
|
|
|
|
|
* rather than the name. Note: We allocate enough space for a 32-bit
|
|
|
|
|
* unsigned location value which is overkill but future proof.
|
|
|
|
|
*/
|
|
|
|
|
char location_str[11];
|
2018-11-20 11:59:28 +00:00
|
|
|
snprintf(location_str, 11, "%d", var->data.location);
|
2015-12-01 10:34:18 +11:00
|
|
|
_mesa_hash_table_insert(ht, ralloc_strdup(mem_ctx, location_str), var);
|
|
|
|
|
} else {
|
2016-03-11 16:15:02 +11:00
|
|
|
_mesa_hash_table_insert(ht,
|
2023-12-14 22:21:26 -08:00
|
|
|
glsl_get_type_name(glsl_without_array(var->get_interface_type())), var);
|
2015-12-01 10:34:18 +11:00
|
|
|
}
|
glsl: Rework interface block linking.
Previously, when doing intrastage and interstage interface block
linking, we only checked the interface type; this prevented us from
catching some link errors.
We now check the following additional constraints:
- For intrastage linking, the presence/absence of interface names must
match.
- For shader ins/outs, the interface names themselves must match when
doing intrastage linking (note: it's not clear from the spec whether
this is necessary, but Mesa's implementation currently relies on
it).
- Array vs. nonarray must be consistent, taking into account the
special rules for vertex-geometry linkage.
- Array sizes must be consistent (exception: during intrastage
linking, an unsized array matches a sized array).
Note: validate_interstage_interface_blocks currently handles both
uniforms and in/out variables. As a result, if all three shader types
are present (VS, GS, and FS), and a uniform interface block is
mentioned in the VS and FS but not the GS, it won't be validated. I
plan to address this in later patches.
Fixes the following piglit tests in spec/glsl-1.50/linker:
- interface-blocks-vs-fs-array-size-mismatch
- interface-vs-array-to-fs-unnamed
- interface-vs-unnamed-to-fs-array
- intrastage-interface-unnamed-array
v2: Simplify logic in intrastage_match() for handling array sizes.
Make extra_array_level const. Use an unnamed temporary
interface_block_definition in validate_interstage_interface_blocks()'s
first call to definitions->store().
Cc: "10.0" <mesa-stable@lists.freedesktop.org>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>
2013-10-29 14:41:32 -07:00
|
|
|
}
|
|
|
|
|
|
|
|
|
|
private:
|
|
|
|
|
/**
|
|
|
|
|
* Ralloc context for data structures allocated by this class.
|
|
|
|
|
*/
|
|
|
|
|
void *mem_ctx;
|
|
|
|
|
|
|
|
|
|
/**
|
|
|
|
|
* Hash table mapping interface block name to an \c
|
2015-12-02 17:53:19 +11:00
|
|
|
* ir_variable.
|
glsl: Rework interface block linking.
Previously, when doing intrastage and interstage interface block
linking, we only checked the interface type; this prevented us from
catching some link errors.
We now check the following additional constraints:
- For intrastage linking, the presence/absence of interface names must
match.
- For shader ins/outs, the interface names themselves must match when
doing intrastage linking (note: it's not clear from the spec whether
this is necessary, but Mesa's implementation currently relies on
it).
- Array vs. nonarray must be consistent, taking into account the
special rules for vertex-geometry linkage.
- Array sizes must be consistent (exception: during intrastage
linking, an unsized array matches a sized array).
Note: validate_interstage_interface_blocks currently handles both
uniforms and in/out variables. As a result, if all three shader types
are present (VS, GS, and FS), and a uniform interface block is
mentioned in the VS and FS but not the GS, it won't be validated. I
plan to address this in later patches.
Fixes the following piglit tests in spec/glsl-1.50/linker:
- interface-blocks-vs-fs-array-size-mismatch
- interface-vs-array-to-fs-unnamed
- interface-vs-unnamed-to-fs-array
- intrastage-interface-unnamed-array
v2: Simplify logic in intrastage_match() for handling array sizes.
Make extra_array_level const. Use an unnamed temporary
interface_block_definition in validate_interstage_interface_blocks()'s
first call to definitions->store().
Cc: "10.0" <mesa-stable@lists.freedesktop.org>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>
2013-10-29 14:41:32 -07:00
|
|
|
*/
|
|
|
|
|
hash_table *ht;
|
|
|
|
|
};
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
}; /* anonymous namespace */
|
|
|
|
|
|
2013-05-20 23:42:49 -07:00
|
|
|
|
2013-07-27 11:08:31 -07:00
|
|
|
void
|
2013-07-27 14:58:43 -07:00
|
|
|
validate_intrastage_interface_blocks(struct gl_shader_program *prog,
|
|
|
|
|
const gl_shader **shader_list,
|
2013-05-20 23:42:49 -07:00
|
|
|
unsigned num_shaders)
|
|
|
|
|
{
|
glsl: Rework interface block linking.
Previously, when doing intrastage and interstage interface block
linking, we only checked the interface type; this prevented us from
catching some link errors.
We now check the following additional constraints:
- For intrastage linking, the presence/absence of interface names must
match.
- For shader ins/outs, the interface names themselves must match when
doing intrastage linking (note: it's not clear from the spec whether
this is necessary, but Mesa's implementation currently relies on
it).
- Array vs. nonarray must be consistent, taking into account the
special rules for vertex-geometry linkage.
- Array sizes must be consistent (exception: during intrastage
linking, an unsized array matches a sized array).
Note: validate_interstage_interface_blocks currently handles both
uniforms and in/out variables. As a result, if all three shader types
are present (VS, GS, and FS), and a uniform interface block is
mentioned in the VS and FS but not the GS, it won't be validated. I
plan to address this in later patches.
Fixes the following piglit tests in spec/glsl-1.50/linker:
- interface-blocks-vs-fs-array-size-mismatch
- interface-vs-array-to-fs-unnamed
- interface-vs-unnamed-to-fs-array
- intrastage-interface-unnamed-array
v2: Simplify logic in intrastage_match() for handling array sizes.
Make extra_array_level const. Use an unnamed temporary
interface_block_definition in validate_interstage_interface_blocks()'s
first call to definitions->store().
Cc: "10.0" <mesa-stable@lists.freedesktop.org>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>
2013-10-29 14:41:32 -07:00
|
|
|
interface_block_definitions in_interfaces;
|
|
|
|
|
interface_block_definitions out_interfaces;
|
|
|
|
|
interface_block_definitions uniform_interfaces;
|
2015-05-13 11:17:23 +02:00
|
|
|
interface_block_definitions buffer_interfaces;
|
2013-05-20 23:42:49 -07:00
|
|
|
|
|
|
|
|
for (unsigned int i = 0; i < num_shaders; i++) {
|
|
|
|
|
if (shader_list[i] == NULL)
|
|
|
|
|
continue;
|
|
|
|
|
|
2014-06-24 21:34:05 -07:00
|
|
|
foreach_in_list(ir_instruction, node, shader_list[i]->ir) {
|
|
|
|
|
ir_variable *var = node->as_variable();
|
2013-05-20 23:42:49 -07:00
|
|
|
if (!var)
|
|
|
|
|
continue;
|
|
|
|
|
|
2013-09-24 14:30:29 -07:00
|
|
|
const glsl_type *iface_type = var->get_interface_type();
|
2013-05-20 23:42:49 -07:00
|
|
|
|
|
|
|
|
if (iface_type == NULL)
|
|
|
|
|
continue;
|
|
|
|
|
|
glsl: Rework interface block linking.
Previously, when doing intrastage and interstage interface block
linking, we only checked the interface type; this prevented us from
catching some link errors.
We now check the following additional constraints:
- For intrastage linking, the presence/absence of interface names must
match.
- For shader ins/outs, the interface names themselves must match when
doing intrastage linking (note: it's not clear from the spec whether
this is necessary, but Mesa's implementation currently relies on
it).
- Array vs. nonarray must be consistent, taking into account the
special rules for vertex-geometry linkage.
- Array sizes must be consistent (exception: during intrastage
linking, an unsized array matches a sized array).
Note: validate_interstage_interface_blocks currently handles both
uniforms and in/out variables. As a result, if all three shader types
are present (VS, GS, and FS), and a uniform interface block is
mentioned in the VS and FS but not the GS, it won't be validated. I
plan to address this in later patches.
Fixes the following piglit tests in spec/glsl-1.50/linker:
- interface-blocks-vs-fs-array-size-mismatch
- interface-vs-array-to-fs-unnamed
- interface-vs-unnamed-to-fs-array
- intrastage-interface-unnamed-array
v2: Simplify logic in intrastage_match() for handling array sizes.
Make extra_array_level const. Use an unnamed temporary
interface_block_definition in validate_interstage_interface_blocks()'s
first call to definitions->store().
Cc: "10.0" <mesa-stable@lists.freedesktop.org>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>
2013-10-29 14:41:32 -07:00
|
|
|
interface_block_definitions *definitions;
|
2013-12-12 13:51:01 +02:00
|
|
|
switch (var->data.mode) {
|
glsl: Rework interface block linking.
Previously, when doing intrastage and interstage interface block
linking, we only checked the interface type; this prevented us from
catching some link errors.
We now check the following additional constraints:
- For intrastage linking, the presence/absence of interface names must
match.
- For shader ins/outs, the interface names themselves must match when
doing intrastage linking (note: it's not clear from the spec whether
this is necessary, but Mesa's implementation currently relies on
it).
- Array vs. nonarray must be consistent, taking into account the
special rules for vertex-geometry linkage.
- Array sizes must be consistent (exception: during intrastage
linking, an unsized array matches a sized array).
Note: validate_interstage_interface_blocks currently handles both
uniforms and in/out variables. As a result, if all three shader types
are present (VS, GS, and FS), and a uniform interface block is
mentioned in the VS and FS but not the GS, it won't be validated. I
plan to address this in later patches.
Fixes the following piglit tests in spec/glsl-1.50/linker:
- interface-blocks-vs-fs-array-size-mismatch
- interface-vs-array-to-fs-unnamed
- interface-vs-unnamed-to-fs-array
- intrastage-interface-unnamed-array
v2: Simplify logic in intrastage_match() for handling array sizes.
Make extra_array_level const. Use an unnamed temporary
interface_block_definition in validate_interstage_interface_blocks()'s
first call to definitions->store().
Cc: "10.0" <mesa-stable@lists.freedesktop.org>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>
2013-10-29 14:41:32 -07:00
|
|
|
case ir_var_shader_in:
|
|
|
|
|
definitions = &in_interfaces;
|
|
|
|
|
break;
|
|
|
|
|
case ir_var_shader_out:
|
|
|
|
|
definitions = &out_interfaces;
|
|
|
|
|
break;
|
|
|
|
|
case ir_var_uniform:
|
|
|
|
|
definitions = &uniform_interfaces;
|
|
|
|
|
break;
|
2015-05-13 11:17:23 +02:00
|
|
|
case ir_var_shader_storage:
|
|
|
|
|
definitions = &buffer_interfaces;
|
|
|
|
|
break;
|
glsl: Rework interface block linking.
Previously, when doing intrastage and interstage interface block
linking, we only checked the interface type; this prevented us from
catching some link errors.
We now check the following additional constraints:
- For intrastage linking, the presence/absence of interface names must
match.
- For shader ins/outs, the interface names themselves must match when
doing intrastage linking (note: it's not clear from the spec whether
this is necessary, but Mesa's implementation currently relies on
it).
- Array vs. nonarray must be consistent, taking into account the
special rules for vertex-geometry linkage.
- Array sizes must be consistent (exception: during intrastage
linking, an unsized array matches a sized array).
Note: validate_interstage_interface_blocks currently handles both
uniforms and in/out variables. As a result, if all three shader types
are present (VS, GS, and FS), and a uniform interface block is
mentioned in the VS and FS but not the GS, it won't be validated. I
plan to address this in later patches.
Fixes the following piglit tests in spec/glsl-1.50/linker:
- interface-blocks-vs-fs-array-size-mismatch
- interface-vs-array-to-fs-unnamed
- interface-vs-unnamed-to-fs-array
- intrastage-interface-unnamed-array
v2: Simplify logic in intrastage_match() for handling array sizes.
Make extra_array_level const. Use an unnamed temporary
interface_block_definition in validate_interstage_interface_blocks()'s
first call to definitions->store().
Cc: "10.0" <mesa-stable@lists.freedesktop.org>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>
2013-10-29 14:41:32 -07:00
|
|
|
default:
|
|
|
|
|
/* Only in, out, and uniform interfaces are legal, so we should
|
|
|
|
|
* never get here.
|
|
|
|
|
*/
|
|
|
|
|
assert(!"illegal interface type");
|
|
|
|
|
continue;
|
|
|
|
|
}
|
2013-05-20 23:42:49 -07:00
|
|
|
|
2015-12-02 17:53:19 +11:00
|
|
|
ir_variable *prev_def = definitions->lookup(var);
|
glsl: Rework interface block linking.
Previously, when doing intrastage and interstage interface block
linking, we only checked the interface type; this prevented us from
catching some link errors.
We now check the following additional constraints:
- For intrastage linking, the presence/absence of interface names must
match.
- For shader ins/outs, the interface names themselves must match when
doing intrastage linking (note: it's not clear from the spec whether
this is necessary, but Mesa's implementation currently relies on
it).
- Array vs. nonarray must be consistent, taking into account the
special rules for vertex-geometry linkage.
- Array sizes must be consistent (exception: during intrastage
linking, an unsized array matches a sized array).
Note: validate_interstage_interface_blocks currently handles both
uniforms and in/out variables. As a result, if all three shader types
are present (VS, GS, and FS), and a uniform interface block is
mentioned in the VS and FS but not the GS, it won't be validated. I
plan to address this in later patches.
Fixes the following piglit tests in spec/glsl-1.50/linker:
- interface-blocks-vs-fs-array-size-mismatch
- interface-vs-array-to-fs-unnamed
- interface-vs-unnamed-to-fs-array
- intrastage-interface-unnamed-array
v2: Simplify logic in intrastage_match() for handling array sizes.
Make extra_array_level const. Use an unnamed temporary
interface_block_definition in validate_interstage_interface_blocks()'s
first call to definitions->store().
Cc: "10.0" <mesa-stable@lists.freedesktop.org>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>
2013-10-29 14:41:32 -07:00
|
|
|
if (prev_def == NULL) {
|
2013-05-20 23:42:49 -07:00
|
|
|
/* This is the first time we've seen the interface, so save
|
glsl: Rework interface block linking.
Previously, when doing intrastage and interstage interface block
linking, we only checked the interface type; this prevented us from
catching some link errors.
We now check the following additional constraints:
- For intrastage linking, the presence/absence of interface names must
match.
- For shader ins/outs, the interface names themselves must match when
doing intrastage linking (note: it's not clear from the spec whether
this is necessary, but Mesa's implementation currently relies on
it).
- Array vs. nonarray must be consistent, taking into account the
special rules for vertex-geometry linkage.
- Array sizes must be consistent (exception: during intrastage
linking, an unsized array matches a sized array).
Note: validate_interstage_interface_blocks currently handles both
uniforms and in/out variables. As a result, if all three shader types
are present (VS, GS, and FS), and a uniform interface block is
mentioned in the VS and FS but not the GS, it won't be validated. I
plan to address this in later patches.
Fixes the following piglit tests in spec/glsl-1.50/linker:
- interface-blocks-vs-fs-array-size-mismatch
- interface-vs-array-to-fs-unnamed
- interface-vs-unnamed-to-fs-array
- intrastage-interface-unnamed-array
v2: Simplify logic in intrastage_match() for handling array sizes.
Make extra_array_level const. Use an unnamed temporary
interface_block_definition in validate_interstage_interface_blocks()'s
first call to definitions->store().
Cc: "10.0" <mesa-stable@lists.freedesktop.org>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>
2013-10-29 14:41:32 -07:00
|
|
|
* it into the appropriate data structure.
|
2013-05-20 23:42:49 -07:00
|
|
|
*/
|
2015-12-02 17:53:19 +11:00
|
|
|
definitions->store(var);
|
2019-04-23 16:52:36 +02:00
|
|
|
} else if (!intrastage_match(prev_def, var, prog,
|
|
|
|
|
true /* match_precision */)) {
|
2013-07-27 14:58:43 -07:00
|
|
|
linker_error(prog, "definitions of interface block `%s' do not"
|
2023-09-12 12:11:18 -07:00
|
|
|
" match\n", glsl_get_type_name(iface_type));
|
2013-07-27 11:08:31 -07:00
|
|
|
return;
|
2013-05-20 23:42:49 -07:00
|
|
|
}
|
|
|
|
|
}
|
|
|
|
|
}
|
|
|
|
|
}
|