Not sure why you Cc'd me, but whatever.  Seems obvious enough.

Reviewed-by: Jason Ekstrand <jason.ekstr...@intel.com>
On 02/27/2015 06:50 PM, Ian Romanick wrote:
From: Ian Romanick <ian.d.roman...@intel.com>

Unused since b18fd23.

brw_fs.cpp:2878:44: warning: unused parameter 'dispatch_width' 
[-Wunused-parameter]
  clear_deps_for_inst_src(fs_inst *inst, int dispatch_width, bool *deps,
                                             ^

Signed-off-by: Ian Romanick <ian.d.roman...@intel.com>
Cc: Jason Ekstrand <jason.ekstr...@intel.com>
---
  src/mesa/drivers/dri/i965/brw_fs.cpp | 12 ++++--------
  1 file changed, 4 insertions(+), 8 deletions(-)

diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp 
b/src/mesa/drivers/dri/i965/brw_fs.cpp
index 0354f56..126b7d0 100644
--- a/src/mesa/drivers/dri/i965/brw_fs.cpp
+++ b/src/mesa/drivers/dri/i965/brw_fs.cpp
@@ -2875,8 +2875,7 @@ fs_visitor::remove_duplicate_mrf_writes()
  }
static void
-clear_deps_for_inst_src(fs_inst *inst, int dispatch_width, bool *deps,
-                        int first_grf, int grf_len)
+clear_deps_for_inst_src(fs_inst *inst, bool *deps, int first_grf, int grf_len)
  {
     /* Clear the flag for registers that actually got read (as expected). */
     for (int i = 0; i < inst->sources; i++) {
@@ -2927,8 +2926,7 @@ 
fs_visitor::insert_gen4_pre_send_dependency_workarounds(bblock_t *block,
     memset(needs_dep, false, sizeof(needs_dep));
     memset(needs_dep, true, write_len);
- clear_deps_for_inst_src(inst, dispatch_width,
-                           needs_dep, first_write_grf, write_len);
+   clear_deps_for_inst_src(inst, needs_dep, first_write_grf, write_len);
/* Walk backwards looking for writes to registers we're writing which
      * aren't read since being written.  If we hit the start of the program,
@@ -2968,8 +2966,7 @@ 
fs_visitor::insert_gen4_pre_send_dependency_workarounds(bblock_t *block,
        }
/* Clear the flag for registers that actually got read (as expected). */
-      clear_deps_for_inst_src(scan_inst, dispatch_width,
-                              needs_dep, first_write_grf, write_len);
+      clear_deps_for_inst_src(scan_inst, needs_dep, first_write_grf, 
write_len);
/* Continue the loop only if we haven't resolved all the dependencies */
        int i;
@@ -3014,8 +3011,7 @@ 
fs_visitor::insert_gen4_post_send_dependency_workarounds(bblock_t *block, fs_ins
        }
/* Clear the flag for registers that actually got read (as expected). */
-      clear_deps_for_inst_src(scan_inst, dispatch_width,
-                              needs_dep, first_write_grf, write_len);
+      clear_deps_for_inst_src(scan_inst, needs_dep, first_write_grf, 
write_len);
/* We insert our reads as late as possible since they're reading the
         * result of a SEND, which has massive latency.

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to