<div dir="ltr">If I understand correctly, this is because when running with minimal GLSL IR, opt_function_inlining doesn't acutally inline them all.  Is that correct?  If so, would it make sense to just repeatedly call do_function_inlining until it stops making progress when we're running with minimal optimizations?  Not that I'm opposed to using NIR for more things, but it bothers me a bit that reducing GLSL IR optimizations is causing us to break previous assumptions about what is, effectively, a lowering pass.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Apr 9, 2018 at 9:34 PM, Timothy Arceri <span dir="ltr"><<a href="mailto:tarceri@itsqueeze.com" target="_blank">tarceri@itsqueeze.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">---<br>
 src/compiler/glsl/glsl_to_nir.<wbr>cpp | 20 ++++++++++++++++++++<br>
 1 file changed, 20 insertions(+)<br>
<br>
diff --git a/src/compiler/glsl/glsl_to_<wbr>nir.cpp b/src/compiler/glsl/glsl_to_<wbr>nir.cpp<br>
index 5a36963607e..55c01024669 100644<br>
--- a/src/compiler/glsl/glsl_to_<wbr>nir.cpp<br>
+++ b/src/compiler/glsl/glsl_to_<wbr>nir.cpp<br>
@@ -26,6 +26,7 @@<br>
  */<br>
<br>
 #include "glsl_to_nir.h"<br>
+#include "ir_optimization.h"<br>
 #include "ir_visitor.h"<br>
 #include "ir_hierarchical_visitor.h"<br>
 #include "ir.h"<br>
@@ -161,6 +162,25 @@ glsl_to_nir(const struct gl_shader_program *shader_prog,<br>
    v2.run(sh->ir);<br>
    visit_exec_list(sh->ir, &v1);<br>
<br>
+   nir_validate_shader(shader);<br>
+<br>
+   /* We have to lower away local constant initializers right before we<br>
+    * inline functions.  That way they get properly initialized at the top<br>
+    * of the function and not at the top of its caller.<br>
+    */<br>
+   nir_lower_constant_<wbr>initializers(shader, nir_var_local);<br>
+   nir_lower_returns(shader);<br>
+   nir_inline_functions(shader);<br>
+<br>
+   /* Now that we have inlined everything remove all of the functions except<br>
+    * main().<br>
+    */<br>
+   foreach_list_typed_safe(nir_<wbr>function, function, node, &(shader)->functions){<br>
+      if (strcmp("main", function->name) != 0) {<br>
+         exec_node_remove(&function-><wbr>node);<br>
+      }<br>
+   }<br>
+<br>
    nir_lower_constant_<wbr>initializers(shader, (nir_variable_mode)~0);<br>
<br>
    /* Remap the locations to slots so those requiring two slots will occupy<br>
<span class="HOEnZb"><font color="#888888">--<br>
2.17.0<br>
<br>
______________________________<wbr>_________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>mailman/listinfo/mesa-dev</a><br>
</font></span></blockquote></div><br></div>