From 5f114452cd73fcad861660b2b715af726c925084 Mon Sep 17 00:00:00 2001
From: Alon Zakai <alonzakai@gmail.com>
Date: Wed, 9 Jan 2019 13:05:08 -0800
Subject: Aligned allocation fixes. Fixes #1845 (#1846)

The error in #1845 shows:

/<<PKGBUILDDIR>>/src/mixed_arena.h: In member function 'void* MixedArena::allocSpace(size_t, size_t)':
/<<PKGBUILDDIR>>/src/mixed_arena.h:125:43: error: 'new' of type 'MixedArena::Chunk' {aka 'std::aligned_storage<32768, 16>::type'} with extended alignment 16 [-Werror=aligned-new=]
       chunks.push_back(new Chunk[numChunks]);
                                           ^
/<<PKGBUILDDIR>>/src/mixed_arena.h:125:43: note: uses 'void* operator new [](std::size_t)', which does not have an alignment parameter
/<<PKGBUILDDIR>>/src/mixed_arena.h:125:43: note: use '-faligned-new' to enable C++17 over-aligned new support

It turns out I had misread the aligned_storage docs, and they don't actually do what we need, which is a convenient cross-platform way to do aligned allocation, since new itself doesn't support that. Sadly it seems there is no cross-platform way to do it right now, so I added a header in support which abstracts over the windows and everything-else ways.

Also add some ctest testing, which runs on windows, so we get basic windows coverage in our CI.
---
 src/mixed_arena.h | 17 +++++++++--------
 1 file changed, 9 insertions(+), 8 deletions(-)

(limited to 'src/mixed_arena.h')

diff --git a/src/mixed_arena.h b/src/mixed_arena.h
index 4c62514d1..46487b7fc 100644
--- a/src/mixed_arena.h
+++ b/src/mixed_arena.h
@@ -19,13 +19,14 @@
 
 #include <atomic>
 #include <cassert>
-#include <cstdlib>
 #include <memory>
 #include <mutex>
 #include <thread>
 #include <type_traits>
 #include <vector>
 
+#include <support/alloc.h>
+
 //
 // Arena allocation for mixed-type data.
 //
@@ -63,11 +64,9 @@ struct MixedArena {
   static const size_t CHUNK_SIZE = 32768;
   static const size_t MAX_ALIGN = 16; // allow 128bit SIMD
 
-  typedef std::aligned_storage<CHUNK_SIZE, MAX_ALIGN>::type Chunk;
-
-  // Each pointer in chunks is to an array of Chunk structs; typically 1,
+  // Each pointer in chunks is to a multiple of CHUNK_SIZE - typically 1,
   // but possibly more.
-  std::vector<Chunk*> chunks;
+  std::vector<void*> chunks;
 
   size_t index = 0; // in last chunk
 
@@ -122,10 +121,12 @@ struct MixedArena {
       // Allocate a new chunk.
       auto numChunks = (size + CHUNK_SIZE - 1) / CHUNK_SIZE;
       assert(size <= numChunks * CHUNK_SIZE);
-      chunks.push_back(new Chunk[numChunks]);
+      auto* allocation = wasm::aligned_malloc(MAX_ALIGN, numChunks * CHUNK_SIZE);
+      if (!allocation) abort();
+      chunks.push_back(allocation);
       index = 0;
     }
-    uint8_t* ret = static_cast<uint8_t*>(static_cast<void*>(chunks.back()));
+    uint8_t* ret = static_cast<uint8_t*>(chunks.back());
     ret += index;
     index += size; // TODO: if we allocated more than 1 chunk, reuse the remainder, right now we allocate another next time
     return static_cast<void*>(ret);
@@ -141,7 +142,7 @@ struct MixedArena {
 
   void clear() {
     for (auto* chunk : chunks) {
-      delete[] chunk;
+      wasm::aligned_free(chunk);
     }
     chunks.clear();
   }
-- 
cgit v1.2.3