Updated native memory allocation profiling to use its own sample buffer instead of sharing a sample…

runtime/vm/malloc_hooks_tcmalloc.cc

 // Copyright (c) 2017, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #include "platform/globals.h"
 
 #if defined(DART_USE_TCMALLOC) && !defined(PRODUCT)
 
 #include "vm/malloc_hooks.h"
 
@@ skipping 125 matching lines @@
     // the first OSThread in order to prevent deadlocks.
     if (MallocHooksState::ProfilingEnabled() &&
         MallocHooksState::stack_trace_collection_enabled()) {
       sample_ = Profiler::SampleNativeAllocation(kSkipCount, address,
                                                  allocation_size);
       ASSERT((sample_ == NULL) ||
              (sample_->native_allocation_address() == address_));
     }
   }
 
+  ~AllocationInfo() {
+    if (sample_ != NULL) {
+      Profiler::allocation_sample_buffer()->FreeAllocationSample(sample_);
+    }
+  }
+
   Sample* sample() const { return sample_; }
   intptr_t allocation_size() const { return allocation_size_; }
 
  private:
   // Note: sample_ is not owned by AllocationInfo, but by the SampleBuffer
   // created by the profiler. As such, this is only here to track if the sample
   // is still associated with a native allocation, and its fields are never
   // accessed from this class.
   Sample* sample_;
   uword address_;
@@ skipping 282 matching lines @@
       const bool result = MallocHooksState::address_map()->Remove(ptr);
       ASSERT(result);
       delete allocation_info;
     }
   }
 }
 
 }  // namespace dart
 
 #endif  // defined(DART_USE_TCMALLOC) && !defined(PRODUCT)