Refactor printing code so that printing Expressions always works (#3450)

This avoids needing to add include wasm-printing if a file doesn't already have it.
To achieve that, add the std::ostream hooks in wasm.h, and also use them
when possible, removing the need for the special WasmPrinter object.

Also stop printing in "full" (print types on each line) in error messages by default. The
user can still get that, as always, using BINARYEN_PRINT_FULL=1 in the env.

1 files changed, 2 insertions, 3 deletions

diff --git a/src/tools/wasm-opt.cpp b/src/tools/wasm-opt.cpp
index 56a26ccfb..8884619fe 100644
--- a/src/tools/wasm-opt.cpp
+++ b/src/tools/wasm-opt.cpp
@@ -34,7 +34,6 @@
 #include "wasm-binary.h"
 #include "wasm-interpreter.h"
 #include "wasm-io.h"
-#include "wasm-printing.h"
 #include "wasm-s-parser.h"
 #include "wasm-validator.h"
 #include "wasm2c-wrapper.h"
@@ -227,7 +226,7 @@ int main(int argc, const char* argv[]) {
     // to print would not be reached).
     if (std::find(options.passes.begin(), options.passes.end(), "print") !=
         options.passes.end()) {
-      WasmPrinter::printModule(&wasm);
+      std::cout << wasm << '\n';
     }
     Fatal() << message;
   };
@@ -279,7 +278,7 @@ int main(int argc, const char* argv[]) {
     reader.build();
     if (options.passOptions.validate) {
       if (!WasmValidator().validate(wasm)) {
-        WasmPrinter::printModule(&wasm);
+        std::cout << wasm << '\n';
         Fatal() << "error after translate-to-fuzz";
       }
     }