diff options
author | Eric Anholt <[email protected]> | 2013-09-20 10:13:32 -0700 |
---|---|---|
committer | Eric Anholt <[email protected]> | 2013-11-15 11:35:01 -0800 |
commit | e5885c119de1e508099cc1111e1c9f8ff00fab88 (patch) | |
tree | 77232f7a5789900eb180e23fe0f2c1988a70a8f3 /src/mesa/main/arbprogram.c | |
parent | bb1f0969756fbb827c4b2520c632daa15342b064 (diff) |
mesa: Dynamically allocate the storage for program local parameters.
The array was 64kb per struct gl_program, plus we statically stored a copy
of one on disk for _mesa_DummyProgram. Given that most struct gl_programs
we generate are for GLSL shaders that don't have local parameters, this
was a waste.
Since you can store and fetch parameters beyond what the program actually
uses, we do have to do a late allocation if necessary at
GetProgramLocalParameter time.
Reduces peak memory usage in the dota2 trace I made by 76MB (4.5%)
Reviewed-by: Brian Paul <[email protected]>
Reviewed-by: Ian Romanick <[email protected]>
Diffstat (limited to 'src/mesa/main/arbprogram.c')
-rw-r--r-- | src/mesa/main/arbprogram.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/src/mesa/main/arbprogram.c b/src/mesa/main/arbprogram.c index 51a299370e9..8bd3f0bd06f 100644 --- a/src/mesa/main/arbprogram.c +++ b/src/mesa/main/arbprogram.c @@ -265,6 +265,12 @@ get_local_param_pointer(struct gl_context *ctx, const char *func, return GL_FALSE; } + if (!prog->LocalParams) { + prog->LocalParams = calloc(maxParams, sizeof(float[4])); + if (!prog->LocalParams) + return GL_FALSE; + } + *param = prog->LocalParams[index]; return GL_TRUE; } |