Simplfy getExpressionName (#3522)

This used to return a simple name like "if" for an If, but it is redundant with
our proper printing logic. This PR turns it into a trivial helper that just prints
out the name of the class, so it now prints "If" with a capital. That is useful
for some logging, like in Metrics I think it is clearer than it was earlier (since
we are actually counting the classes, and our old emitting of text-format-like
names are just confusing, as we emitted "binary" there which is not valid).

Also replace some usages of that method with proper printing.

1 files changed, 1 insertions, 1 deletions

diff --git a/src/wasm/wasm-emscripten.cpp b/src/wasm/wasm-emscripten.cpp
index 2e3a5cf23..8f0d4f9c8 100644
--- a/src/wasm/wasm-emscripten.cpp
+++ b/src/wasm/wasm-emscripten.cpp
@@ -310,7 +310,7 @@ void AsmConstWalker::visitCall(Call* curr) {
       }
     }
 
-    Fatal() << "Unexpected arg0 type (" << getExpressionName(arg)
+    Fatal() << "Unexpected arg0 type (" << *arg
             << ") in call to: " << importName;
   }