[Beignet] [PATCH 1/3] Benchmark: Fix benchmark kernel of workgroup, function input

From: Grigore Lupescu Signed-off-by: Grigore Lupescu --- kernels/bench_workgroup.cl | 78 ++ 1 file changed, 51 insertions(+), 27 deletions(-) diff --git a/kernels/bench_workgroup.cl b/kernels/bench_workgroup.cl index 596e936..f26537b 100644 ---

[Beignet] [PATCH 2/3] Backend: Workgroup functions no check for src until fix on wellFormed

From: Grigore Lupescu Signed-off-by: Grigore Lupescu --- backend/src/ir/instruction.cpp | 4 1 file changed, 4 deletions(-) diff --git a/backend/src/ir/instruction.cpp b/backend/src/ir/instruction.cpp index 75e1b3b..a0b0e13 100644 --- a/backend/src/ir/instruction.cpp +++

[Beignet] [PATCH] Backend: Fix barrier placement in workgroup reduce/scan

From: Grigore Lupescu Signed-off-by: Grigore Lupescu --- backend/src/backend/gen_context.cpp| 15 +++ backend/src/backend/gen_insn_selection.cpp | 1 - 2 files changed, 11 insertions(+), 5 deletions(-) diff --git a/backend/src/backend/gen_context.cpp

Re: [Beignet] [PATCH 1/7] GBE: Fix type mismatch bug.

Please help review this version, I just rebased to latest ocl2.0 branch. You can skip previous version of these patches. Thanks! Ruiling > -Original Message- > From: Song, Ruiling > Sent: Friday, April 1, 2016 2:53 PM > To: beignet@lists.freedesktop.org > Cc: Song, Ruiling

[Beignet] [PATCH 7/7] GBE: retype double register to long type when do spilling.

Signed-off-by: Ruiling Song --- backend/src/backend/gen_context.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/backend/src/backend/gen_context.cpp b/backend/src/backend/gen_context.cpp index 3484e63..b263cc7 100644 ---

[Beignet] [PATCH 1/7] GBE: Fix type mismatch bug.

the move instruction should have same type src & dst. Signed-off-by: Ruiling Song --- backend/src/llvm/llvm_gen_backend.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/backend/src/llvm/llvm_gen_backend.cpp

[Beignet] [PATCH 2/7] GBE: Fix SEL.bool issue.

the flag register is used by the condition source, we have to store the dst register in GRF. when using this dst, will update from the allocated GRF to flag register. v2: should use ir::FAMILY_BOOL instead of ir::TYPE_BOOL. Signed-off-by: Ruiling Song ---

[Beignet] [PATCH 6/7] GBE: the dst grf should use same width as source register

When the source is double,the compare dst register should use QWORD. Signed-off-by: Ruiling Song --- backend/src/backend/gen_reg_allocation.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/backend/src/backend/gen_reg_allocation.cpp

[Beignet] [PATCH 4/7] GBE: Fix bug when unspill a long type value from scratch.

the register maybe a stride two register with type UD. So, retype it to long when do unspill. Signed-off-by: Ruiling Song --- backend/src/backend/gen_context.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/backend/src/backend/gen_context.cpp

[Beignet] [PATCH 5/7] GBE: don't try to erase a llvm:Constant.

If it is a llvm::Constant, don't add it to erase candidate. The reason I make this change is it cannot cast to llvm::Instruction. I think we still need to make clear whether or not we should delete a constant and how. Signed-off-by: Ruiling Song ---