brw_shader_precompile should just do a precompile; it makes more sense for the caller to decide whether we should do one. Simpler.
Signed-off-by: Kenneth Graunke <kenn...@whitecape.org> --- src/mesa/drivers/dri/i965/brw_shader.cpp | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_shader.cpp b/src/mesa/drivers/dri/i965/brw_shader.cpp index 10f8db7..78db484 100644 --- a/src/mesa/drivers/dri/i965/brw_shader.cpp +++ b/src/mesa/drivers/dri/i965/brw_shader.cpp @@ -59,13 +59,13 @@ brw_shader_precompile(struct gl_context *ctx, struct gl_shader_program *prog) { struct brw_context *brw = brw_context(ctx); - if (brw->precompile && !brw_fs_precompile(ctx, prog)) + if (!brw_fs_precompile(ctx, prog)) return false; - if (brw->precompile && !brw_gs_precompile(ctx, prog)) + if (!brw_gs_precompile(ctx, prog)) return false; - if (brw->precompile && !brw_vs_precompile(ctx, prog)) + if (!brw_vs_precompile(ctx, prog)) return false; return true; @@ -255,7 +255,7 @@ brw_link_shader(struct gl_context *ctx, struct gl_shader_program *shProg) } } - if (!brw_shader_precompile(ctx, shProg)) + if (brw->precompile && !brw_shader_precompile(ctx, shProg)) return false; return true; -- 2.1.3 _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev