<div dir="ltr">On 4 April 2013 12:13, Kenneth Graunke <span dir="ltr"><<a href="mailto:kenneth@whitecape.org" target="_blank">kenneth@whitecape.org</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On 04/04/2013 08:13 AM, Paul Berry wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 2 April 2013 23:33, Kenneth Graunke <<a href="mailto:kenneth@whitecape.org" target="_blank">kenneth@whitecape.org</a>> wrote:<br>
</blockquote></div>
[snip]<div class="im"><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
    diff --git a/src/glsl/main.cpp b/src/glsl/main.cpp<br>
    index ce084b4..13dfdd3 100644<br>
    --- a/src/glsl/main.cpp<br>
    +++ b/src/glsl/main.cpp<br>
    @@ -176,7 +176,7 @@ compile_shader(struct gl_context *ctx, struct<br>
    gl_shader *shader)<br>
         if (!state->error && !shader->ir->is_empty()) {<br>
            bool progress;<br>
            do {<br>
    -        progress = do_common_optimization(shader-<u></u>>ir, false, false,<br>
    32);<br>
    +        progress = do_common_optimization(shader-<u></u>>ir, false, false,<br>
    32, false);<br>
<br>
<br>
What's the reason for passing false in this case?  It seems like we<br>
ought to pass ctx->mvp_with_dp4 in all cases.<br>
</blockquote>
<br></div>
Fair enough.  For the standalone compiler, I just picked something rather arbitrarily.  ctx->mvp_with_dp4 is false for now.</blockquote><div><br></div><div>Ah! I got confused and didn't realize that this call site was in the standalone compiler.  Makes sense.<br>
</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im"><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
For that matter, I'm curious why we don't just check the value of<br>
ctx->mvp_with_dp4 from inside do_common_optimization()--it seems like<br>
that would be easier to maintain.<br>
</blockquote>
<br></div>
It doesn't currently have access to gl_context.  I could instead pass that...or move this flag inside ctx->ShaderCompilerOptions and pass a const pointer to that instead.  Preferences?</blockquote><div><br></div><div>
I like the idea of moving the flag inside ctx->ShaderCompilerOptions and passing a const pointer to that.  Assuming it's not too much trouble.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im"><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
With that question addressed, this series is:<br>
<br>
Reviewed-by: Paul Berry <<a href="mailto:stereotype441@gmail.com" target="_blank">stereotype441@gmail.com</a>><br>
</blockquote>
<br></div>
Thanks Paul!<br>
</blockquote></div><br></div></div>