Re: [Mesa-dev] [PATCH 1/2] i965/blorp: patch jump counters also for endif

2014-01-21 Thread Pohjolainen, Topi
On Wed, Jan 22, 2014 at 06:57:28AM +0200, Pohjolainen, Topi wrote: > On Tue, Jan 21, 2014 at 12:48:12PM -0800, Eric Anholt wrote: > > Topi Pohjolainen writes: > > > > > No known bugs fixed but this is now in line with fs-generator. > > > No regresssions on IVB. > > > > The endif jump, since it's

Re: [Mesa-dev] [PATCH 1/2] i965/blorp: patch jump counters also for endif

2014-01-21 Thread Pohjolainen, Topi
On Tue, Jan 21, 2014 at 12:48:12PM -0800, Eric Anholt wrote: > Topi Pohjolainen writes: > > > No known bugs fixed but this is now in line with fs-generator. > > No regresssions on IVB. > > The endif jump, since it's forward, is just an optimization to have set > right -- otherwise, the GPU will

Re: [Mesa-dev] [PATCH 1/2] i965/blorp: patch jump counters also for endif

2014-01-21 Thread Eric Anholt
Topi Pohjolainen writes: > No known bugs fixed but this is now in line with fs-generator. > No regresssions on IVB. The endif jump, since it's forward, is just an optimization to have set right -- otherwise, the GPU will just step forward instruction by instruction until it hits something else t

[Mesa-dev] [PATCH 1/2] i965/blorp: patch jump counters also for endif

2014-01-21 Thread Topi Pohjolainen
No known bugs fixed but this is now in line with fs-generator. No regresssions on IVB. CC: Paul Berry Signed-off-by: Topi Pohjolainen --- src/mesa/drivers/dri/i965/brw_blorp_blit.cpp | 2 ++ src/mesa/drivers/dri/i965/test_blorp_blit_eu_gen.cpp | 6 +++--- 2 files changed, 5 insertions(+