Merge pull request #1086 from bloomberg/sub_module

calculate maybe depends as early as we can in case later optimizations remove effectful modules

Author: bobzhang

jscomp/Makefile

@@ -217,11 +217,15 @@ SYNTAX_SRCS=ast_derive_constructor ast_derive_util ast_exp ast_external ast_lift
 SYNTAX_CMXS=$(addprefix syntax/, $(addsuffix .cmx, $(SYNTAX_SRCS)))
 DEPENDS_SRCS= depends_post_process bs_exception ast_extract binary_ast  
 DEPENDS_CMXS=$(addprefix depends/, $(addsuffix .cmx, $(DEPENDS_SRCS)))
-CORE_SRCS= type_int_to_string type_util  ocaml_stdlib_slots bs_conditional_initial ocaml_options ocaml_parse lam\
+CORE_SRCS= type_int_to_string type_util  ocaml_stdlib_slots bs_conditional_initial ocaml_options ocaml_parse\
+	js_op\
+	lam_module_ident\
+	lam\
 	lam_print lam_beta_reduce_util lam_inline_util lam_analysis\
 	lam_closure\
-	js_cmj_format js_fun_env js_call_info js_closure js_op js_number js_cmj_datasets\
-	lam_exit_code j lam_module_ident lam_compile_util lam_stats config_util lam_compile_defs js_map js_fold js_fold_basic js_pass_scope\
+	js_cmj_format js_fun_env js_call_info js_closure  js_number js_cmj_datasets\
+	lam_exit_code j \
+	lam_compile_util lam_stats config_util lam_compile_defs js_map js_fold js_fold_basic js_pass_scope\
 	js_op_util js_analyzer js_shake js_exp_make js_long js_of_lam_exception js_of_lam_module js_of_lam_array js_of_lam_block js_of_lam_string\
 	js_of_lam_tuple js_of_lam_record js_of_lam_float_record js_arr lam_compile_const \
 	lam_inner lam_util \

jscomp/all.depend

@@ -234,7 +234,9 @@ core/type_util.cmi :
 core/bs_conditional_initial.cmi :
 core/ocaml_options.cmi :
 core/ocaml_parse.cmi :
-core/lam.cmi : ext/ident_set.cmi
+core/lam_module_ident.cmi : core/js_op.cmx common/js_config.cmi core/j.cmx \
+    ext/hashtbl_gen.cmx ext/hash_set_gen.cmx
+core/lam.cmi : core/lam_module_ident.cmi ext/ident_set.cmi
 core/lam_print.cmi : core/lam.cmi
 core/lam_beta_reduce_util.cmi : core/lam.cmi
 core/lam_inline_util.cmi : core/lam.cmi
@@ -248,11 +250,9 @@ core/js_closure.cmi : ext/ident_set.cmi
 core/js_number.cmi :
 core/js_cmj_datasets.cmi : ext/string_map.cmi core/js_cmj_format.cmi
 core/lam_exit_code.cmi : core/lam.cmi
-core/lam_module_ident.cmi : core/js_op.cmx common/js_config.cmi core/j.cmx \
-    ext/hashtbl_gen.cmx ext/hash_set_gen.cmx
 core/lam_compile_util.cmi : core/js_op.cmx
-core/lam_stats.cmi : core/lam_module_ident.cmi core/lam.cmi \
-    ext/int_hash_set.cmi ext/ident_set.cmi ext/ident_hashtbl.cmi
+core/lam_stats.cmi : core/lam.cmi ext/int_hash_set.cmi ext/ident_set.cmi \
+    ext/ident_hashtbl.cmi
 core/config_util.cmi : core/js_cmj_format.cmi
 core/lam_compile_defs.cmi : core/lam_stats.cmi core/j.cmx ext/ident_map.cmi
 core/js_fold_basic.cmi : core/lam_module_ident.cmi core/j.cmx \
@@ -327,12 +327,16 @@ core/bs_conditional_initial.cmx : common/bs_version.cmx \
 core/ocaml_options.cmx : common/bs_version.cmx core/ocaml_options.cmi
 core/ocaml_parse.cmx : syntax/ppx_entry.cmx common/js_config.cmx \
     core/ocaml_parse.cmi
