Comments at the end (where they are supposed to be :).
On 06/17/2011 05:54 PM, Dan McCabe wrote:
There are three changes from the patch set I published on 6/15:
1) Removed IR pointers from AST classes and moved them to
glsl_parser_state,
2) Manage the new IR in a stack-like manner to properly h
On 06/19/2011 02:21 PM, Eric Anholt wrote:
On Sat, 18 Jun 2011 09:29:32 +0200, Keith Packard wrote:
On Fri, 17 Jun 2011 17:43:14 -0700, Dan McCabe wrote:
break; // implicit exit from loop at end of switch
} while (true);
Seems like this could just be
} whil
On Sat, 18 Jun 2011 09:29:32 +0200, Keith Packard wrote:
> On Fri, 17 Jun 2011 17:43:14 -0700, Dan McCabe wrote:
>
> > break; // implicit exit from loop at end of switch
> > } while (true);
>
> Seems like this could just be
>
> } while (false);
>
> as I
On Fri, 17 Jun 2011 17:43:14 -0700, Dan McCabe wrote:
> break; // implicit exit from loop at end of switch
> } while (true);
Seems like this could just be
} while (false);
as I don't see any way the loop could go around more than once.
--
keith.pack...@intel.com
There are three changes from the patch set I published on 6/15:
1) Removed IR pointers from AST classes and moved them to glsl_parser_state,
2) Manage the new IR in a stack-like manner to properly handle nesting, and
3) Squash 6/7 and 7/7 together, since both deal with IR generation
#1 was in res
This patch set adds support for switch statements to the GLSL compiler. We
modify the grammar for the compiler with productions for switch statements
and case labels, while adding supporting supporting productions not already
present. New AST classes are defined to support those productions. Howev