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

src/ast/scopes.h

Index: src/ast/scopes.h
diff --git a/src/ast/scopes.h b/src/ast/scopes.h
index 5be698fc1027b767726716467f95da37f142a5db..dec99933bae92d03d94a8b08c8ee2bf74b802f03 100644
--- a/src/ast/scopes.h
+++ b/src/ast/scopes.h
@@ -331,10 +331,9 @@ class Scope: public ZoneObject {
 
   // Whether this needs to be represented by a runtime context.
   bool NeedsContext() const {
-    // Catch and module scopes always have heap slots.
+    // Catch scopes always have heap slots.
     DCHECK(!is_catch_scope() || num_heap_slots() > 0);
-    DCHECK(!is_module_scope() || num_heap_slots() > 0);
-    return is_with_scope() || num_heap_slots() > 0;
+    return num_heap_slots() > 0;
   }
 
   // ---------------------------------------------------------------------------