[Mesa-dev] [PATCH] i965: allow image_size on float images
Martin Peres
martin.peres at linux.intel.com
Fri Aug 21 07:47:48 PDT 2015
On 21/08/15 16:52, Francisco Jerez wrote:
> The commit message should be prefixed "glsl:", this fix isn't
> i965-specific.
>
> Martin Peres <martin.peres at linux.intel.com> 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
No, I had the same reaction at first, but I actually do not use
_image_prototype.
I use _image_size_prototype which does not handle this flag.
>
>> Signed-off-by: Martin Peres <martin.peres at linux.intel.com>
>> ---
>> 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 <currojerez at riseup.net>
Thanks, will push it now, before Emil pushes tags the first rc.
>
>> }
>>
>> ir_variable *
>> --
>> 2.5.0
>>
>> _______________________________________________
>> mesa-dev mailing list
>> mesa-dev at lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
More information about the mesa-dev
mailing list