fix(es/minifier): Remove unnecessary check when invoke IIFE (#10257)

**Description:**

The original code explicitly disables invoking IIFE if there's a param's method is called. However, I don't think we need that check.


**Related issue:**

 - Closes #10255

Author: Austaras

crates/swc/tests/libs-size.snapshot.md

@@ -1,14 +1,14 @@
 | File | Original Size | Compressed Size | Gzipped Size |
 | --- | --- | --- | --- |
 | antd.js | 6.38 MiB | 2.05 MiB | 444.48 KiB |
-| d3.js | 542.74 KiB | 259.01 KiB | 85.06 KiB |
-| echarts.js | 3.41 MiB | 971.23 KiB | 313.32 KiB |
+| d3.js | 542.74 KiB | 259.00 KiB | 85.06 KiB |
+| echarts.js | 3.41 MiB | 971.04 KiB | 313.32 KiB |
 | jquery.js | 280.89 KiB | 87.11 KiB | 30.15 KiB |
 | lodash.js | 531.35 KiB | 68.18 KiB | 24.52 KiB |
 | moment.js | 169.83 KiB | 57.03 KiB | 18.21 KiB |
 | react.js | 70.45 KiB | 22.28 KiB | 8.00 KiB |
-| terser.js | 1.08 MiB | 444.89 KiB | 120.44 KiB |
-| three.js | 1.19 MiB | 627.93 KiB | 154.56 KiB |
-| typescript.js | 10.45 MiB | 3.17 MiB | 845.82 KiB |
+| terser.js | 1.08 MiB | 444.88 KiB | 120.45 KiB |
+| three.js | 1.19 MiB | 627.90 KiB | 154.58 KiB |
+| typescript.js | 10.45 MiB | 3.17 MiB | 846.03 KiB |
 | victory.js | 2.30 MiB | 689.65 KiB | 153.64 KiB |
 | vue.js | 334.13 KiB | 112.95 KiB | 41.69 KiB |

crates/swc/tests/tsc-references/contextuallyTypedIife.2.minified.js

@@ -35,10 +35,8 @@
         u: 23
     };
     _ref.u;
-}(), function(o) {
-    o.a(11);
-}({
+}(), ({
     a: function(n) {
         return n;
     }
-});
+}).a(11);

crates/swc/tests/tsc-references/contextuallyTypedIifeStrict.2.minified.js

@@ -35,10 +35,8 @@
         u: 23
     };
     _ref.u;
-}(), function(o) {
-    o.a(11);
-}({
+}(), ({
     a: function(n) {
         return n;
     }
-});
+}).a(11);

crates/swc/tests/tsc-references/importCallExpression2ES2020.2.minified.js

@@ -5,8 +5,6 @@ export class B {
     }
 }
 //// [2.ts]
-!function(x) {
-    x.then((value)=>{
-        new value.B().print();
-    });
-}(import("./0"));
+import("./0").then((value)=>{
+    new value.B().print();
+});

crates/swc/tests/tsc-references/importCallExpressionInAMD2.2.minified.js

@@ -25,11 +25,9 @@ define([
 ], function(require, exports, _interop_require_wildcard) {
     Object.defineProperty(exports, "__esModule", {
         value: !0
-    }), function(x) {
-        x.then((value)=>{
-            new value.B().print();
-        });
-    }(new Promise((resolve, reject)=>require([
+    }), new Promise((resolve, reject)=>require([
             "./0"
-        ], (m)=>resolve(/*#__PURE__*/ _interop_require_wildcard._(m)), reject)));
+        ], (m)=>resolve(/*#__PURE__*/ _interop_require_wildcard._(m)), reject)).then((value)=>{
+        new value.B().print();
+    });
 });

crates/swc/tests/tsc-references/importCallExpressionInCJS3.2.minified.js

@@ -17,8 +17,6 @@ Object.defineProperty(exports, "__esModule", {
     value: !0
 });
 const _interop_require_wildcard = require("@swc/helpers/_/_interop_require_wildcard");
-!function(x) {
-    x.then((value)=>{
-        new value.B().print();
-    });
-}(Promise.resolve().then(()=>/*#__PURE__*/ _interop_require_wildcard._(require("./0"))));
+Promise.resolve().then(()=>/*#__PURE__*/ _interop_require_wildcard._(require("./0"))).then((value)=>{
+    new value.B().print();
+});

crates/swc/tests/tsc-references/importCallExpressionInSystem2.2.minified.js

@@ -16,11 +16,9 @@ System.register([], function(_export, _context) {
     return {
         setters: [],
         execute: function() {
-            !function(x) {
-                x.then((value)=>{
-                    new value.B().print();
-                });
-            }(_context.import("./0"));
+            _context.import("./0").then((value)=>{
+                new value.B().print();
+            });
         }
     };
 });

crates/swc/tests/tsc-references/importCallExpressionInUMD2.2.minified.js

@@ -22,11 +22,9 @@ var global, factory;
 global = this, factory = function(exports1, _interop_require_wildcard) {
     Object.defineProperty(exports1, "__esModule", {
         value: !0
-    }), function(x) {
-        x.then((value)=>{
-            new value.B().print();
-        });
-    }(import("./0"));
+    }), import("./0").then((value)=>{
+        new value.B().print();
+    });
 }, "object" == typeof module && "object" == typeof module.exports ? factory(exports, require("@swc/helpers/_/_interop_require_wildcard")) : "function" == typeof define && define.amd ? define([
     "exports",
     "@swc/helpers/_/_interop_require_wildcard"

crates/swc_ecma_minifier/src/compress/optimize/iife.rs

@@ -12,7 +12,7 @@ use crate::debug::dump;
 use crate::{
     compress::optimize::Ctx,
     program_data::{ProgramData, ScopeData},
-    util::{idents_captured_by, idents_used_by, make_number},
+    util::{idents_captured_by, make_number},
 };
 
 /// Methods related to the option `negate_iife`.
@@ -868,28 +868,6 @@ impl Optimizer<'_> {
                 Stmt::Expr(e) => match &*e.expr {
                     Expr::Await(..) => false,
 
-                    // TODO: Check if parameter is used and inline if call is not related to
-                    // parameters.
-                    Expr::Call(e) => {
-                        if e.callee.as_expr().and_then(|e| e.as_ident()).is_some() {
-                            return true;
-                        }
-
-                        let used = idents_used_by(&e.callee);
-
-                        if used.iter().all(|id| {
-                            self.data
-                                .vars
-                                .get(id)
-                                .map(|usage| usage.ref_count == 1 && usage.callee_count > 0)
-                                .unwrap_or(false)
-                        }) {
-                            return true;
-                        }
-
-                        param_ids.iter().all(|param| !used.contains(&param.to_id()))
-                    }
-
                     _ => !stmt.is_use_strict(),
                 },
 

crates/swc_ecma_minifier/tests/benches-full/d3.js

@@ -2699,9 +2699,7 @@ function(global, factory) {
             }
             function ended(event) {
                 var extent;
-                if (!function(event) {
-                    event.stopImmediatePropagation();
-                }(event), event.touches) {
+                if (event.stopImmediatePropagation(), event.touches) {
                     if (event.touches.length) return;
                     touchending && clearTimeout(touchending), touchending = setTimeout(function() {
                         touchending = null;