On Tue, May 6, 2014 at 3:47 PM, Eric Anholt <e...@anholt.net> wrote: > Now that we aren't using pixel_[xy] in live variables, nothing is looking > at these regs after the visitor stage. > --- > src/mesa/drivers/dri/i965/brw_fs.cpp | 37 > ------------------------------------ > 1 file changed, 37 deletions(-) > > diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp > b/src/mesa/drivers/dri/i965/brw_fs.cpp > index c550c41..6b1b866 100644 > --- a/src/mesa/drivers/dri/i965/brw_fs.cpp > +++ b/src/mesa/drivers/dri/i965/brw_fs.cpp > @@ -1740,34 +1740,6 @@ fs_visitor::compact_virtual_grfs() > } > } > > - /* In addition to registers used in instructions, fs_visitor keeps > - * direct references to certain special values which must be patched: > - */ > - struct { > - fs_reg *reg; > - unsigned count; > - } special[] = { > - { &frag_depth, 1 }, > - { &pixel_x, 1 }, > - { &pixel_y, 1 }, > - { &pixel_w, 1 }, > - { &wpos_w, 1 }, > - { &dual_src_output, 1 }, > - { outputs, ARRAY_SIZE(outputs) }, > - { delta_x, ARRAY_SIZE(delta_x) }, > - { delta_y, ARRAY_SIZE(delta_y) }, > - { &sample_mask, 1 }, > - { &shader_start_time, 1 }, > - }; > - > - /* Treat all special values as used, to be conservative */ > - for (unsigned i = 0; i < ARRAY_SIZE(special); i++) { > - for (unsigned j = 0; j < special[i].count; j++) { > - if (special[i].reg[j].file == GRF) > - remap_table[special[i].reg[j].reg] = 0; > - } > - } > - > /* Compact the GRF arrays. */ > int new_index = 0; > for (int i = 0; i < this->virtual_grf_count; i++) { > @@ -1793,15 +1765,6 @@ fs_visitor::compact_virtual_grfs() > inst->src[i].reg = remap_table[inst->src[i].reg]; > } > } > - > - /* Patch all the references to special values */ > - for (unsigned i = 0; i < ARRAY_SIZE(special); i++) { > - for (unsigned j = 0; j < special[i].count; j++) { > - fs_reg *reg = &special[i].reg[j]; > - if (reg->file == GRF && remap_table[reg->reg] != -1) > - reg->reg = remap_table[reg->reg]; > - } > - } > } > > /* > -- > 1.9.2
Series is Reviewed-by: Matt Turner <matts...@gmail.com> _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev