Remove renameMainArgcArgv from wasm-emscripten-finalize (#4700)

This part to finalize is currently not used and was added in preparation
for https://reviews.llvm.org/D75277.

However, the better solution to dealing with this alternative name for
main is on the emscripten side.  The main reason for this is that
doing the rename here in binaryen would require finalize to always
re-write the binary, which is expensive.

1 files changed, 0 insertions, 6 deletions

diff --git a/src/tools/wasm-emscripten-finalize.cpp b/src/tools/wasm-emscripten-finalize.cpp
index 8fdc203a9..b4e27ca5c 100644
--- a/src/tools/wasm-emscripten-finalize.cpp
+++ b/src/tools/wasm-emscripten-finalize.cpp
@@ -266,12 +266,6 @@ int main(int argc, const char* argv[]) {
   generator.onlyI64DynCalls = onlyI64DynCalls;
   generator.noDynCalls = noDynCalls;
 
-  if (!standaloneWasm) {
-    // This is also not needed in standalone mode since standalone mode uses
-    // crt1.c to invoke the main and is aware of __main_argc_argv mangling.
-    generator.renameMainArgcArgv();
-  }
-
   PassRunner passRunner(&wasm, options.passOptions);
   passRunner.setDebug(options.debug);
   passRunner.setDebugInfo(debugInfo);