[Mesa-dev] [PATCH 1/2] mesa/texformat: enable ASTC formats for TexStorage
nanleychery at gmail.com
Wed Aug 19 15:31:13 PDT 2015
On Mon, Aug 17, 2015 at 11:54 AM, Chad Versace <chad.versace at intel.com>
> On Tue 28 Jul 2015, Nanley Chery wrote:
> > From: Nanley Chery <nanley.g.chery at intel.com>
> > According to the spec, Tex*Storage* commands are supported if the GL
> > implementation has support for immutable textures.
> > Signed-off-by: Nanley Chery <nanley.g.chery at intel.com>
> > ---
> > src/mesa/main/texformat.c | 56
> > 1 file changed, 56 insertions(+)
> This patch's code looks correct to me, but I'm confused by the commit
> message. The commit message specifically calls out the glTexStorage*
> functions, but _mesa_choose_tex_format() is also used by many other
> non-storage glTex* functions.
> Why specifically mention the glTexStorage* function? Is there some
> larger context I'm missing?
I called out the glTex*Storage* functions because they were the only
commands that wouldn't work correctly during testing. Perhaps I missed some
cases (e.g., TextureViews). This patch is superseded by the following patch
(which needs review): "mesa/texformat: Use format conversion function in
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the mesa-dev