Mesa (master): radeonsi: add a 'break' in si_check_render_feedback_*()

Samuel Pitoiset hakzsam at kemper.freedesktop.org
Mon Apr 10 21:08:52 UTC 2017


Module: Mesa
Branch: master
Commit: 9cc91ba6d5fafa5435ff4b6081f731acc99af47f
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=9cc91ba6d5fafa5435ff4b6081f731acc99af47f

Author: Samuel Pitoiset <samuel.pitoiset at gmail.com>
Date:   Thu Apr  6 00:07:32 2017 +0200

radeonsi: add a 'break' in si_check_render_feedback_*()

No need to check all color buffers.

Signed-off-by: Samuel Pitoiset <samuel.pitoiset at gmail.com>
Reviewed-by: Marek Olšák <marek.olsak at amd.com>

---

 src/gallium/drivers/radeonsi/si_blit.c | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/src/gallium/drivers/radeonsi/si_blit.c b/src/gallium/drivers/radeonsi/si_blit.c
index 9b3096d734..4256c51efb 100644
--- a/src/gallium/drivers/radeonsi/si_blit.c
+++ b/src/gallium/drivers/radeonsi/si_blit.c
@@ -555,8 +555,10 @@ static void si_check_render_feedback_textures(struct si_context *sctx,
 			    surf->base.u.tex.level >= view->u.tex.first_level &&
 			    surf->base.u.tex.level <= view->u.tex.last_level &&
 			    surf->base.u.tex.first_layer <= view->u.tex.last_layer &&
-			    surf->base.u.tex.last_layer >= view->u.tex.first_layer)
+			    surf->base.u.tex.last_layer >= view->u.tex.first_layer) {
 				render_feedback = true;
+				break;
+			}
 		}
 
 		if (render_feedback)
@@ -595,8 +597,10 @@ static void si_check_render_feedback_images(struct si_context *sctx,
 			if (tex == (struct r600_texture*)surf->base.texture &&
 			    surf->base.u.tex.level == view->u.tex.level &&
 			    surf->base.u.tex.first_layer <= view->u.tex.last_layer &&
-			    surf->base.u.tex.last_layer >= view->u.tex.first_layer)
+			    surf->base.u.tex.last_layer >= view->u.tex.first_layer) {
 				render_feedback = true;
+				break;
+			}
 		}
 
 		if (render_feedback)




More information about the mesa-commit mailing list