On Fri, Aug 28, 2015 at 12:51 PM, Neil Roberts <n...@linux.intel.com> wrote: > Ilia Mirkin <imir...@alum.mit.edu> writes: > >> @@ -2757,6 +2764,11 @@ builtin_builder::add_image_functions(bool glsl) >> "__intrinsic_image_size", >> &builtin_builder::_image_size_prototype, 1, >> flags | IMAGE_FUNCTION_SUPPORTS_FLOAT_DATA_TYPE); >> + >> + add_image_function(glsl ? "imageSamples" : "__intrinsic_image_samples", >> + "__intrinsic_image_samples", >> + &builtin_builder::_image_samples_prototype, 1, >> + flags | IMAGE_FUNCTION_MS_ONLY); > > I think this needs IMAGE_FUNCTION_SUPPORTS_FLOAT_DATA_TYPE as well. > Otherwise it won't get a declaration for imageSamples(image2DMS), right?
Yes, I think you're right. I'm going to make some compiler piglits that test it all to double-check. -ilia _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
