On 26 November 2013 00:02, Francisco Jerez <curroje...@riseup.net> wrote:
> ---
> src/glsl/glsl_parser.yy | 83
> +++++++++++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 83 insertions(+)
>
> diff --git a/src/glsl/glsl_parser.yy b/src/glsl/glsl_parser.yy
> index e436b16..e207510 100644
> --- a/src/glsl/glsl_parser.yy
> +++ b/src/glsl/glsl_parser.yy
> @@ -1298,6 +1298,63 @@ layout_qualifier_id:
> /* Layout qualifiers for ARB_shader_image_load_store. */
> if (state->ARB_shader_image_load_store_enable ||
> state->is_version(420, 0)) {
> + if (!$$.flags.i) {
> + struct {
> + const char *name;
> + GLenum format;
> + glsl_base_type base_type;
> + } map[] = {
>
Let's make this array static const so that the compiler doesn't generate
code to initialize it every time this code path is taken.
With that changed, this patch is:
Reviewed-by: Paul Berry <stereotype...@gmail.com>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
