Attachment 67549 Details for Bug 55224 – Fix NV50 RA interference register tracking.

[patch] Fix NV50 RA interference register tracking.

fix_nv50_ra_interference.patch (text/plain), 1.69 KB, created by Jay Cornwall on 2012-09-22 16:32:40 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jay Cornwall

Created: 2012-09-22 16:32:40 UTC

Size: 1.69 KB

patch

obsolete

>From ed10ca591239a42b9b74fc6862e256bdfd3e1ff3 Mon Sep 17 00:00:00 2001
>From: Jay Cornwall <jay@jcornwall.me>
>Date: Sat, 22 Sep 2012 11:15:11 -0500
>Subject: [PATCH] Fix NV50 RA interference register tracking.
>
>---
> src/gallium/drivers/nv50/codegen/nv50_ir_ra.cpp |    8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
>diff --git a/src/gallium/drivers/nv50/codegen/nv50_ir_ra.cpp b/src/gallium/drivers/nv50/codegen/nv50_ir_ra.cpp
>index 1399876..714837e 100644
>--- a/src/gallium/drivers/nv50/codegen/nv50_ir_ra.cpp
>+++ b/src/gallium/drivers/nv50/codegen/nv50_ir_ra.cpp
>@@ -43,7 +43,7 @@ public:
> 
>    bool assign(int32_t& reg, DataFile f, unsigned int size);
>    void release(DataFile f, int32_t reg, unsigned int size);
>-   bool occupy(DataFile f, int32_t reg, unsigned int size);
>+   bool occupy(DataFile f, int32_t reg, unsigned int size, bool noTest = false);
>    bool occupy(const Value *);
>    void occupyMask(DataFile f, int32_t reg, uint8_t mask);
> 
>@@ -167,9 +167,9 @@ RegisterSet::occupyMask(DataFile f, int32_t reg, uint8_t mask)
> }
> 
> bool
>-RegisterSet::occupy(DataFile f, int32_t reg, unsigned int size)
>+RegisterSet::occupy(DataFile f, int32_t reg, unsigned int size, bool noTest)
> {
>-   if (bits[f].testRange(reg, size))
>+   if (!noTest && bits[f].testRange(reg, size))
>       return false;
> 
>    bits[f].setRange(reg, size);
>@@ -1235,7 +1235,7 @@ GCRA::checkInterference(const RIG_Node *node, Graph::EdgeIterator& ei)
>       INFO_DBG(prog->dbgFlags, REG_ALLOC,
>                "(%%%i) X (%%%i): $r%i + %u\n",
>                vA->id, vB->id, intf->reg, intf->colors);
>-      regs.occupy(node->f, intf->reg, intf->colors);
>+      regs.occupy(node->f, intf->reg, intf->colors, true);
>    }
> }
> 
>-- 
>1.7.9.5
>

Actions: View | Splinter Review

Attachments on bug 55224: 67548 | 67549

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.