[Mesa-dev] [PATCH v2 0/5] panfrost: Add support for TXS instructions
Boris Brezillon
boris.brezillon at collabora.com
Tue Jun 18 07:38:45 UTC 2019
Hello,
TXS is needed to support the OpenGL textureSize() function but not only.
TXS instructions are also used by nir_lower_tex() to lower a RECT
texture sampling into a 2D one, which I wanted to have working to test
gallium-hud on panfrost.
Note that I decided to add a new generic lowering step to
nir_lower_tex() to lower a TXS(LOD) instruction into
max(1, TXS(0) >> LOD), which is what V3D is doing internally and would
be duplicated in the panfrost driver if we go for a non-generic
approach (see patch 2 for more details).
Let me know if you think that's preferable to have this logic moved to
the panfrost driver.
Regards,
Boris
Changes in v2:
* Drop the patch adding nir_imm_ivec3()
* Rework the emit_tex() logic to avoid duplicate the switch(texop)
statement
* Split options for the first and second tex lowering passes
Boris Brezillon (5):
nir/lower_tex: Add a way to lower TXS(non-0-LOD) instructions
panfrost: Make the sysval logic more generic
panfrost: Move sysval upload logic out of panfrost_emit_for_draw()
panfrost: Prepare things to support non-native texture ops
panfrost: Add support for TXS instructions
src/compiler/nir/nir.h | 6 +
src/compiler/nir/nir_lower_tex.c | 46 +++++++
.../panfrost/midgard/midgard_compile.c | 125 ++++++++++++------
.../panfrost/midgard/midgard_compile.h | 12 +-
src/gallium/drivers/panfrost/pan_context.c | 96 +++++++++++---
5 files changed, 226 insertions(+), 59 deletions(-)
--
2.20.1
More information about the mesa-dev
mailing list