[Beignet] [PATCH 2/2] add utest for __gen_ocl_get_simd_id

2015-04-15 Thread Guo Yejun
Signed-off-by: Guo Yejun --- kernels/compiler_get_simd_id.cl | 8 utests/CMakeLists.txt | 3 ++- utests/compiler_get_simd_id.cpp | 33 + 3 files changed, 43 insertions(+), 1 deletion(-) create mode 100644 kernels/compiler_get_simd_id.cl creat

[Beignet] [PATCH 1/2] add simd level function __gen_ocl_get_simd_id

2015-04-15 Thread Guo Yejun
uint __gen_ocl_get_simd_id(); return value ranges from 0 to simdsize - 1 Signed-off-by: Guo Yejun --- backend/src/backend/gen_context.cpp| 9 - backend/src/backend/gen_insn_selection.cpp | 8 backend/src/backend/program.h | 1 + backend/src/ir/instruction.cp

Re: [Beignet] [PATCH V4 1/2] add simd level function __gen_ocl_get_simd_size

2015-04-15 Thread Yang, Rong R
The patchset LGTM, thanks. > -Original Message- > From: Beignet [mailto:beignet-boun...@lists.freedesktop.org] On Behalf Of > Guo Yejun > Sent: Thursday, April 16, 2015 10:51 > To: beignet@lists.freedesktop.org > Cc: Guo, Yejun > Subject: [Beignet] [PATCH V4 1/2] add simd level function >

[Beignet] [PATCH V4 1/2] add simd level function __gen_ocl_get_simd_size

2015-04-15 Thread Guo Yejun
uint __gen_ocl_get_simd_size(); returns 8 if SIMD8, returns 16 if SIMD16 V2: add missing files remove some unnecessary functions V3: correct the dst register setting, it is possible not uniform V4: remove unnecessary function Signed-off-by: Guo Yejun --- backend/src/backend/gen_insn_selecti

Re: [Beignet] [PATCH V3 1/2] add simd level function __gen_ocl_get_simd_size

2015-04-15 Thread Guo, Yejun
Yes, thanks, will send out a new version -Original Message- From: Yang, Rong R Sent: Thursday, April 16, 2015 10:34 AM To: Guo, Yejun; beignet@lists.freedesktop.org Cc: Guo, Yejun Subject: RE: [Beignet] [PATCH V3 1/2] add simd level function __gen_ocl_get_simd_size +void ALU0(Select

Re: [Beignet] [PATCH V3 1/2] add simd level function __gen_ocl_get_simd_size

2015-04-15 Thread Yang, Rong R
+void ALU0(SelectionOpcode opcode, Reg dst); It can also remove. The others of the patchset LGTM. > -Original Message- > From: Beignet [mailto:beignet-boun...@lists.freedesktop.org] On Behalf Of > Guo Yejun > Sent: Thursday, April 16, 2015 08:50 > To: beignet@lists.freedesktop.org > Cc

[Beignet] [PATCH 2/2] add utest for __gen_ocl_get_simd_size

2015-04-15 Thread Guo Yejun
Signed-off-by: Guo Yejun --- kernels/compiler_get_simd_size.cl | 5 + utests/CMakeLists.txt | 3 ++- utests/compiler_get_simd_size.cpp | 32 3 files changed, 39 insertions(+), 1 deletion(-) create mode 100644 kernels/compiler_get_simd_size.cl c

[Beignet] [PATCH V3 1/2] add simd level function __gen_ocl_get_simd_size

2015-04-15 Thread Guo Yejun
uint __gen_ocl_get_simd_size(); returns 8 if SIMD8, returns 16 if SIMD16 V2: add missing files remove unnecessary function V3: correct the dst register setting, it is possible not uniform Signed-off-by: Guo Yejun --- backend/src/backend/gen_insn_selection.cpp | 35 ++