Mesa (master): r300/compiler: remove useless check

Tom Stellard tstellar at kemper.freedesktop.org
Tue Oct 14 16:24:06 UTC 2014


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

Author: David Heidelberger <david.heidelberger at ixit.cz>
Date:   Tue Oct 14 02:25:01 2014 +0200

r300/compiler: remove useless check

This code is already in if (!variable->C->is_r500) so no need check
twice.

Reviewed-by: Tom Stellard <thomas.stellard at amd.com>
Signed-off-by: David Heidelberger <david.heidelberger at ixit.cz>

---

 src/gallium/drivers/r300/compiler/radeon_pair_regalloc.c |    7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/src/gallium/drivers/r300/compiler/radeon_pair_regalloc.c b/src/gallium/drivers/r300/compiler/radeon_pair_regalloc.c
index 64b225d..14f93fb 100644
--- a/src/gallium/drivers/r300/compiler/radeon_pair_regalloc.c
+++ b/src/gallium/drivers/r300/compiler/radeon_pair_regalloc.c
@@ -387,11 +387,8 @@ static enum rc_reg_class variable_get_class(
 						 * instructions can't be
 						 * swizzle on r300/r400 GPUs.
 						 */
-						if (!variable->C->is_r500) {
-							can_change_writemask = 0;
-							break;
-						}
-						old_swizzle = r.U.I.Src->Swizzle;
+						can_change_writemask = 0;
+						break;
 					}
 					new_swizzle = rc_adjust_channels(
 						old_swizzle, conversion_swizzle);




More information about the mesa-commit mailing list