Rework test of pointers at GC to use malloc instead of C++ constructors (#3069)

***NO_CI***

Author: josesimoes

src/CLR/Core/GarbageCollector_Info.cpp

@@ -214,7 +214,12 @@ bool CLR_RT_GarbageCollector::TestPointers_PopulateOld_Worker(void **ref)
 
     if (dst)
     {
-        RelocationRecord *ptr = new RelocationRecord();
+        RelocationRecord *ptr = (RelocationRecord *)platform_malloc(sizeof(RelocationRecord));
+
+        if (!ptr)
+        {
+            return false;
+        }
 
         s_lstRecords.push_back(ptr);
 
@@ -233,6 +238,8 @@ bool CLR_RT_GarbageCollector::TestPointers_PopulateOld_Worker(void **ref)
 #else
             CLR_Debug::Printf("Duplicate base OLD: %08x\r\n", ref);
 #endif
+            // need to free the memory allocated for the record
+            platform_free(ptr);
 
             NANOCLR_DEBUG_STOP();
         }
@@ -247,6 +254,9 @@ bool CLR_RT_GarbageCollector::TestPointers_PopulateOld_Worker(void **ref)
             CLR_Debug::Printf("Some data points into a free list: %08x\r\n", dst);
 #endif
 
+            // need to free the memory allocated for the record
+            platform_free(ptr);
+
             NANOCLR_DEBUG_STOP();
         }
     }
@@ -263,7 +273,8 @@ void CLR_RT_GarbageCollector::TestPointers_PopulateOld()
     {
         RelocationRecord *ptr = *itLst;
 
-        delete ptr;
+        // need to free the memory allocated for the record
+        platform_free(ptr);
     }
 
     s_lstRecords.clear();