glsl: Require that all image uniforms have a format qualifier in GLSL ES.
authorFrancisco Jerez <currojerez@riseup.net>
Sun, 16 Aug 2015 22:26:40 +0000 (01:26 +0300)
committerFrancisco Jerez <currojerez@riseup.net>
Thu, 20 Aug 2015 09:26:56 +0000 (12:26 +0300)
commit527ae5d4286e76fc2ec3d70f4b6cea3798539372
tree6e4ec785bb3d489c478377068afb50ddfb3d43d6
parent241774aa03d6dda5fe4cd86c1988f1678d4c0e5f
glsl: Require that all image uniforms have a format qualifier in GLSL ES.

Note that this is slightly more permissive than the spec language
requires: "Any image variable must specify a format layout qualifier."

The GLSL ES spec seems really sketchy regarding format layout
qualifiers on function formal parameters -- On the one hand they are
required, but on the other hand it doesn't provide any syntax to
specify them (see section 6.1.1), they don't participate in parameter
type matching for overload resolution, and are in fact explictly
forbidden ("Layout qualifiers cannot be used on formal function
parameters").  Of course none of the image built-in functions defined
by the spec specify format layout qualifiers (and they probably
couldn't sensibly), to contradict its own requirement.

This probably qualifies for a spec bug, but in the meantime do the
sensible thing and require layout qualifiers on uniforms *only*.

Reviewed-by: Timothy Arceri <t_arceri@yahoo.com.au>
src/glsl/ast_to_hir.cpp