The commit message should be prefixed "glsl:", this fix isn't i965-specific.
Martin Peres <[email protected]> writes: > This got missed because the piglit test only tested int images to avoid a > combinatiorial explosion of format, targets, stages and sizes which > takes more than 5 minutes to test on nvidia's driver. > > This patch also drops the IMAGE_FUNCTION_AVAIL_ATOMIC which is not applicable > to the image_size codepath but was not hurting in any way. > Well, it would have caused the intrinsic not to be exposed on unextended GLES 3.1, which is definitely a bug. :P > Signed-off-by: Martin Peres <[email protected]> > --- > src/glsl/builtin_functions.cpp | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/src/glsl/builtin_functions.cpp b/src/glsl/builtin_functions.cpp > index 05dbe2e..0308aa6 100644 > --- a/src/glsl/builtin_functions.cpp > +++ b/src/glsl/builtin_functions.cpp > @@ -2695,7 +2695,8 @@ builtin_builder::add_image_functions(bool glsl) > > add_image_function(glsl ? "imageSize" : "__intrinsic_image_size", > "__intrinsic_image_size", > - &builtin_builder::_image_size_prototype, 1, > atom_flags); > + &builtin_builder::_image_size_prototype, 1, > + flags | IMAGE_FUNCTION_SUPPORTS_FLOAT_DATA_TYPE); Reviewed-by: Francisco Jerez <[email protected]> > } > > ir_variable * > -- > 2.5.0 > > _______________________________________________ > mesa-dev mailing list > [email protected] > http://lists.freedesktop.org/mailman/listinfo/mesa-dev
signature.asc
Description: PGP signature
_______________________________________________ mesa-dev mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/mesa-dev
