Re: [Mesa-dev] [PATCH] radeonsi: use minnum and maxnum LLVM intrinsics for MIN and MAX opcodes

2014-11-22 Thread Marek Olšák
AFAICS, the R600 backend doesn't implement the intrinsics for R600. Marek On Sat, Nov 22, 2014 at 3:53 AM, Michel Dänzer mic...@daenzer.net wrote: On 21.11.2014 06:21, Marek Olšák wrote: From: Marek Olšák marek.ol...@amd.com So far it has been compiled into pretty ugly code (8 instructions

Re: [Mesa-dev] [PATCH] radeonsi: use minnum and maxnum LLVM intrinsics for MIN and MAX opcodes

2014-11-22 Thread Matt Arsenault
On Nov 22, 2014, at 7:35 AM, Marek Olšák mar...@gmail.com wrote: AFAICS, the R600 backend doesn't implement the intrinsics for R600. Marek Should it? It’s trivial to switch to these for it, but I wasn’t sure what the actual semantics of its instructions were. There’s MAX and MAX_DX10,

Re: [Mesa-dev] [PATCH] radeonsi: use minnum and maxnum LLVM intrinsics for MIN and MAX opcodes

2014-11-22 Thread Marek Olšák
The R600 LLVM support for graphics is disabled by default and is missing some features. Unless somebody implements the missing features, discussing how each TGSI instruction is compiled is fairly low priority. We should rather be asking whether that project is really dead. It's sad but true. For

Re: [Mesa-dev] [PATCH] radeonsi: use minnum and maxnum LLVM intrinsics for MIN and MAX opcodes

2014-11-21 Thread Michel Dänzer
On 21.11.2014 06:21, Marek Olšák wrote: From: Marek Olšák marek.ol...@amd.com So far it has been compiled into pretty ugly code (8 instructions or so for either opcode). --- src/gallium/drivers/radeonsi/si_shader.c | 7 +++ 1 file changed, 7 insertions(+) diff --git

[Mesa-dev] [PATCH] radeonsi: use minnum and maxnum LLVM intrinsics for MIN and MAX opcodes

2014-11-20 Thread Marek Olšák
From: Marek Olšák marek.ol...@amd.com So far it has been compiled into pretty ugly code (8 instructions or so for either opcode). --- src/gallium/drivers/radeonsi/si_shader.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/src/gallium/drivers/radeonsi/si_shader.c

Re: [Mesa-dev] [PATCH] radeonsi: use minnum and maxnum LLVM intrinsics for MIN and MAX opcodes

2014-11-20 Thread Tom Stellard
On Thu, Nov 20, 2014 at 10:21:07PM +0100, Marek Olšák wrote: From: Marek Olšák marek.ol...@amd.com Reviewed-by: Tom Stellard thomas.stell...@amd.com So far it has been compiled into pretty ugly code (8 instructions or so for either opcode). --- src/gallium/drivers/radeonsi/si_shader.c |