Oilpan: Support polymorphic objects in HeapVectorBackings

Source/wtf/Vector.h

 /*
  *  Copyright (C) 2005, 2006, 2007, 2008 Apple Inc. All rights reserved.
  *
  *  This library is free software; you can redistribute it and/or
  *  modify it under the terms of the GNU Library General Public
  *  License as published by the Free Software Foundation; either
  *  version 2 of the License, or (at your option) any later version.
  *
  *  This library is distributed in the hope that it will be useful,
  *  but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 72 matching lines @@
         static void clear(T*, T*) { }
 #if ENABLE(ASSERT)
         static void checkCleared(const T*, const T*) { }
 #endif
     };
 
     template<typename T>
     struct VectorUnusedSlotClearer<true, T> {
         static void clear(T* begin, T* end)
         {
-            memset(begin, 0, sizeof(T) * (end - begin));
+            memset(reinterpret_cast<void*>(begin), 0, sizeof(T) * (end - begin));

[tkent, 2015/05/07 00:43:21]

Why do you do reinterpret_cast<>?

[haraken, 2015/05/07 00:49:40]

Because without the reinterpret_cast, clang complains that we should not use
memset for slots that contain vtables. We use this technique to avoid the
compile error in some places in platform/heap/.

[tkent, 2015/05/08 06:42:00]

Thanks.  lgtm

         }
 
 #if ENABLE(ASSERT)
         static void checkCleared(const T* begin, const T* end)
         {
             const unsigned char* unusedArea = reinterpret_cast<const unsigned char*>(begin);
             const unsigned char* endAddress = reinterpret_cast<const unsigned char*>(end);
             ASSERT(endAddress >= unusedArea);
             for (int i = 0; i < endAddress - unusedArea; ++i)
                 ASSERT(!unusedArea[i]);
@@ skipping 1219 matching lines @@
     struct NeedsTracing<Vector<T, N>> {
         static const bool value = false;
     };
 #endif
 
 } // namespace WTF
 
 using WTF::Vector;
 
 #endif // WTF_Vector_h