fix: do not inline regexp literal when declaration in loop

JLHwung · JLHwung · commit f64e9066ed05 · 2022-05-04T15:36:40.000-04:00
diff --git a/packages/babel-plugin-minify-dead-code-elimination/__tests__/fixtures/issue-981/actual.js b/packages/babel-plugin-minify-dead-code-elimination/__tests__/fixtures/issue-981/actual.js
@@ -0,0 +1,5 @@
+function f(query) {
+  var r = /./g,
+    part;
+  while (part = r.exec(query));
+}
diff --git a/packages/babel-plugin-minify-dead-code-elimination/__tests__/fixtures/issue-981/expected.js b/packages/babel-plugin-minify-dead-code-elimination/__tests__/fixtures/issue-981/expected.js
@@ -0,0 +1,6 @@
+function f(query) {
+  var r = /./g,
+      part;
+
+  while (part = r.exec(query));
+}
diff --git a/packages/babel-plugin-minify-dead-code-elimination/src/index.js b/packages/babel-plugin-minify-dead-code-elimination/src/index.js
@@ -421,7 +421,8 @@ module.exports = ({ types: t, traverse }) => {
               const isObj = n =>
                 t.isFunction(n) ||
                 t.isObjectExpression(n) ||
-                t.isArrayExpression(n);
+                t.isArrayExpression(n) ||
+                t.isRegExpLiteral(n);
 
               const isReplacementObj =
                 isObj(replacement) || some(replacement, isObj);
