Re: [Beignet] [PATCH] GBE: Support storing/loading pointers to/from private array

2015-05-31 Thread Zhigang Gong
It seems that this patch cause one regression in the unit test cases. You can reproduce it as below: utests/utest_run compiler_local_slm Thanks, Zhigang Gong. On Thu, May 21, 2015 at 04:39:05PM +0800, Ruiling Song wrote: > The idea is create two additional array for holding > pointer-base and bt

Re: [Beignet] [PATCH 1/2] GBE: Fix a typo that cause gpu hang.

2015-05-31 Thread Zhigang Gong
The patchset LGTM, will push latter. Thanks, Zhigang Gong. On Mon, Jun 01, 2015 at 09:43:06AM +0800, Ruiling Song wrote: > Signed-off-by: Ruiling Song > --- > backend/src/backend/gen75_encoder.cpp | 2 +- > backend/src/backend/gen8_encoder.cpp | 2 +- > 2 files changed, 2 insertions(+), 2 dele

[Beignet] [PATCH 1/2] GBE: Fix a typo that cause gpu hang.

2015-05-31 Thread Ruiling Song
Signed-off-by: Ruiling Song --- backend/src/backend/gen75_encoder.cpp | 2 +- backend/src/backend/gen8_encoder.cpp | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/backend/src/backend/gen75_encoder.cpp b/backend/src/backend/gen75_encoder.cpp index 602f9c7..135be02 100644 -

[Beignet] [PATCH 2/2] GBE: add offsetID in SelectionVector.

2015-05-31 Thread Ruiling Song
SelectionVector may not start from zero. Signed-off-by: Ruiling Song --- backend/src/backend/gen_insn_selection.cpp | 16 backend/src/backend/gen_insn_selection.hpp | 2 ++ backend/src/backend/gen_reg_allocation.cpp | 2 +- 3 files changed, 19 insertions(+), 1 deletion(-) dif