Changeset 670 for trunkTweet

Timestamp:

Feb 20, 2011, 1:02:42 PM (12 years ago)

Author:

sam

Message:

Cache shader source CRCs to avoid costly recompilations.

File:

• trunk/src/shader.cpp (modified) (4 diffs)

trunk/src/shader.cpp

@@ -39 +39 @@
 private:
     GLuint prog_id, vert_id, frag_id;
+    uint32_t vert_crc, frag_crc;
+
+    /* Global shader cache */
+    static Shader *shaders[];
+    static int nshaders;
 };
+
+Shader *ShaderData::shaders[256];
+int ShaderData::nshaders = 0;
 
 /*
@@ -47 +55 @@
 Shader *Shader::Create(char const *vert, char const *frag)
 {
-    return new Shader(vert, frag);
+    uint32_t new_vert_crc = Hash::Crc32(vert);
+    uint32_t new_frag_crc = Hash::Crc32(frag);
+
+    for (int n = 0; n < ShaderData::nshaders; n++)
+    {
+        if (ShaderData::shaders[n]->data->vert_crc == new_vert_crc
+             && ShaderData::shaders[n]->data->frag_crc == new_frag_crc)
+            return ShaderData::shaders[n];
+    }
+
+    Shader *ret = new Shader(vert, frag);
+    ShaderData::shaders[ShaderData::nshaders] = ret;
+    ShaderData::nshaders++;
+    return ret;
 }
 
 void Shader::Destroy(Shader *shader)
 {
-    delete shader;
+    /* XXX: do nothing! the shader should remain in cache */
+    (void)shader;
 }
 
@@ -61 +83 @@
     GLsizei len;
 
+    data->vert_crc = Hash::Crc32(vert);
     data->vert_id = glCreateShader(GL_VERTEX_SHADER);
     glShaderSource(data->vert_id, 1, &vert, NULL);
@@ -69 +92 @@
         fprintf(stderr, "ERROR: failed to compile vertex shader: %s", buf);
 
+    data->frag_crc = Hash::Crc32(frag);
     data->frag_id = glCreateShader(GL_FRAGMENT_SHADER);
     glShaderSource(data->frag_id, 1, &frag, NULL);