v3d/tex: don't configure tmu config 1 if not needed
authorAlejandro Piñeiro <apinheiro@igalia.com>
Mon, 13 Apr 2020 09:45:27 +0000 (11:45 +0200)
committerAlejandro Piñeiro <apinheiro@igalia.com>
Wed, 22 Apr 2020 21:38:18 +0000 (23:38 +0200)
commitd0b644d9f9d9673d3fe28c8c200209f553adeda1
treeb67495fe41ca08c0aa34312dc20232a3ecc2a1e6
parentc552b5fd1d106adc04f62fcbe71d650a9a17f7e0
v3d/tex: don't configure tmu config 1 if not needed

TMU configuration parameter 1 configures the sampler for the texture
operation. But there are some texture operations that doesn't need a
sampler. Skipping the configuration could provide a small perf
improvement on OpenGL. On the incoming Vulkan driver, would allow us
to avoid to set up an unneeded sampler.

Note that we still need to add the sampler configuration parameter if
the output is a 32bit, as it is on the sampler where we configure that
info.

Also, note that for images this is done comparing against a unpacked
p1 default. But in order to do that it is needed to go through the
code that fills up the unpacked p1. We can skip that too.

Reviewed-by: Eric Anholt <eric@anholt.net>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/4677>
src/broadcom/compiler/v3d40_tex.c