On Thu, Dec 22, 2011 at 02:37:58PM -0800, Kenneth Graunke wrote: > 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 <kenn...@whitecape.org>
Thanks. > > > i965: increase the brw eu instruction store size dynamically > > Patch 8 does not get a R-b just yet. Ok , will fix it. > > Thanks for doing this, Yuanhan, I'm really glad to see the arbitrary > 10000 limit die. Welcome and it's my pleasure. > And Eric, thanks for cleaning up the rest of the > control flow stack code---it's /so/ much nicer now! _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
