We can get away with passing memory_view on the stack here...

1 files changed, 9 insertions, 4 deletions

diff --git a/system/include/emscripten/wire.h b/system/include/emscripten/wire.h
index 32080e54..a5892216 100644
--- a/system/include/emscripten/wire.h
+++ b/system/include/emscripten/wire.h
@@ -377,11 +377,16 @@ namespace emscripten {
     namespace internal {
         template<>
         struct BindingType<memory_view> {
-            typedef memory_view* WireType;
+            // This non-word-sized WireType only works because I
+            // happen to know that clang will pass aggregates as
+            // pointers to stack elements and we never support
+            // converting JavaScript typed arrays back into
+            // memory_view.  (That is, fromWireType is not implemented
+            // on the C++ side, nor is toWireType implemented in
+            // JavaScript.)
+            typedef memory_view WireType;
             static WireType toWireType(const memory_view& mv) {
-                WireType wt = (WireType)malloc(sizeof(memory_view));
-                new(wt) memory_view(mv);
-                return wt;
+                return mv;
             }
         };
     }