glsl: Silence the last shift/reduce conflict warning in the grammar.
authorKenneth Graunke <kenneth@whitecape.org>
Sat, 13 Jul 2013 06:10:14 +0000 (23:10 -0700)
committerKenneth Graunke <kenneth@whitecape.org>
Tue, 16 Jul 2013 18:31:58 +0000 (11:31 -0700)
commit73620709c9dce79e58bd0d29be95c658ac95779b
tree4be0f3c885b865e61516f81031fe7a4a249c88c4
parentfa7829c36b78b8ecc42238cbc0a02d1059320c77
glsl: Silence the last shift/reduce conflict warning in the grammar.

The single remaining shift/reduce conflict was the classic ELSE problem:

  292 selection_rest_statement: statement . ELSE statement
  293                         | statement .

    ELSE  shift, and go to state 479

    ELSE      [reduce using rule 293 (selection_rest_statement)]
    $default  reduce using rule 293 (selection_rest_statement)

The correct behavior here is to shift, which is what happens by default.
However, resolving it explicitly will make it possible to fail the build
on new errors, making them much easier to detect.

The classic way to solve this is to use right associativity:
http://www.gnu.org/software/bison/manual/html_node/Non-Operators.html

Since there is no THEN token in GLSL, we need to fake one.  %right THEN
creates a new terminal symbol; the %prec directive says to use the
precedence of that terminal.

Signed-off-by: Kenneth Graunke <kenneth@whitecape.org>
Reviewed-by: Chris Forbes <chrisf@ijw.co.nz>
src/glsl/glsl_parser.yy