diff options
| author | Grazvydas Ignotas <[email protected]> | 2017-03-26 19:30:23 +0300 |
|---|---|---|
| committer | Timothy Arceri <[email protected]> | 2017-03-27 15:05:10 +1100 |
| commit | b97faea162fe9999d241de679f99ad935a8bce26 (patch) | |
| tree | 9eb7ed304a4d3923072ef59d3f8c81f8ad275e7b /src/mesa | |
| parent | f2d4d116113c8eddf7e8f372d466510c31d3dba4 (diff) | |
glsl, st/shader_cache: check the whole sha1 for zero
The checks were only looking at the first byte, while the intention
seems to be to check if the whole sha1 is zero. This prevented all
shaders with first byte zero in their sha1 from being saved.
This shaves around a second from Deus Ex load time on a hot cache.
Signed-off-by: Grazvydas Ignotas <[email protected]>
Reviewed-by: Timothy Arceri <[email protected]>
Reviewed-by: Edward O'Callaghan <[email protected]>
Diffstat (limited to 'src/mesa')
| -rw-r--r-- | src/mesa/state_tracker/st_shader_cache.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/mesa/state_tracker/st_shader_cache.c b/src/mesa/state_tracker/st_shader_cache.c index 061b27221b3..e8c7289ec6d 100644 --- a/src/mesa/state_tracker/st_shader_cache.c +++ b/src/mesa/state_tracker/st_shader_cache.c @@ -64,7 +64,8 @@ st_store_tgsi_in_disk_cache(struct st_context *st, struct gl_program *prog, /* Exit early when we are dealing with a ff shader with no source file to * generate a source from. */ - if (*prog->sh.data->sha1 == 0) + static const char zero[sizeof(prog->sh.data->sha1)] = {0}; + if (memcmp(prog->sh.data->sha1, zero, sizeof(prog->sh.data->sha1)) == 0) return; unsigned char *sha1; |