Attachment 112689 Details for Bug 88717 – Possible fix

[patch] Possible fix

0001-r300g-compiler-Don-t-try-to-allocate-a-source-slot-f.patch (text/plain), 1.03 KB, created by Tom Stellard on 2015-01-22 21:12:06 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Tom Stellard

Created: 2015-01-22 21:12:06 UTC

Size: 1.03 KB

patch

obsolete

>From cabeffa7f8c0520d9da5e33d201440249e20097a Mon Sep 17 00:00:00 2001
>From: Tom Stellard <thomas.stellard@amd.com>
>Date: Thu, 22 Jan 2015 16:10:41 -0500
>Subject: [PATCH] r300g/compiler: Don't try to allocate a source slot for
> inline constants
>
>They don't need one.
>
>https://bugs.freedesktop.org/show_bug.cgi?id=88717
>---
> src/gallium/drivers/r300/compiler/radeon_program_pair.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/src/gallium/drivers/r300/compiler/radeon_program_pair.c b/src/gallium/drivers/r300/compiler/radeon_program_pair.c
>index 5231595..baecaf3 100644
>--- a/src/gallium/drivers/r300/compiler/radeon_program_pair.c
>+++ b/src/gallium/drivers/r300/compiler/radeon_program_pair.c
>@@ -45,7 +45,7 @@ int rc_pair_alloc_source(struct rc_pair_instruction *pair,
> 	unsigned int rgb_used = 0;
> 	int i;
> 
>-	if ((!rgb && !alpha) || file == RC_FILE_NONE)
>+	if ((!rgb && !alpha) || file == RC_FILE_NONE || file == RC_FILE_INLINE)
> 		return 0;
> 
> 	/* Make sure only one presubtract operation is used per instruction. */
>-- 
>1.9.0
>

Actions: View | Splinter Review

Attachments on bug 88717: 112679 | 112689 | 112717 | 112936 | 112937 | 112981 | 112982

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.