+core/js_op.cmx :
+core/lam_module_ident.cmx : core/js_op.cmx common/js_config.cmx core/j.cmx \
+    ext/hashtbl_make.cmx ext/hash_set.cmx ext/ext_ident.cmx \
+    stubs/bs_hash_stubs.cmx core/lam_module_ident.cmi
 core/lam.cmx : ext/ordered_hash_map_local_ident.cmx \
-    core/ocaml_stdlib_slots.cmx ext/literals.cmx common/js_config.cmx \
-    ext/int_vec_vec.cmx ext/int_vec_util.cmx ext/int_vec.cmx \
-    ext/ident_set.cmx ext/ident_hashtbl.cmx ext/ident_hash_set.cmx \
-    ext/hash_set_ident_mask.cmx ext/ext_string.cmx ext/ext_scc.cmx \
-    core/lam.cmi
+    core/ocaml_stdlib_slots.cmx ext/literals.cmx core/lam_module_ident.cmx \
+    common/js_config.cmx ext/int_vec_vec.cmx ext/int_vec_util.cmx \
+    ext/int_vec.cmx ext/ident_set.cmx ext/ident_hashtbl.cmx \
+    ext/ident_hash_set.cmx ext/hash_set_ident_mask.cmx ext/ext_string.cmx \
+    ext/ext_scc.cmx core/lam.cmi
 core/lam_print.cmx : core/lam.cmx core/lam_print.cmi
 core/lam_beta_reduce_util.cmx : core/lam.cmx ext/ident_hashtbl.cmx \
     core/lam_beta_reduce_util.cmi
@@ -345,20 +349,15 @@ core/js_cmj_format.cmx : ext/string_map.cmx core/lam.cmx \
 core/js_fun_env.cmx : ext/ident_set.cmx ext/ext_list.cmx core/js_fun_env.cmi
 core/js_call_info.cmx : core/js_call_info.cmi
 core/js_closure.cmx : ext/ident_set.cmx core/js_closure.cmi
-core/js_op.cmx :
 core/js_number.cmx : core/js_number.cmi
 core/js_cmj_datasets.cmx : ext/string_map.cmx core/js_cmj_format.cmx \
     core/js_cmj_datasets.cmi
 core/lam_exit_code.cmx : core/lam.cmx core/lam_exit_code.cmi
 core/j.cmx : core/js_op.cmx core/js_fun_env.cmx core/js_closure.cmx \
     core/js_call_info.cmx ext/ident_set.cmx
-core/lam_module_ident.cmx : core/js_op.cmx common/js_config.cmx core/j.cmx \
-    ext/hashtbl_make.cmx ext/hash_set.cmx ext/ext_ident.cmx \
-    stubs/bs_hash_stubs.cmx core/lam_module_ident.cmi
 core/lam_compile_util.cmx : core/js_op.cmx core/lam_compile_util.cmi
-core/lam_stats.cmx : core/lam_module_ident.cmx core/lam.cmx \
-    ext/int_hash_set.cmx ext/ident_set.cmx ext/ident_hashtbl.cmx \
-    core/lam_stats.cmi
+core/lam_stats.cmx : core/lam.cmx ext/int_hash_set.cmx ext/ident_set.cmx \
+    ext/ident_hashtbl.cmx core/lam_stats.cmi
 core/config_util.cmx : core/js_cmj_format.cmx depends/bs_exception.cmx \
     core/config_util.cmi
 core/lam_compile_defs.cmx : core/lam_stats.cmx core/j.cmx ext/ident_map.cmx \
@@ -403,10 +402,9 @@ core/lam_compile_const.cmx : core/lam_compile_util.cmx \
     core/lam_compile_const.cmi
 core/lam_inner.cmx : core/lam.cmx core/lam_inner.cmi
 core/lam_util.cmx : ext/literals.cmx core/lam_stats.cmx core/lam_print.cmx \
