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

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