Am 14.06.2014 03:46, schrieb Marek Olšák:
From: Marek Olšák <marek.ol...@amd.com>

This was wrong for a very long time. I wonder if the array size has any
effect on anything.

We only do a bit of GEP pointer arithmetic with them and so probably ignore range limits anyway.

Nevertheless it's obviously wrong and should be fixed. Patch is Reviewed-by: Christian König <christian.koe...@amd.com>

Regards,
Christian.

---
  src/gallium/drivers/radeonsi/si_shader.c | 4 ++--
  1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/gallium/drivers/radeonsi/si_shader.c 
b/src/gallium/drivers/radeonsi/si_shader.c
index a7ca35b..4ed5906 100644
--- a/src/gallium/drivers/radeonsi/si_shader.c
+++ b/src/gallium/drivers/radeonsi/si_shader.c
@@ -2266,9 +2266,9 @@ static void create_function(struct si_shader_context 
*si_shader_ctx)
        /* We assume at most 16 textures per program at the moment.
         * This need probably need to be changed to support bindless textures */
        params[SI_PARAM_SAMPLER] = LLVMPointerType(
-               LLVMArrayType(LLVMVectorType(i8, 16), NUM_SAMPLER_VIEWS), 
CONST_ADDR_SPACE);
+               LLVMArrayType(LLVMVectorType(i8, 16), NUM_SAMPLER_STATES), 
CONST_ADDR_SPACE);
        params[SI_PARAM_RESOURCE] = LLVMPointerType(
-               LLVMArrayType(LLVMVectorType(i8, 32), NUM_SAMPLER_STATES), 
CONST_ADDR_SPACE);
+               LLVMArrayType(LLVMVectorType(i8, 32), NUM_SAMPLER_VIEWS), 
CONST_ADDR_SPACE);
switch (si_shader_ctx->type) {
        case TGSI_PROCESSOR_VERTEX:

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to