[module] Track script "module code" status

src/compiler.cc

Index: src/compiler.cc
diff --git a/src/compiler.cc b/src/compiler.cc
index 055777af77db2118a08e06d68cb7e798535dc4e6..5eef4e8e943d866238077696dc7e0e0c561955d7 100644
--- a/src/compiler.cc
+++ b/src/compiler.cc
@@ -1522,16 +1522,12 @@ Handle<SharedFunctionInfo> Compiler::GetSharedFunctionInfoForScript(
     if (!source_map_url.is_null()) {
       script->set_source_mapping_url(*source_map_url);
     }
+    script->set_is_module(is_module);
 
     // Compile the function and add it to the cache.
     Zone zone(isolate->allocator());
     ParseInfo parse_info(&zone, script);
     CompilationInfo info(&parse_info, Handle<JSFunction>::null());
-    if (is_module) {
-      parse_info.set_module();
-    } else {
-      parse_info.set_global();
-    }

[vogelheim, 2016/06/14 15:20:49]

You've moved this into the ParseInfo constructor. That's a good idea, but I
don't think this actually works. The thing is, a parse can be one either an
eval, or global, or a module [1]. *In* *this* *context* it cannot be an eval -
because we're compiling a script. But in the context of a ParseInfo constructor
it could be either of those three. Which is why here, we can set_global() if
it's not a module, but in the ParseInfo constructor that no longer makes sense.

[1] 
https://cs.chromium.org/chromium/src/v8/src/compiler.cc?type=cs&sq=package:ch...


I guess I'd set_global if !is_module; maybe just replace the set_module with a
DCHECK(parse_info.is_module()). That's not super pretty code, but at least it
lets the reader know what the expectation is in either case.

[mike3, 2016/06/19 16:58:59]

Acknowledged.

     if (compile_options != ScriptCompiler::kNoCompileOptions) {
       parse_info.set_cached_data(cached_data);
     }