From: Ian Romanick Date: Fri, 17 Jan 2014 23:28:23 +0000 (-0800) Subject: i965: Consider only the scissor rectangle for viewport 0 for clears X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=bdff9a6e47c42c0ba8cec2d8b5c5438c0068f874;p=mesa.git i965: Consider only the scissor rectangle for viewport 0 for clears noop_scissor (correctly) only examines the scissor rectangle for viewport 0. Therefore, it should only be called when that scissor rectangle is enabled. v2: Remove spurious change to radeon code. Noticed by Ken. Signed-off-by: Ian Romanick Reviewed-by: Kenneth Graunke --- diff --git a/src/mesa/drivers/dri/i965/brw_clear.c b/src/mesa/drivers/dri/i965/brw_clear.c index 72950cb518b..659d339e1a5 100644 --- a/src/mesa/drivers/dri/i965/brw_clear.c +++ b/src/mesa/drivers/dri/i965/brw_clear.c @@ -121,7 +121,7 @@ brw_fast_clear_depth(struct gl_context *ctx) * a previous clear had happened at a different clear value and resolve it * first. */ - if (ctx->Scissor.EnableFlags && !noop_scissor(ctx, fb)) { + if ((ctx->Scissor.EnableFlags & 1) && !noop_scissor(ctx, fb)) { perf_debug("Failed to fast clear depth due to scissor being enabled. " "Possible 5%% performance win if avoided.\n"); return false;