<div dir="ltr">On 26 November 2013 00:02, Francisco Jerez <span dir="ltr"><<a href="mailto:currojerez@riseup.net" target="_blank">currojerez@riseup.net</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">---<br>
 src/glsl/glsl_parser.yy | 83 +++++++++++++++++++++++++++++++++++++++++++++++++<br>
 1 file changed, 83 insertions(+)<br>
<br>
diff --git a/src/glsl/glsl_parser.yy b/src/glsl/glsl_parser.yy<br>
index e436b16..e207510 100644<br>
--- a/src/glsl/glsl_parser.yy<br>
+++ b/src/glsl/glsl_parser.yy<br>
@@ -1298,6 +1298,63 @@ layout_qualifier_id:<br>
       /* Layout qualifiers for ARB_shader_image_load_store. */<br>
       if (state->ARB_shader_image_load_store_enable ||<br>
           state->is_version(420, 0)) {<br>
+         if (!$$.flags.i) {<br>
+            struct {<br>
+               const char *name;<br>
+               GLenum format;<br>
+               glsl_base_type base_type;<br>
+            } map[] = {<br></blockquote><div><br></div><div>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.<br><br></div><div>With that changed, this patch is:<br>
<br>Reviewed-by: Paul Berry <<a href="mailto:stereotype441@gmail.com">stereotype441@gmail.com</a>><br></div></div></div></div>