I'm dropping this patch. I've also sent a new version of patch 2 as
well as new patches that dynamically allocate some of the arrays.
The complete patch series with all feedback comments resolved is also here:
http://cgit.freedesktop.org/~mareko/mesa/log/?h=tgsi-arrays
(starting from "glsl_to_tgsi
It makes it easier for the statically-sized arrays. Not that there is
anything good about statically-sized arrays.
The hard limit is 1022 arrays per file, because ArrayID has 10 bits.
Marek
On Mon, May 25, 2015 at 8:42 PM, Roland Scheidegger wrote:
> Do we really need this?
> I think ideally we
Do we really need this?
I think ideally we wouldn't really need any limit on this, as it is
already limited by the max number of the temps (though obviously that
number is higher). d3d10 doesn't seem to have any such limit neither.
If some driver can't do it, I think it's not unreasonable to expect
From: Marek Olšák
---
src/gallium/drivers/radeon/radeon_llvm.h| 3 +--
src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c | 4 ++--
src/gallium/include/pipe/p_state.h | 1 +
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 11 +++
4 files changed, 7