[Mesa-dev] [PATCH 0/8] i965: dynamic eu instruction store size

Kenneth Graunke kenneth at whitecape.org
Thu Dec 22 14:37:58 PST 2011

On 12/21/2011 01:33 AM, Yuanhan Liu wrote:
> Hi, this is a new series of patches for dynamic eu instruction store
> size. The first 4 is from Eric. I just grabed it to make it rebase to
> current repo. The last 4 patch is from mine which some are based on
> those patches from Eric.
> Please help to review it.
> BTW, I checked those patches with all oglc test cases, and found
> no regression. (Sandybridge only).
> Thanks,
> Yuanhan Liu
> --
> Eric Anholt (4):
>   i965: Drop unused do_insn argument from gen6_CONT().
>   i965: Don't make consumers of brw_DO()/brw_WHILE() track loop start
>   i965: Don't make consumers of brw_WHILE do pre-gen6 BREAK/CONT
>     patching
>   i965: Don't make consumers of brw_CONT/brw_WHILE track if depth in
>     loop
> Yuanhan Liu (4):
>   i965: let the if_stack just store the instruction index
>   i965: get the jmp distance by instruction index
>   i965: call next_insn() before referencing a instruction by index

Patches 1-7 (v2 of 6 and after changing to bool in 7) are:
Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>

>   i965: increase the brw eu instruction store size dynamically

Patch 8 does not get a R-b just yet.

Thanks for doing this, Yuanhan, I'm really glad to see the arbitrary
10000 limit die.  And Eric, thanks for cleaning up the rest of the
control flow stack code---it's /so/ much nicer now!

More information about the mesa-dev mailing list