On Feb 5, 2015 5:12 PM, "Connor Abbott" <<a href="mailto:cwabbott0@gmail.com">cwabbott0@gmail.com</a>> 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 <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> 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 > > <a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a> > > <a href="http://lists.freedesktop.org/mailman/listinfo/mesa-dev">http://lists.freedesktop.org/mailman/listinfo/mesa-dev</a>