apparently we don't want to leak memory. Wish people would make up their mind.

Signed-off-by: Chris Perkins <[email protected]>

Author: cperkinsintel

sycl/source/detail/device_global_map.hpp

@@ -88,7 +88,11 @@ class DeviceGlobalMap {
         if (findDevGlobalByValue != MPtr2DeviceGlobal.end())
           MPtr2DeviceGlobal.erase(findDevGlobalByValue);
 
-        MDeviceGlobals.erase(DevGlobalIt);
+        // Must properly free the memory, will do in
+        // removeAssociatedResources(). Now we blank the key so it isn't used.
+        auto h = MDeviceGlobals.extract(DevGlobalIt);
+        h.key() = "";
+        MDeviceGlobals.insert(std::move(h));
       }
     }
   }