Re: [Beignet] [PATCH V3] GBE: Optimize scratch memory usage using register interval

2014-02-27 Thread Zhigang Gong
LGTM, pushed, thanks. On Fri, Feb 28, 2014 at 10:16:45AM +0800, Ruiling Song wrote: > As scratch memory is a limited resource in HW. And different > register have the opptunity to share same scratch memory. So > I introduce an allocator for scratch memory management. > > v2: > In order to reuse t

Re: [Beignet] [PATCH] GBE: show correct line number in build log

2014-02-27 Thread Zhigang Gong
LGTM, pushed, thanks. On Fri, Feb 28, 2014 at 01:58:20AM +0800, Guo Yejun wrote: > Sometimes, we insert some code into the kernel, > it makes the line number reported in build log > mismatch with the line number in the kernel from > programer's view, use #line to correct it. > > Signed-off-by: Gu

Re: [Beignet] [PATCH V2] GBE: support getelementptr with ConstantExpr operand

2014-02-27 Thread Zhigang Gong
LGTM, pushed, thanks. On Thu, Feb 27, 2014 at 06:54:26AM +0800, Guo Yejun wrote: > Add support during LLVM IR -> Gen IR period when the > first operand of getelementptr is ConstantExpr. > > utest is also added. > > Signed-off-by: Guo Yejun > --- > backend/src/llvm/llvm_gen_backend.cpp |

Re: [Beignet] [PATCH V2] GBE: Optimize scratch memory usage using register interval

2014-02-27 Thread Song, Ruiling
Please skip this version, as I made a mistake. Please help review v3. Thanks! Ruiling ___ Beignet mailing list Beignet@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/beignet

[Beignet] [PATCH V3] GBE: Optimize scratch memory usage using register interval

2014-02-27 Thread Ruiling Song
As scratch memory is a limited resource in HW. And different register have the opptunity to share same scratch memory. So I introduce an allocator for scratch memory management. v2: In order to reuse the registerFilePartitioner, I rename it as SimpleAllocator, and derive ScratchAllocator & Registe

[Beignet] [PATCH] GBE: show correct line number in build log

2014-02-27 Thread Guo Yejun
Sometimes, we insert some code into the kernel, it makes the line number reported in build log mismatch with the line number in the kernel from programer's view, use #line to correct it. Signed-off-by: Guo Yejun --- backend/src/backend/program.cpp |5 + 1 file changed, 5 insertions(+) d