On Thu, May 7, 2015 at 6:20 AM, Neil Roberts <n...@linux.intel.com> wrote: > opt_sampler_eot enables a direct write to framebuffer from a sample. > In order to do this the sample message needs to have a message header > so if there wasn't one already then the function adds one. In addition > the function sets the destination register to null because it's no > longer used. However it was only doing this in cases where it was > adding a message header. This patch just moves setting the destination > so that it happens even if there's a messge header. In practice this > doesn't seem to make any difference but it's a bit cleaner. > --- > src/mesa/drivers/dri/i965/brw_fs.cpp | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp > b/src/mesa/drivers/dri/i965/brw_fs.cpp > index 1ca7ca6..72d408b 100644 > --- a/src/mesa/drivers/dri/i965/brw_fs.cpp > +++ b/src/mesa/drivers/dri/i965/brw_fs.cpp > @@ -2675,6 +2675,7 @@ fs_visitor::opt_sampler_eot() > > tex_inst->offset |= fb_write->target << 24; > tex_inst->eot = true; > + tex_inst->dst = reg_null_ud; > fb_write->remove(cfg->blocks[cfg->num_blocks - 1]); > > /* If a header is present, marking the eot is sufficient. Otherwise, we > need > @@ -2712,7 +2713,6 @@ fs_visitor::opt_sampler_eot() > tex_inst->header_present = true; > tex_inst->insert_before(cfg->blocks[cfg->num_blocks - 1], > new_load_payload); > tex_inst->src[0] = send_header; > - tex_inst->dst = reg_null_ud; > > return true; > } > -- > 1.9.3 > > _______________________________________________ > mesa-dev mailing list > mesa-dev@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/mesa-dev
LGTM. Reviewed-by: Anuj Phogat <anuj.pho...@gmail.com> _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev