Re: [Mesa-dev] [PATCH 01/11] gallium: define the maximum number of shader arrays

2015-05-26 Thread Marek Olšák
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

Re: [Mesa-dev] [PATCH 01/11] gallium: define the maximum number of shader arrays

2015-05-25 Thread Marek Olšák
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

Re: [Mesa-dev] [PATCH 01/11] gallium: define the maximum number of shader arrays

2015-05-25 Thread Roland Scheidegger
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

[Mesa-dev] [PATCH 01/11] gallium: define the maximum number of shader arrays

2015-05-24 Thread Marek Olšák
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