allocate only expressions in arenas - functions, imports, exports, function types, can more simply be held by unique_ptrs on the owning module. this avoids need to coordinate arena allocation for their elements, and only the far more plentiful expression nodes are a perf factor anyhow

1 files changed, 4 insertions, 4 deletions

diff --git a/src/passes/Print.cpp b/src/passes/Print.cpp
index eb0a6d33b..256968bb7 100644
--- a/src/passes/Print.cpp
+++ b/src/passes/Print.cpp
@@ -522,17 +522,17 @@ struct PrintSExpression : public Visitor<PrintSExpression> {
     }
     for (auto& child : curr->functionTypes) {
       doIndent(o, indent);
-      visitFunctionType(child, true);
+      visitFunctionType(child.get(), true);
       o << maybeNewLine;
     }
     for (auto& child : curr->imports) {
       doIndent(o, indent);
-      visitImport(child);
+      visitImport(child.get());
       o << maybeNewLine;
     }
     for (auto& child : curr->exports) {
       doIndent(o, indent);
-      visitExport(child);
+      visitExport(child.get());
       o << maybeNewLine;
     }
     if (curr->table.names.size() > 0) {
@@ -542,7 +542,7 @@ struct PrintSExpression : public Visitor<PrintSExpression> {
     }
     for (auto& child : curr->functions) {
       doIndent(o, indent);
-      visitFunction(child);
+      visitFunction(child.get());
       o << maybeNewLine;
     }
     decIndent();