[Mesa-dev] [PATCH 1/2] i965: Use fewer temporary variables in clip setup.

Kenneth Graunke kenneth at whitecape.org
Mon Nov 19 20:36:43 PST 2012


On 11/15/2012 10:44 PM, Eric Anholt wrote:
> When you're comparing to the spec, you're trying to immediately see what
> numbered dword of the packet your bit ends up in.
> ---
>   src/mesa/drivers/dri/i965/gen6_clip_state.c |   21 ++++++++-------------
>   src/mesa/drivers/dri/i965/gen7_clip_state.c |   25 ++++++++++---------------
>   2 files changed, 18 insertions(+), 28 deletions(-)

Patch 1 looks great, and I can definitely believe that it's necessary to 
restrict the guardband when the viewport's smaller.  The diagrams in the 
bspec suggested as much.

for both:
Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>


More information about the mesa-dev mailing list