From: Connor Abbott <connor.w.abbott@intel.com>
Date: Wed, 29 Jul 2015 00:06:12 +0000 (-0700)
Subject: i965/fs: fix assign_constant_locations() for doubles
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=4f3888c1caf3455f61b2e20ccf7c39e59f4feaf3;p=mesa.git

i965/fs: fix assign_constant_locations() for doubles

Uniform doubles will read two registers, in which case we need to mark
both as being live.

v2 (Sam):
  - Use a formula to get the number of registers read with proper
    units (Curro).

Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>
Reviewed-by: Francisco Jerez <currojerez@riseup.net>
---

diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp b/src/mesa/drivers/dri/i965/brw_fs.cpp
index d0ce79d101a..6eae49fdf4a 100644
--- a/src/mesa/drivers/dri/i965/brw_fs.cpp
+++ b/src/mesa/drivers/dri/i965/brw_fs.cpp
@@ -2056,8 +2056,12 @@ fs_visitor::assign_constant_locations()
             }
             is_live[last] = true;
          } else {
-            if (constant_nr >= 0 && constant_nr < (int) uniforms)
-               is_live[constant_nr] = true;
+            if (constant_nr >= 0 && constant_nr < (int) uniforms) {
+               int regs_read = inst->components_read(i) *
+                  type_sz(inst->src[i].type) / 4;
+               for (int j = 0; j < regs_read; j++)
+                  is_live[constant_nr + j] = true;
+            }
          }
       }
    }