search

KhronosGroup / OpenGL-Registry

OpenGL, OpenGL ES, and OpenGL ES-SC API and Extension Registry
679 stars 274 forks source link

Finish adding GL_EXT_texture_storage to big-GL #484

Closed nwnk closed 2 years ago

nwnk commented 2 years ago

The spec text says this is defined even for GL 1.2, but we don't have a GL extension number for it and the headers don't mention it. Fix that so we can extend immutable-format texture image support to GL < 4.2.

nwnk commented 2 years ago

I guess ARB_texture_storage does exist, so possibly a better move is to remove the big-gl reference from EXT?

dgkoch commented 2 years ago

I guess ARB_texture_storage does exist, so possibly a better move is to remove the big-gl reference from EXT?

And the ARB version has ES interactions.. They look like 90% or more similar.. I forget the history, but I guess we didn't manage to get the GL and ES groups to quite agree to promote it at the same time, so we couldn't end up making it KHR and having one spec.

Maybe we should just add a note to both and say for ES use the EXT, and for GL use the ARB?

oddhack commented 2 years ago

Maybe we should just add a note to both and say for ES use the EXT, and for GL use the ARB?

Might be worth checking gpuinfo to see if drivers don't comply with that approach.

There are a couple of other extensions with similar disclaimers, IIRC. Before the ARB merged into Khronos, there was no way to have an extension approved by both the Khronos ES WG and the ARB.

dgkoch commented 2 years ago

It actually looks like NV/AMD/Intel expose GL_EXT_texture_storage in desktop drivers, so I guess the proposed change here is doing the right thing.

pdaniell-nv commented 2 years ago

This is approved to merge, but @oddhack will probably want this PR to be updated to not include the generated files.