On Mon, Apr 16, 2018 at 7:36 PM, Tapani Pälli <tapani.pa...@intel.com> wrote:
> Hi;
>
> On 16.04.2018 18:19, Ilia Mirkin wrote:
>>
>> You never check EXT_texture_norm16 anywhere... is that right? Should
>> probably respect that in the _mesa_is_es3_* helpers. Also what about
>
>
> Oops that is very correct, will add the checks.
>
>> shader support? I see no mention in the spec that these should be
>> usable as images, except with NV_image_formats is enabled. You're just
>> exposing them no matter what in ES 3.10.
>
>
> Layout qualifiers added by the spec are exposed only when 3.10 ES and
> NV_image_formats are both there, the boolean in map below (last column) is
> the check for NV_image_formats, without it you'll get compilation error.
That's not quite right:
if ((state->is_version(map[i].required_glsl,
map[i].required_essl) ||
(state->NV_image_formats_enable &&
map[i].nv_image_formats)) &&
So you'd need to leave the essl version alone, otherwise
NV_image_formats check has no effect. Also, do you plan on
differentiating the case "have NV_image_formats but not
EXT_texture_norm16"?
Sounds like you need to add some negative compiler tests.
-ilia
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
