Re: [Beignet] [PATCH 09/10] Backend: Full support workgroup reduce, scan inc/exc on DWORD and bellow datatypes

2016-04-06 Thread Xiuli Pan
On Thu, Mar 31, 2016 at 06:28:33PM +0300, grigore.lupe...@intel.com wrote: > From: Grigore Lupescu > > Signed-off-by: Grigore Lupescu > --- > backend/src/backend/gen_context.cpp | 329 > > 1 file changed, 180 insertions(+), 149 deletions(-) > > diff --git

Re: [Beignet] [PATCH] runtime: handle PROGRAM_BUILD_GLOBAL_VARIABLE_TOTAL_SIZE

2016-04-06 Thread Pan, Xiuli
LGTM! This is more considerate than my version. -Original Message- From: Beignet [mailto:beignet-boun...@lists.freedesktop.org] On Behalf Of Ruiling Song Sent: Wednesday, April 6, 2016 2:29 PM To: beignet@lists.freedesktop.org Cc: Song, Ruiling Subject: [Beignet]

Re: [Beignet] [PATCH OCL20] Runtime: Add new param_name to clGetProgramBuildInfo

2016-04-06 Thread Song, Ruiling
Hi Xiuli, I have made such modification in my local machine. I have just send it out to the mail list. I think we need to make a small wrapper for interp_program_get_global_constant_size(program->opaque); And I also add a minor check against is_built to make sure the program was built

[Beignet] [PATCH] runtime: handle PROGRAM_BUILD_GLOBAL_VARIABLE_TOTAL_SIZE

2016-04-06 Thread Ruiling Song
Signed-off-by: Ruiling Song --- src/cl_api.c | 5 + src/cl_program.c | 5 + src/cl_program.h | 2 ++ 3 files changed, 12 insertions(+) diff --git a/src/cl_api.c b/src/cl_api.c index 131a84c..161ee46 100644 --- a/src/cl_api.c +++ b/src/cl_api.c @@ -1456,6