-    core/lam_module_ident.cmx core/lam_analysis.cmx core/lam.cmx \
-    common/js_config.cmx ext/ident_set.cmx ext/ident_map.cmx \
-    ext/ident_hashtbl.cmx ext/ext_list.cmx ext/ext_filename.cmx \
-    ext/ext_array.cmx core/lam_util.cmi
+    core/lam_analysis.cmx core/lam.cmx common/js_config.cmx ext/ident_set.cmx \
+    ext/ident_map.cmx ext/ident_hashtbl.cmx ext/ext_list.cmx \
+    ext/ext_filename.cmx ext/ext_array.cmx core/lam_util.cmi
 core/lam_group.cmx : core/lam_print.cmx core/lam.cmx core/lam_group.cmi
 core/lam_pass_deep_flatten.cmx : core/lam_util.cmx core/lam_group.cmx \
     core/lam.cmx core/js_number.cmx ext/ident_set.cmx \

jscomp/bin/whole_compiler.d

@@ -163,8 +163,22 @@ bin/whole_compiler.ml : ext/ident_set.ml
 bin/whole_compiler.ml : ext/ident_set.mli
 bin/whole_compiler.ml : ext/int_vec_util.ml
 bin/whole_compiler.ml : ext/int_vec_util.mli
+bin/whole_compiler.ml : ext/hash_set.ml
+bin/whole_compiler.ml : ext/hash_set.mli
+bin/whole_compiler.ml : ext/hashtbl_make.ml
+bin/whole_compiler.ml : ext/hashtbl_make.mli
+bin/whole_compiler.ml : core/js_call_info.ml
+bin/whole_compiler.ml : core/js_call_info.mli
+bin/whole_compiler.ml : core/js_closure.ml
+bin/whole_compiler.ml : core/js_closure.mli
+bin/whole_compiler.ml : core/js_fun_env.ml
+bin/whole_compiler.ml : core/js_fun_env.mli
 bin/whole_compiler.ml : ../ocaml/bytecomp/lambda.ml
 bin/whole_compiler.ml : ../ocaml/bytecomp/lambda.mli
+bin/whole_compiler.ml : core/js_op.ml
+bin/whole_compiler.ml : core/j.ml
+bin/whole_compiler.ml : core/lam_module_ident.ml
+bin/whole_compiler.ml : core/lam_module_ident.mli
 bin/whole_compiler.ml : core/ocaml_stdlib_slots.ml
 bin/whole_compiler.ml : ext/ordered_hash_map_gen.ml
 bin/whole_compiler.ml : ext/ordered_hash_map_local_ident.ml
@@ -175,14 +189,6 @@ bin/whole_compiler.ml : core/js_cmj_format.ml
 bin/whole_compiler.ml : core/js_cmj_format.mli
 bin/whole_compiler.ml : core/config_util.ml
 bin/whole_compiler.ml : core/config_util.mli
-bin/whole_compiler.ml : core/js_call_info.ml
-bin/whole_compiler.ml : core/js_call_info.mli
-bin/whole_compiler.ml : core/js_closure.ml
-bin/whole_compiler.ml : core/js_closure.mli
-bin/whole_compiler.ml : core/js_fun_env.ml
-bin/whole_compiler.ml : core/js_fun_env.mli
-bin/whole_compiler.ml : core/js_op.ml
-bin/whole_compiler.ml : core/j.ml
 bin/whole_compiler.ml : core/js_fold.ml
 bin/whole_compiler.ml : core/js_analyzer.ml
 bin/whole_compiler.ml : core/js_analyzer.mli
@@ -196,12 +202,6 @@ bin/whole_compiler.ml : ext/ident_map.ml
 bin/whole_compiler.ml : ext/ident_map.mli
 bin/whole_compiler.ml : core/lam_analysis.ml
 bin/whole_compiler.ml : core/lam_analysis.mli
-bin/whole_compiler.ml : ext/hash_set.ml
-bin/whole_compiler.ml : ext/hash_set.mli
-bin/whole_compiler.ml : ext/hashtbl_make.ml
-bin/whole_compiler.ml : ext/hashtbl_make.mli
-bin/whole_compiler.ml : core/lam_module_ident.ml
-bin/whole_compiler.ml : core/lam_module_ident.mli
 bin/whole_compiler.ml : core/lam_print.ml
 bin/whole_compiler.ml : core/lam_print.mli
 bin/whole_compiler.ml : ext/ext_int.ml