[modules] Don't force context allocation of variables declared in module scope.

src/ast/scopes.cc

Index: src/ast/scopes.cc
diff --git a/src/ast/scopes.cc b/src/ast/scopes.cc
index 2acf68f5e3cf884e4f3060b75f5fd2a4f68c5080..4e107bd3fdab1a258a5ff747ebc63893094cd5e8 100644
--- a/src/ast/scopes.cc
+++ b/src/ast/scopes.cc
@@ -1491,7 +1491,7 @@ bool Scope::MustAllocate(Variable* var) {
   if ((var->is_this() || !var->raw_name()->IsEmpty()) &&
       (var->has_forced_context_allocation() || scope_calls_eval_ ||
        inner_scope_calls_eval_ || is_catch_scope() || is_block_scope() ||
-       is_module_scope() || is_script_scope())) {
+       is_script_scope())) {
     var->set_is_used();
     if (scope_calls_eval_ || inner_scope_calls_eval_) var->set_maybe_assigned();
   }
@@ -1512,7 +1512,7 @@ bool Scope::MustAllocateInContext(Variable* var) {
   // always context-allocated.
   if (has_forced_context_allocation()) return true;
   if (var->mode() == TEMPORARY) return false;
-  if (is_catch_scope() || is_module_scope()) return true;
+  if (is_catch_scope()) return true;
   if (is_script_scope() && IsLexicalVariableMode(var->mode())) return true;
   return var->has_forced_context_allocation() || scope_calls_eval_ ||
          inner_scope_calls_eval_;