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

glsl: Parse non-array uniform block instance names in GLSL ES 3.00.

Browse Source 
In GLSL ES 3.00 (and GLSL 1.50), uniform blocks can have an associated
"instance name", which essentially namespaces the variables inside.

This patch adds basic parsing for this new feature, but doesn't yet hook
it up to actually do anything yet.

It does not support for arrays of interface blocks; a later commit will
take care of that.

This change temporarily regresses the piglit test
interface-name-access-without-interface-name.vert.  This shader failed
to compile before (the expected result), but it failed to compile for
the wrong reason.  This is not a real regression.

v2: Add some comments to ast_uniform_block::instance_name.  Suggested by
Paul Berry.

Reviewed-by: Carl Worth <cworth@cworth.org>
Reviewed-by: Chad Versace <chad.versace@linux.intel.com>
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
This commit is contained in:
Kenneth Graunke
2012-11-27 23:45:17 -08:00
committed by Ian Romanick
parent 0d2e6336a2
commit 34f966bdcb
2 changed files with 30 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
src/glsl/ast.h
View File

@@ -805,8 +805,9 @@ class ast_uniform_block : public ast_node {
public:
ast_uniform_block(ast_type_qualifier layout,
const char *block_name,
ast_declarator_list *member_list)
: layout(layout), block_name(block_name)
ast_declarator_list *member_list,
const char *instance_name)
: layout(layout), block_name(block_name), instance_name(instance_name)
{
declarations.push_degenerate_list_at_head(&member_list->link);
}
@@ -816,6 +817,15 @@ public:
ast_type_qualifier layout;
const char *block_name;
/**
* Declared name of the block instance, if specified.
*
* If the block does not have an instance name, this field will be
* \c NULL.
*/
const char *instance_name;
/** List of ast_declarator_list * */
exec_list declarations;
};

20
src/glsl/glsl_parser.yy
View File

@@ -112,6 +112,7 @@ static void yyerror(YYLTYPE *loc, _mesa_glsl_parse_state *st, const char *msg)
%token STRUCT VOID_TOK WHILE
%token <identifier> IDENTIFIER TYPE_IDENTIFIER NEW_IDENTIFIER
%type <identifier> any_identifier
%type <identifier> instance_name_opt
%token <real> FLOATCONSTANT
%token <n> INTCONSTANT UINTCONSTANT BOOLCONSTANT
%token <identifier> FIELD_SELECTION
@@ -1899,11 +1900,11 @@ uniform_block:
;
basic_uniform_block:
UNIFORM NEW_IDENTIFIER '{' member_list '}' ';'
UNIFORM NEW_IDENTIFIER '{' member_list '}' instance_name_opt ';'
{
void *ctx = state;
$$ = new(ctx) ast_uniform_block(*state->default_uniform_qualifier,
$2, $4);
$2, $4, $6);
if (!state->ARB_uniform_buffer_object_enable) {
_mesa_glsl_error(& @1, state,
@@ -1917,6 +1918,21 @@ basic_uniform_block:
}
;
instance_name_opt:
/* empty */
{
$$ = NULL;
}
| NEW_IDENTIFIER
{
if (!(state->language_version == 300 && state->es_shader)) {
_mesa_glsl_error(& @1, state,
"#version 300 es required for using uniform "
"blocks with an instance name\n");
}
}
;
member_list:
member_declaration
{