Mark evalScopes for DCE (Fix #366)

As DCE is a separate pass and we recalculate path and scope by clearing the path and scope caches, we need to mark evalscopes again for DCE
babel · Feb 18, 2017 · 9ebc3e1 · 9ebc3e1
1 parent 31abdc2
commit 9ebc3e1
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/packages/babel-plugin-minify-dead-code-elimination/src/index.js b/packages/babel-plugin-minify-dead-code-elimination/src/index.js
@@ -1,7 +1,7 @@
 "use strict";
 
 const some = require("lodash.some");
-const { markEvalScopes, isMarked: isEvalScopesMarked, hasEval } = require("babel-helper-mark-eval-scopes");
+const { markEvalScopes, hasEval } = require("babel-helper-mark-eval-scopes");
 
 function prevSiblings(path) {
   const parentPath = path.parentPath;
@@ -773,9 +773,7 @@ module.exports = ({ types: t, traverse }) => {
           traverse.clearCache();
           path.scope.crawl();
 
-          if (!isEvalScopesMarked(path.scope)) {
-            markEvalScopes(path);
-          }
+          markEvalScopes(path);
 
           // We need to run this plugin in isolation.
           path.traverse(main, {