From bdff9a6e47c42c0ba8cec2d8b5c5438c0068f874 Mon Sep 17 00:00:00 2001 From: Ian Romanick Date: Fri, 17 Jan 2014 15:28:23 -0800 Subject: [PATCH] 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 --- src/mesa/drivers/dri/i965/brw_clear.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 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; -- 2.30.2