Re: [Mesa-dev] [PATCH] radeonsi: Do not set both inreg and byval

2013-10-20 Thread Marek Olšák
Reviewed-by: Marek Olšák BTW, the LLVM changes must land first. Marek On Fri, Oct 11, 2013 at 5:32 PM, Vincent Lejeune wrote: > --- > src/gallium/drivers/radeonsi/radeonsi_shader.c | 9 ++--- > 1 file changed, 6 insertions(+), 3 deletions(-) > > diff --git a/src/gallium/drivers/radeonsi/r

[Mesa-dev] [PATCH] radeonsi: Do not set both inreg and byval

2013-10-11 Thread Vincent Lejeune
--- src/gallium/drivers/radeonsi/radeonsi_shader.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/src/gallium/drivers/radeonsi/radeonsi_shader.c b/src/gallium/drivers/radeonsi/radeonsi_shader.c index ab996cc..209b77e 100644 --- a/src/gallium/drivers/radeonsi/radeonsi

Re: [Mesa-dev] [PATCH] radeonsi: Do not set both inreg and byval

2013-10-10 Thread Marek Olšák
Will the byval function arguments break the driver with LLVM 3.3? Marek On Thu, Oct 10, 2013 at 12:34 AM, Vincent Lejeune wrote: > --- > src/gallium/drivers/radeonsi/radeonsi_shader.c | 7 --- > 1 file changed, 4 insertions(+), 3 deletions(-) > > diff --git a/src/gallium/drivers/radeonsi/ra

[Mesa-dev] [PATCH] radeonsi: Do not set both inreg and byval

2013-10-09 Thread Vincent Lejeune
--- src/gallium/drivers/radeonsi/radeonsi_shader.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/gallium/drivers/radeonsi/radeonsi_shader.c b/src/gallium/drivers/radeonsi/radeonsi_shader.c index ab996cc..9d95997 100644 --- a/src/gallium/drivers/radeonsi/radeonsi_s