[Mesa-dev] [PATCH] r300/compiler: remove useless check

David Heidelberger david.heidelberger at ixit.cz
Mon Oct 13 18:27:58 PDT 2014


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

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);
-- 
2.1.2
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-r300-compiler-remove-useless-check.patch
Type: text/x-diff
Size: 1193 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20141014/578b906d/attachment.patch>


More information about the mesa-dev mailing list