On Feb 5, 2015 5:12 PM, "Connor Abbott" <cwabbo...@gmail.com> wrote: > > Maybe we should run this after DCE so that we don't do unnecessary > work by scheduling stuff that we then delete.
Yeah, I thought about that. Also, this has interesting interactions with the select peephole. I think more analysis is needed before deciding on a final placement. > On Thu, Feb 5, 2015 at 5:29 PM, Jason Ekstrand <ja...@jlekstrand.net> wrote: > > --- > > src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp b/src/mesa/drivers/dri/i965/brw_fs_nir.cpp > > index c3c268c..ba03291 100644 > > --- a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp > > +++ b/src/mesa/drivers/dri/i965/brw_fs_nir.cpp > > @@ -42,6 +42,8 @@ nir_optimize(nir_shader *nir) > > nir_validate_shader(nir); > > progress |= nir_copy_prop(nir); > > nir_validate_shader(nir); > > + nir_opt_gcm(nir); > > + nir_validate_shader(nir); > > progress |= nir_opt_dce(nir); > > nir_validate_shader(nir); > > progress |= nir_opt_cse(nir); > > -- > > 2.2.2 > > > > _______________________________________________ > > mesa-dev mailing list > > mesa-dev@lists.freedesktop.org > > http://lists.freedesktop.org/mailman/listinfo/mesa-dev
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
