summaryrefslogtreecommitdiffstats
path: root/src/mesa/main/arbprogram.c
diff options
context:
space:
mode:
authorEric Anholt <eric@anholt.net>2013-09-20 10:13:32 -0700
committerEric Anholt <eric@anholt.net>2013-11-15 11:35:01 -0800
commite5885c119de1e508099cc1111e1c9f8ff00fab88 (patch)
tree77232f7a5789900eb180e23fe0f2c1988a70a8f3 /src/mesa/main/arbprogram.c
parentbb1f0969756fbb827c4b2520c632daa15342b064 (diff)
downloadexternal_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.c6
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;
}