From 81b8497164cff31af3bbd66f8b2aee55503b74cb Mon Sep 17 00:00:00 2001 From: Sam Clegg Date: Fri, 22 Mar 2024 17:26:43 -0700 Subject: Remove "minimal" JS import/export legalization (#6428) This change removes the "minimal" mode from `LegalizeJSInterface` which was added in #1883. The idea behind this change was to avoid legalizing most function except those we know that JS will be calling. The idea was that for dynamic linking we always want the non-legalized version to be shared between wasm module. These days we solve this problem in a different way with the `legalize-js-interface-export-originals` which exports the original functions alongside the legalized ones. Emscripten then always prefers the `$orig` functions when doing dynamic linking. --- src/passes/pass.cpp | 4 ---- 1 file changed, 4 deletions(-) (limited to 'src/passes/pass.cpp') diff --git a/src/passes/pass.cpp b/src/passes/pass.cpp index 6faf77276..0955082ac 100644 --- a/src/passes/pass.cpp +++ b/src/passes/pass.cpp @@ -217,10 +217,6 @@ void PassRegistry::registerPasses() { registerPass("legalize-js-interface", "legalizes i64 types on the import/export boundary", createLegalizeJSInterfacePass); - registerPass("legalize-js-interface-minimally", - "legalizes i64 types on the import/export boundary in a minimal " - "manner, only on things only JS will call", - createLegalizeJSInterfaceMinimallyPass); registerPass("legalize-and-prune-js-interface", "legalizes the import/export boundary and prunes when needed", createLegalizeAndPruneJSInterfacePass); -- cgit v1.2.3