<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">2018-06-18 10:39 GMT+02:00 Iago Toral <span dir="ltr"><<a href="mailto:itoral@igalia.com" target="_blank">itoral@igalia.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><span class=""><div>On Mon, 2018-06-18 at 09:43 +0200, Gustaw Smolarczyk wrote:</div><blockquote type="cite" style="margin:0 0 0 .8ex;border-left:2px #729fcf solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">2018-06-18 4:39 GMT+02:00 Timothy Arceri <span dir="ltr"><<a href="mailto:tarceri@itsqueeze.com" target="_blank">tarceri@itsqueeze.com</a>></span>:<br><blockquote type="cite" style="margin:0 0 0 .8ex;border-left:2px #729fcf solid;padding-left:1ex">This is required for compatibility profile support. <br>
---<br>
 src/mesa/main/ff_fragment_<wbr>shader.cpp | 6 +++++-<br>
 1 file changed, 5 insertions(+), 1 deletion(-)<br>
<br>
diff --git a/src/mesa/main/ff_fragment_<wbr>shader.cpp b/src/mesa/main/ff_fragment_<wbr>shader.cpp<br>
index a698931d99e..935a21624af 100644<br>
--- a/src/mesa/main/ff_fragment_<wbr>shader.cpp<br>
+++ b/src/mesa/main/ff_fragment_<wbr>shader.cpp<br>
@@ -229,7 +229,11 @@ static GLbitfield filter_fp_input_mask( GLbitfield fp_inputs,<br>
     * since vertex shader state validation comes after fragment state<br>
     * validation (see additional comments in state.c).<br>
     */<br>
-   if (vertexShader)<br>
+   if (ctx->_Shader->CurrentProgram[<wbr>MESA_SHADER_GEOMETRY] != NULL)<br>
+      vprog = ctx->_Shader->CurrentProgram[<wbr>MESA_SHADER_GEOMETRY];<br>
+   else if (ctx->_Shader->CurrentProgram[<wbr>MESA_SHADER_TESS_EVAL] != NULL)<br>
+      vprog = ctx->_Shader->CurrentProgram[<wbr>MESA_SHADER_TESS_EVAL];<br>
+   else if (vertexShader)<br></blockquote><div>

<div style="font-size:small;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial"><br class="m_4318638278443436045gmail-Apple-interchange-newline"><br></div><div style="font-size:small;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial">Shouldn't you also update the if condition on line 178? Otherwise, you won't reach the if tree you change when the vertex shader is missing (unless that was intended - I am not really familiar with how fixed function shaders work alongside new features).</div></div></div></div></div></blockquote><div><br></div></span><div>You don't have Tesselation / Geometry with fixed function GL, so I think this should be fine.</div></div></blockquote><div><br></div><div>Well, this whole file implements fixed function fragment shader, so it will only be reached when the fragment shader is missing. Unless you meant that 

<span style="font-size:small;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline">tessellation/geometry shaders</span>
cannot be combined with f<span style="font-size:small;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline">ixed function vertex shader</span>

 but fixed function fragment shader is fine.</div><div><br></div><div>Regards,</div><div>Gustaw Smolarczyk</div></div></div></div>