Attachment 79572 Details for Bug 63520 – Possible Fix

[patch] Possible Fix

0001-r300g-compiler-Prevent-the-register-allocator-from-s.patch (text/plain), 1.13 KB, created by Tom Stellard on 2013-05-20 15:06:54 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Tom Stellard

Created: 2013-05-20 15:06:54 UTC

Size: 1.13 KB

patch

obsolete

>From 653b6dc651a4ceba060cdfe481b1418886813e1d Mon Sep 17 00:00:00 2001
>From: Tom Stellard <thomas.stellard@amd.com>
>Date: Mon, 20 May 2013 08:05:03 -0700
>Subject: [PATCH] r300g/compiler: Prevent the register allocator from
> swizzling texture operands
>
>https://bugs.freedesktop.org/show_bug.cgi?id=63520
>---
> src/gallium/drivers/r300/compiler/radeon_pair_regalloc.c | 6 ++++++
> 1 file changed, 6 insertions(+)
>
>diff --git a/src/gallium/drivers/r300/compiler/radeon_pair_regalloc.c b/src/gallium/drivers/r300/compiler/radeon_pair_regalloc.c
>index 6442e0d..d6f9deb 100644
>--- a/src/gallium/drivers/r300/compiler/radeon_pair_regalloc.c
>+++ b/src/gallium/drivers/r300/compiler/radeon_pair_regalloc.c
>@@ -318,6 +318,12 @@ static void variable_get_class_read_cb(
> 	struct variable_get_class_cb_data * d = userdata;
> 	unsigned int new_swizzle = rc_adjust_channels(arg->Swizzle,
> 							d->conversion_swizzle);
>+
>+	/* We can't change source swizzles for texture instructions. */ 
>+	if (inst->Type == RC_INSTRUCTION_NORMAL) {
>+		*d->can_change_writemask = 0;
>+	}
>+
> 	if (!r300_swizzle_is_native_basic(new_swizzle)) {
> 		*d->can_change_writemask = 0;
> 	}
>-- 
>1.7.11.4
>

Actions: View | Splinter Review

Attachments on bug 63520: 77936 | 77937 | 77938 | 77939 | 78616 | 78617 | 79150 | 79152 | 79154 | 79155 | 79156 | 79157 | 79158 | 79160 | 79572 | 79628 | 80888

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.