Revert "pgraph: Don't dupe shared GeomVertexData when premunging Geom"

This reverts commit f058bad802aa881a5604209afe5ef5bfd3d8fe35.

Reverting because I realised the current behaviour is happening due to different mungers, which my fix ignores.

Unfixes #1185

panda/src/pgraph/geomNode.cxx

@@ -876,36 +876,17 @@ do_premunge(GraphicsStateGuardianBase *gsg,
             GeomTransformer &transformer) {
   Thread *current_thread = Thread::get_current_thread();
 
-  // Cache munged vertex data, so that we don't duplicate vertex data
-  // unnecessarily.
-  std::map<CPT(GeomVertexData), CPT(GeomVertexData)> munged_vdata;
-
   OPEN_ITERATE_CURRENT_AND_UPSTREAM(_cycler, current_thread) {
     CDStageWriter cdata(_cycler, pipeline_stage, current_thread);
 
+    GeomList::iterator gi;
     PT(GeomList) geoms = cdata->modify_geoms();
-    for (GeomEntry &entry : *geoms) {
+    for (gi = geoms->begin(); gi != geoms->end(); ++gi) {
+      GeomEntry &entry = (*gi);
       CPT(RenderState) geom_state = node_state->compose(entry._state);
       CPT(Geom) geom = entry._geom.get_read_pointer();
       PT(GeomMunger) munger = gsg->get_geom_munger(geom_state, current_thread);
-
-      CPT(GeomVertexData) vdata = geom->get_vertex_data();
-
-      auto it = munged_vdata.find(vdata);
-      if (it != munged_vdata.end()) {
-        vdata = it->second;
-      } else {
-        CPT(GeomVertexData) old_vdata = std::move(vdata);
-        vdata = munger->premunge_data(old_vdata);
-        munged_vdata[std::move(old_vdata)] = vdata;
-      }
-
-      CPT(Geom) pgeom = geom;
-      munger->premunge_geom(pgeom, vdata);
-
-      PT(Geom) geom_copy = pgeom->make_copy();
-      geom_copy->set_vertex_data(vdata);
-      entry._geom = std::move(geom_copy);
+      entry._geom = transformer.premunge_geom(geom, munger);
     }
   }
   CLOSE_ITERATE_CURRENT_AND_UPSTREAM(_cycler);