<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Dec 30, 2015 at 9:04 AM, Rob Clark <span dir="ltr"><<a href="mailto:robdclark@gmail.com" target="_blank">robdclark@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Wed, Dec 30, 2015 at 10:32 AM, Rob Clark <<a href="mailto:robdclark@gmail.com">robdclark@gmail.com</a>> wrote:<br>
> diff --git a/src/mesa/drivers/dri/i965/brw_nir.c b/src/mesa/drivers/dri/i965/brw_nir.c<br>
> index e031173..a8bed70 100644<br>
> --- a/src/mesa/drivers/dri/i965/brw_nir.c<br>
> +++ b/src/mesa/drivers/dri/i965/brw_nir.c<br>
> @@ -405,42 +405,8 @@ brw_nir_lower_uniforms(nir_shader *nir, bool is_scalar)<br>
>     }<br>
>  }<br>
><br>
> -#include "util/debug.h"<br>
> -<br>
> -static bool<br>
> -should_clone_nir()<br>
> -{<br>
> -   static int should_clone = -1;<br>
> -   if (should_clone < 1)<br>
<br>
</span>btw, EdB pointed out on irc..  shouldn't this be if (should_clone < 0) ?<br></blockquote><div><br></div><div>Yes, it should.<br><br></div><div>With that,<br><br></div><div>Reviewed-by: Jason Ekstrand <<a href="mailto:jason.ekstraand@intel.com">jason.ekstraand@intel.com</a>><br><br></div><div>As I mentioned on IRC, I recently realized that Ken's mechanism for enforcing metadata kind-of falls over in the face of multiple functions.  In particular, you may not modify one of the functions and therefore don't need to invalidate metadata but if you invalidate so much as one function, you return progress and it then asserts that all functions have invalidated metadata.  Not sure what the best solution there is.  Maybe just have a nir_metadata_preserve(impl, nir_metadata_all) and require you to call that?  I don't really like that either because someone will use nir_metadata_all when they make a change that doesn't invalidate anything and then someone will add metadata that does get invalidated and we have a problem.  Thoughts?<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
BR,<br>
-R<br>
<div class="HOEnZb"><div class="h5"><br>
> -      should_clone = env_var_as_boolean("NIR_TEST_CLONE", false);<br>
> -<br>
> -   return should_clone;<br>
> -}<br>
</div></div></blockquote></div><br></div></div>