Re: [Mesa-dev] [PATCH 1/2] clover: Added missing address space checking of kernel parameters

2013-07-24 Thread Jonathan Charest
ld_module_llvm(mod, kernels); + return build_module_llvm(mod, kernels, address_spaces); } } -- 1.8.3.3 On 2013-07-24 03:58, Francisco Jerez wrote: > Tom Stellard writes: > >> On Mon, Jul 22, 2013 at 09:24:12AM -0400, Jonathan Charest wrote: >>> To have non-static buf

[Mesa-dev] [PATCH 1/2] clover: Added missing address space checking of kernel parameters

2013-07-22 Thread Jonathan Charest
To have non-static buffers in local memory, it is necessary to pass them as arguments to the kernel. This was almost supported but the address space mapping was missing to perform the check (thanks to tstellar for pointing me in the right direction). --- .../state_trackers/clover/llvm/invocation.

[Mesa-dev] [PATCH 2/2] Added missing address space checking of kernel parameters

2013-07-22 Thread Jonathan Charest
To have non-static buffers in local memory, it is necessary to pass them as arguments to the kernel. For r600, the correct lds size must be set to the SQ_LDS_ALLOC register. The correct size is the clover size plus the size reported by the compiler. --- src/gallium/drivers/r600/evergreen_compute