[Mesa-dev] [PATCH 01/10] mesa: add infra for ARB_shader_texture_image_samples

[Ilia Mirkin]

On Tue, Sep 8, 2015 at 11:46 PM, Ilia Mirkin <imirkin at alum.mit.edu> wrote:
> On Tue, Sep 8, 2015 at 11:38 PM, Ian Romanick <idr at freedesktop.org> wrote:
>> On 09/08/2015 08:17 PM, Ian Romanick wrote:
>>> On 09/08/2015 08:10 PM, Ilia Mirkin wrote:
>>>> I'll give it a shot on NVIDIA and see what happens... my tests might
>>>> require GL 3.2 because I didn't want to deal with the various
>>>> differences that happen otherwise.
>>>
>>> That should be good enough.  Perhaps someone else can try on AMD
>>> closed-source drivers.  If they both ignore the spec, then I feel pretty
>>> comfortable ignoring it too... while I work on getting it changed.
>>
>> I have attached the spec changes that I am proposing.
>
> This drops the dependence on images, which means that gallium drivers
> can get this straight away. From a "allow the extension to run on as
> many devices as possible" standpoint, this makes sense. There's an
> alternate viewpoint which is that this should perhaps require *both*
> images and ARB_texture_multisample (without requiring any specific
> GLSL version).
>
> I've only looked at things from the driver writer's standpoint, i.e.
> "make as many exts available as possible", so I'm definitely in favor
> :) But I could also easily live with leaving the images requirement in
> there.

BTW, I knew something felt off about your change... I just figured out
what -- there is no GLSL 3.20 :) There's a GL 3.2 and a GLSL 1.50.

  -ilia