On 11 November 2016 at 00:45, Timothy Arceri
<timothy.arc...@collabora.com> wrote:
> It's common for games to compile 2000 programs or more so at
>
> 32bits x 2000 programs x 22 fields x 2 (at least) stages
>
> This should give us something like 352 kilobytes in savings.
> ---
At first I was going to say "sed job", only to notice that you've been
in a few places fixing indentation/wrapping.
Now that's some dedication !
> - prog->Instructions = NULL;
> - prog->NumInstructions = 0;
> -
> prog->SamplersUsed = shader->active_samplers;
> prog->ShadowSamplers = shader->shadow_samplers;
> prog->ExternalSamplersUsed = gl_external_samplers(shader);
> diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
> b/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
> index a4679e5..7a4ee36 100644
> --- a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
> +++ b/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
> @@ -6501,9 +6501,6 @@ get_mesa_program_tgsi(struct gl_context *ctx,
> _mesa_log("\n\n");
> }
>
> - prog->Instructions = NULL;
> - prog->NumInstructions = 0;
> -
Skimmed through all the NewProgram callback implementations and to
check that those two are safe.
Can you please mention that in the commit summary and/or leave it as
separate fix ?
Regardless of the route taken here and previous patch (but with
LocalParams hunk properly handled here), with first 9 are
Reviewed-by: Emil Velikov <emil.veli...@collabora.com>
Mildly related: why does i965 (brw_link.cpp) use
ctx->Driver.NewProgram instead of exporting brwNewProgram and using it
directly ?
Emil
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
