Rewrite SamplingCircularQueue

src/allocation.h

 // Copyright 2012 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 12 matching lines @@
 // DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
 // THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
 // (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
 // OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 
 #ifndef V8_ALLOCATION_H_
 #define V8_ALLOCATION_H_
 
 #include "globals.h"
 
+/* Round up n to be a multiple of sz, where sz is a power of 2. */
+#define ROUND_UP(n, sz) (((n) + ((sz) - 1)) & ~((sz) - 1))
+
+// Specify memory alignment for structs, classes, etc.

[Benedikt Meurer, 2013/08/14 11:38:09]

Please move the ALIGN_AT definition to globals.h, near OFFSET_OF maybe.

[yurys, 2013/08/14 13:19:01]

Done.

+// Use like:
+//   class ALIGN_AT(16) MyClass { ... }
+//   ALIGN_AT(16) int array[4];
+#if defined(_MSC_VER)
+#define ALIGN_AT(byte_alignment) __declspec(align(byte_alignment))
+#elif defined(__GNUC__) || defined(__APPLE__)

[Benedikt Meurer, 2013/08/14 11:38:09]

Why __APPLE__ here? This is actually a compiler thing and therefore unrelated to
the operating system (and there's only GCC and clang on OS X anyway, which both
define __GNUC__).

[yurys, 2013/08/14 13:19:01]

Done. Removed __APPLE__.

+#define ALIGN_AT(byte_alignment) __attribute__((aligned(byte_alignment)))
+#else
+#error "ALIGN_AT is not supported for your compiler"

[Benedikt Meurer, 2013/08/14 11:38:09]

Since the alignment is an optimization, I think it's fine to just #define
ALIGN_AT(x) in this case.

[yurys, 2013/08/14 13:19:01]

Done.

+#endif
+
 namespace v8 {
 namespace internal {
 
 // Called when allocation routines fail to allocate.
 // This function should not return, but should terminate the current
 // processing.
 void FatalProcessOutOfMemory(const char* message);
 
 // Superclass for classes managed with new & delete.
 class Malloced {
@@ skipping 90 matching lines @@
 
 struct PreallocatedStorageAllocationPolicy {
   INLINE(void* New(size_t size));
   INLINE(static void Delete(void* ptr));
 };
 
 
 } }  // namespace v8::internal
 
 #endif  // V8_ALLOCATION_H_