diff options
author | Eric Anholt <eric@anholt.net> | 2013-09-20 10:13:32 -0700 |
---|---|---|
committer | Eric Anholt <eric@anholt.net> | 2013-11-15 11:35:01 -0800 |
commit | e5885c119de1e508099cc1111e1c9f8ff00fab88 (patch) | |
tree | 77232f7a5789900eb180e23fe0f2c1988a70a8f3 /src/mesa/main/arbprogram.c | |
parent | bb1f0969756fbb827c4b2520c632daa15342b064 (diff) | |
download | external_mesa3d-e5885c119de1e508099cc1111e1c9f8ff00fab88.zip external_mesa3d-e5885c119de1e508099cc1111e1c9f8ff00fab88.tar.gz external_mesa3d-e5885c119de1e508099cc1111e1c9f8ff00fab88.tar.bz2 |
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 <brianp@vmware.com>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
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 51a2993..8bd3f0b 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; } |